[tor-commits] [nyx/master] Only attempt to fetch manual information once

[atagar at torproject.org]

commit 61a2c4bd701802627ac3262b90342a9c07d3af66
Author: Damian Johnson <atagar at torproject.org>
Date:   Sun Aug 27 10:56:40 2017 -0700

    Only attempt to fetch manual information once
    
    No reason to repeatedly fetch information that doesn't exist.
---
 nyx/panel/config.py | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/nyx/panel/config.py b/nyx/panel/config.py
index db04dbb..5f09e66 100644
--- a/nyx/panel/config.py
+++ b/nyx/panel/config.py
@@ -54,6 +54,8 @@ class ConfigEntry(object):
   def __init__(self, name, value_type):
     self.name = name
     self.value_type = value_type
+
+    self._is_fetched = False
     self._category = None
     self._usage = None
     self._summary = None
@@ -133,27 +135,28 @@ class ConfigEntry(object):
 
   @property
   def summary(self):
-    if not self._summary:
+    if not self._is_fetched:
       self._fetch_attr()
 
     return self._summary
 
   @property
   def description(self):
-    if not self._description:
+    if not self._is_fetched:
       self._fetch_attr()
 
     return self._description
 
   @property
   def position(self):
-    if not self._position:
+    if not self._is_fetched:
       self._fetch_attr()
 
-    return self._position
+    return 99999 if self._position is None else self._position
 
   def _fetch_attr(self):
     result = stem.manual.query('SELECT category, usage, summary, description, position FROM torrc WHERE key=?', self.name.upper()).fetchone()
+    self._is_fetched = True
 
     if result:
       self._category, self._usage, self._summary, self._description, self._position = result