You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bloodhound.apache.org by as...@apache.org on 2013/05/30 19:46:11 UTC

svn commit: r1487947 - /bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py

Author: astaric
Date: Thu May 30 17:46:10 2013
New Revision: 1487947

URL: http://svn.apache.org/r1487947
Log:
Store supported views in a tuple instead of a dict to maintain their order.

Modified:
    bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py

Modified: bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py
URL: http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py?rev=1487947&r1=1487946&r2=1487947&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py (original)
+++ bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py Thu May 30 17:46:10 2013
@@ -407,10 +407,10 @@ class RequestContext(object):
     DATA_SEARCH_EXTRAS = 'extra_search_fields'
 
     #bhsearch may support more pluggable views later
-    VIEWS_SUPPORTED = {
-        None: "Free text",
-        DATA_VIEW_GRID: "Grid"
-    }
+    VIEWS_SUPPORTED = (
+        (None, "Free text"),
+        (DATA_VIEW_GRID, "Grid"),
+    )
 
     VIEWS_WITH_KNOWN_FIELDS = [DATA_VIEW_GRID]
     OBLIGATORY_FIELDS_TO_SELECT = [IndexFields.ID, IndexFields.TYPE]
@@ -592,12 +592,12 @@ class RequestContext(object):
         active_view = self.parameters.view
 
         all_views = []
-        for view, label in self.VIEWS_SUPPORTED.iteritems():
+        for view, label in self.VIEWS_SUPPORTED:
             all_views.append(dict(
                 label=_(label),
                 href=self.parameters.create_href(
                     view=view, skip_view=(view is None)),
-                is_active = (view == active_view)
+                is_active=(view == active_view)
             ))
         self.data[self.DATA_ALL_VIEWS] = all_views