Update style of plugins config page too

Also fix a bug that would cause the config page to show no fields
when saving.

Signed-off-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>

1 files changed, 9 insertions, 10 deletions

diff --git a/ipsilon/admin/common.py b/ipsilon/admin/common.py
index b3c318b..4d69bcd 100755
--- a/ipsilon/admin/common.py
+++ b/ipsilon/admin/common.py
@@ -65,16 +65,20 @@ class AdminPluginConfig(AdminPage):
             for k in add:
                 self.options_order.append(k)
 
-    @admin_protect
-    def GET(self, *args, **kwargs):
+    def root_with_msg(self, message=None, message_type=None):
         return self._template('admin/plugin_config.html', title=self.title,
+                              menu=self.menu, action=self.url, back=self.back,
+                              message=message, message_type=message_type,
                               name='admin_%s_%s_form' % (self.facility,
                                                          self._po.name),
-                              menu=self.menu, action=self.url, back=self.back,
                               options_order=self.options_order,
                               plugin=self._po)
 
     @admin_protect
+    def GET(self, *args, **kwargs):
+        return self.root_with_msg()
+
+    @admin_protect
     def POST(self, *args, **kwargs):
 
         message = "Nothing was modified."
@@ -106,13 +110,8 @@ class AdminPluginConfig(AdminPage):
             # And only if it succeeds we change the live object
             self._po.refresh_plugin_config(self.facility)
 
-        return self._template('admin/plugin_config.html', title=self.title,
-                              message=message,
-                              message_type=message_type,
-                              name='admin_%s_%s_form' % (self.facility,
-                                                         self._po.name),
-                              menu=self.menu, action=self.url,
-                              plugin=self._po)
+        return self.root_with_msg(message=message,
+                                  message_type=message_type)
 
 
 class AdminPluginsOrder(AdminPage):