From: Simo Sorce <simo@redhat.com>
Date: Wed, 26 Mar 2014 19:44:26 +0000 (-0400)
Subject: Automatically build configuration page menu
X-Git-Tag: v0.2.2~53
X-Git-Url: http://git.cascardo.info/?p=cascardo%2Fipsilon.git;a=commitdiff_plain;h=dbcead832f700ad63dc382648f0e3b1b84cd4d23;hp=134914682267592e2b42276255aff24bd2fabe6b

Automatically build configuration page menu

Do not hardcode it, rather build it out of the pages tree.

Signed-off-by: Simo Sorce <simo@redhat.com>
---

diff --git a/ipsilon/admin/common.py b/ipsilon/admin/common.py
index 0a46797..18b71ab 100755
--- a/ipsilon/admin/common.py
+++ b/ipsilon/admin/common.py
@@ -25,11 +25,13 @@ from ipsilon.util.page import admin_protect
 
 class AdminPluginPage(Page):
 
-    def __init__(self, obj, site, baseurl, facility):
+    def __init__(self, obj, site, parent):
         super(AdminPluginPage, self).__init__(site)
         self._obj = obj
-        self.url = '%s/%s' % (baseurl, obj.name)
-        self.facility = facility
+        self.title = '%s plugin' % obj.name
+        self.url = '%s/%s' % (parent.url, obj.name)
+        self.facility = parent.facility
+        self.menu = [parent]
 
         # Get the defaults
         self.plugin_config = obj.get_config_desc()
@@ -42,11 +44,10 @@ class AdminPluginPage(Page):
 
     @admin_protect
     def GET(self, *args, **kwargs):
-        return self._template('admin/plugin_config.html',
-                              title='%s plugin' % self._obj.name,
+        return self._template('admin/plugin_config.html', title=self.title,
                               name='admin_%s_%s_form' % (self.facility,
                                                          self._obj.name),
-                              action=self.url,
+                              menu=self.menu, action=self.url,
                               options=self.plugin_config)
 
     @admin_protect
@@ -80,13 +81,12 @@ class AdminPluginPage(Page):
                 self._obj.set_config_value(name, value)
                 self.plugin_config[name][2] = value
 
-        return self._template('admin/plugin_config.html',
+        return self._template('admin/plugin_config.html', title=self.title,
                               message=message,
                               message_type=message_type,
-                              title='%s plugin' % self._obj.name,
                               name='admin_%s_%s_form' % (self.facility,
                                                          self._obj.name),
-                              action=self.url,
+                              menu=self.menu, action=self.url,
                               options=self.plugin_config)
 
     def root(self, *args, **kwargs):
@@ -101,6 +101,17 @@ class Admin(Page):
     def __init__(self, site, mount):
         super(Admin, self).__init__(site)
         self.url = '%s/%s' % (self.basepath, mount)
+        self.menu = []
 
     def root(self, *args, **kwargs):
-        return self._template('admin/index.html', title='Configuration')
+        return self._template('admin/index.html',
+                              title='Configuration',
+                              menu=self.menu)
+
+    def add_subtree(self, name, page):
+        self.__dict__[name] = page
+        self.menu.append(page)
+
+    def del_subtree(self, name):
+        self.menu.remove(self.__dict__[name])
+        del self.__dict__[name]
diff --git a/ipsilon/admin/login.py b/ipsilon/admin/login.py
index 1f5e9fa..b8325c8 100755
--- a/ipsilon/admin/login.py
+++ b/ipsilon/admin/login.py
@@ -27,16 +27,17 @@ from ipsilon.login.common import FACILITY
 
 class LoginPluginsOrder(Page):
 
-    def __init__(self, site, baseurl):
+    def __init__(self, site, parent):
         super(LoginPluginsOrder, self).__init__(site)
-        self.url = '%s/order' % baseurl
+        self.url = '%s/order' % parent.url
+        self.menu = [parent]
 
     @admin_protect
     def GET(self, *args, **kwargs):
         return self._template('admin/login_order.html',
                               title='login plugins order',
                               name='admin_login_order_form',
-                              action=self.url,
+                              menu=self.menu, action=self.url,
                               options=self._site[FACILITY]['enabled'])
 
     @admin_protect
@@ -87,7 +88,7 @@ class LoginPluginsOrder(Page):
                               message_type=message_type,
                               title='login plugins order',
                               name='admin_login_order_form',
-                              action=self.url,
+                              menu=self.menu, action=self.url,
                               options=self._site[FACILITY]['enabled'])
 
     def root(self, *args, **kwargs):
@@ -100,19 +101,22 @@ class LoginPluginsOrder(Page):
 class LoginPlugins(Page):
     def __init__(self, site, parent):
         super(LoginPlugins, self).__init__(site)
-        parent.login = self
+        self._master = parent
+        self.title = 'Login Plugins'
         self.url = '%s/login' % parent.url
+        self.facility = FACILITY
+        parent.add_subtree('login', self)
 
         for plugin in self._site[FACILITY]['available']:
             cherrypy.log.error('Admin login plugin: %s' % plugin)
             obj = self._site[FACILITY]['available'][plugin]
-            self.__dict__[plugin] = AdminPluginPage(obj, self._site,
-                                                    self.url, FACILITY)
+            self.__dict__[plugin] = AdminPluginPage(obj, self._site, self)
 
-        self.order = LoginPluginsOrder(self._site, self.url)
+        self.order = LoginPluginsOrder(self._site, self)
 
     def root(self, *args, **kwargs):
         login_plugins = self._site[FACILITY]
-        return self._template('admin/login.html', title='Login Plugins',
+        return self._template('admin/login.html', title=self.title,
                               available=login_plugins['available'],
-                              enabled=login_plugins['enabled'])
+                              enabled=login_plugins['enabled'],
+                              menu=self._master.menu)
diff --git a/templates/master-admin.html b/templates/master-admin.html
index 43770c1..a5a696d 100644
--- a/templates/master-admin.html
+++ b/templates/master-admin.html
@@ -41,15 +41,11 @@
           </li>
         </ul>
         <ul class="nav navbar-nav navbar-primary">
+          {% for page in menu %}
           <li>
-            <a href="{{ basepath }}/admin/login">Login plugins</a>
-          </li>
-          <li>
-            <a href="{{ basepath }}/admin">Providers</a>
-          </li>
-          <li>
-            <a href="{{ basepath }}/admin">Sites</a>
+            <a href="{{ page.url }}">{{ page.title }}</a>
           </li>
+          {% endfor %}
         </ul>
       </div>
     </nav>