ldap: ConnectionProperties -> AuthProperties
authorManuel Nickschas <sputnick@quassel-irc.org>
Sun, 23 Oct 2016 22:21:05 +0000 (00:21 +0200)
committerBen Rosser <rosser.bjr@gmail.com>
Sat, 27 May 2017 18:01:06 +0000 (14:01 -0400)
This commit changes the settings key to something making slightly
more sense in the context of authentication.

src/core/core.cpp
src/core/ldapauthenticator.cpp
src/core/ldapauthenticator.h
src/qtui/coreconfigwizard.cpp
src/qtui/coreconfigwizard.h

index ec19aaa..bb13e88 100644 (file)
@@ -191,7 +191,7 @@ void Core::init()
 
     // Not entirely sure what is 'legacy' about the above, but it seems to be the way things work!
     QVariantMap authSettings = cs.authSettings().toMap();
-    initAuthenticator(authSettings.value("Authenticator").toString(), authSettings.value("ConnectionProperties").toMap());
+    initAuthenticator(authSettings.value("Authenticator").toString(), authSettings.value("AuthProperties").toMap());
 
     if (Quassel::isOptionSet("select-backend") || Quassel::isOptionSet("select-authenticator")) {
         if (Quassel::isOptionSet("select-backend")) {
@@ -1098,7 +1098,7 @@ void Core::saveAuthenticatorSettings(const QString &backend, const QVariantMap &
 {
     QVariantMap dbsettings;
     dbsettings["Authenticator"] = backend;
-    dbsettings["ConnectionProperties"] = settings;
+    dbsettings["AuthProperties"] = settings;
     CoreSettings().setAuthSettings(dbsettings);
 }
 
index d79ebf1..b9f9922 100644 (file)
@@ -102,7 +102,7 @@ QVariantMap LdapAuthenticator::setupDefaults() const
 }
 
 
-void LdapAuthenticator::setConnectionProperties(const QVariantMap &properties)
+void LdapAuthenticator::setAuthProperties(const QVariantMap &properties)
 {
     _hostName = properties["Hostname"].toString();
     _port = properties["Port"].toInt();
@@ -142,7 +142,7 @@ UserId LdapAuthenticator::validateUser(const QString &username, const QString &p
 
 bool LdapAuthenticator::setup(const QVariantMap &settings)
 {
-    setConnectionProperties(settings);
+    setAuthProperties(settings);
     bool status = ldapConnect();
     return status;
 }
@@ -150,7 +150,7 @@ bool LdapAuthenticator::setup(const QVariantMap &settings)
 
 Authenticator::State LdapAuthenticator::init(const QVariantMap &settings)
 {
-    setConnectionProperties(settings);
+    setAuthProperties(settings);
 
     bool status = ldapConnect();
     if (!status) {
index a149806..59a35b6 100644 (file)
@@ -68,7 +68,7 @@ public slots:
     UserId validateUser(const QString &user, const QString &password);
 
 protected:
-    virtual void setConnectionProperties(const QVariantMap &properties);
+    virtual void setAuthProperties(const QVariantMap &properties);
     bool ldapConnect();
     void ldapDisconnect();
     bool ldapAuth(const QString &username, const QString &password);
index a4cfe16..99fc354 100644 (file)
@@ -246,7 +246,7 @@ QString AuthenticationSelectionPage::selectedBackend() const
 }
 
 
-QVariantMap AuthenticationSelectionPage::connectionProperties() const
+QVariantMap AuthenticationSelectionPage::authProperties() const
 {
     QString backend = ui.backendList->itemData(ui.backendList->currentIndex()).toString();
 
@@ -512,7 +512,7 @@ void SyncPage::initializePage()
     // Fill in sync info about the authentication layer.
     AuthenticationSelectionPage *authPage = qobject_cast<AuthenticationSelectionPage *>(wizard()->page(CoreConfigWizard::AuthenticationSelectionPage));
     QString authenticator = authPage->selectedBackend();
-    QVariantMap authProperties = authPage->connectionProperties();
+    QVariantMap authProperties = authPage->authProperties();
     Q_ASSERT(!authenticator.isEmpty());
     ui.authenticator->setText(authenticator);
 
index b22fd19..bcc765f 100644 (file)
@@ -119,7 +119,7 @@ public:
     AuthenticationSelectionPage(const QHash<QString, QVariant> &backends, QWidget *parent = 0);
     int nextId() const;
     QString selectedBackend() const;
-    QVariantMap connectionProperties() const;
+    QVariantMap authProperties() const;
 
 private slots:
     void on_backendList_currentIndexChanged();