From dda18add9a4bd4688b842846020e1e4121d510d3 Mon Sep 17 00:00:00 2001
From: Ben Kurtovic <ben.kurtovic@gmail.com>
Date: Sun, 25 Dec 2016 03:44:21 -0500
Subject: [PATCH] Display empty member list correctly.

---
 calefaction/eve/esi.py         |  4 ++--
 templates/members/members.mako | 33 ++++++++++++++++++---------------
 2 files changed, 20 insertions(+), 17 deletions(-)

diff --git a/calefaction/eve/esi.py b/calefaction/eve/esi.py
index 9b12766..278c9d6 100644
--- a/calefaction/eve/esi.py
+++ b/calefaction/eve/esi.py
@@ -155,8 +155,8 @@ class EVESwaggerInterface:
             cached = None
 
         self._debug("[%s] [%s] %s", method.__name__.upper(),
-                    "cache" if cached else "fresh", query)
-        if cached:
+                    "fresh" if cached is None else "cache", query)
+        if cached is not None:
             return cached
 
         headers = {
diff --git a/templates/members/members.mako b/templates/members/members.mako
index 09031bf..374da37 100644
--- a/templates/members/members.mako
+++ b/templates/members/members.mako
@@ -1,17 +1,20 @@
 <%inherit file="../_default.mako"/>
 <h2>Members</h2>
-<!-- ... handle case where members is empty - means something went wrong -->
-<ul id="members-list">
-    % for member in members:
-        <li>
-            <a href="${g.eve.image.character(member.id, 1024)}">
-                <img class="styled-border" title="${member.name}" alt="${member.name}'s Portrait" src="${g.eve.image.character(member.id, 256)}"/>
-            </a>
-            % if member.roles:
-                <span>${member.name}<em>${member.roles}</em></span>
-            % else:
-                <span>${member.name}</span>
-            % endif
-        </li>
-    % endfor
-</ul>
+% if members:
+    <ul id="members-list">
+        % for member in members:
+            <li>
+                <a href="${g.eve.image.character(member.id, 1024)}">
+                    <img class="styled-border" title="${member.name}" alt="${member.name}'s Portrait" src="${g.eve.image.character(member.id, 256)}"/>
+                </a>
+                % if member.roles:
+                    <span>${member.name}<em>${member.roles}</em></span>
+                % else:
+                    <span>${member.name}</span>
+                % endif
+            </li>
+        % endfor
+    </ul>
+% else:
+    <p>You don't have access to the members list.</p>
+% endif