diff options
author | Simo Sorce <simo@redhat.com> | 2014-10-24 11:20:00 -0400 |
---|---|---|
committer | Patrick Uiterwijk <puiterwijk@redhat.com> | 2014-10-27 16:31:45 +0100 |
commit | 0167e69a38734586c1a1f45786313efb3b5f73c3 (patch) | |
tree | 1a9cfcaa0e6fa7c7d95cd8c0e31154f388bc33ba /ipsilon/admin | |
parent | 7a88f6bc436a95536b5af896f2f4997c859571f9 (diff) | |
download | ipsilon-0167e69a38734586c1a1f45786313efb3b5f73c3.tar.gz ipsilon-0167e69a38734586c1a1f45786313efb3b5f73c3.tar.xz ipsilon-0167e69a38734586c1a1f45786313efb3b5f73c3.zip |
Fix plugins enablement code
Signed-off-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
Diffstat (limited to 'ipsilon/admin')
-rwxr-xr-x | ipsilon/admin/common.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/ipsilon/admin/common.py b/ipsilon/admin/common.py index b68e53e..b9dfbf4 100755 --- a/ipsilon/admin/common.py +++ b/ipsilon/admin/common.py @@ -127,11 +127,18 @@ class AdminPluginsOrder(AdminPage): def GET(self, *args, **kwargs): return self.parent.root_with_msg() + def _get_enabled_by_name(self): + by_name = dict() + for p in self._site[self.facility]['available'].values(): + if p.is_enabled: + by_name[p.name] = p + return by_name + @admin_protect def POST(self, *args, **kwargs): message = "Nothing was modified." message_type = "info" - by_name = {p.name: p for p in self._site[self.facility]['enabled']} + by_name = self._get_enabled_by_name() if 'order' in kwargs: order = kwargs['order'].split(',') @@ -214,7 +221,8 @@ class AdminPlugins(AdminPage): enabled = [] if self.order: for plugin in plugins['enabled']: - enabled.append(plugin.name) + if plugin.is_enabled: + enabled.append(plugin.name) else: for _, plugin in plugins['available'].iteritems(): if plugin.is_enabled: |