Fix broken login plugins order config handling
authorNathan Kinder <nkinder@redhat.com>
Sat, 10 May 2014 00:38:32 +0000 (17:38 -0700)
committerSimo Sorce <simo@redhat.com>
Sat, 10 May 2014 14:00:17 +0000 (10:00 -0400)
commitc950d7553e3f04a0fd5452afb705cf04e8f62a2b
treed4a3a83d0e892065b70ad70e819e010ee76265db
parent513aa5d4dbb72b73a9d60c89080868c43dedc358
Fix broken login plugins order config handling

The administrative page for configuring login plugins order had
a number of problems.  The html template expects a list of plugin
names to be supplied,  but a list of the actual plugin objects
was being supplied.  This caused a 500 error since join() would
throw an exception when it encounters something other than a string.

Even after fixing the 500 error, actually modifying the plugin
order would not work due to further issues with plugin objects
being used when strings representing the plugin names are expected
(and vice-versa).

This patch ensures that strings representing plugin names are
supplied to the html template, and that plugin objects are used
when re-ordering the live plugin list.

Resolves: https://fedorahosted.org/ipsilon/ticket/2

Signed-off-by: Nathan Kinder <nkinder@redhat.com>
Reviewed-by: Simo Sorce <simo@redhat.com>
ipsilon/admin/login.py