diff --git a/protected/humhub/modules/admin/controllers/AuthenticationController.php b/protected/humhub/modules/admin/controllers/AuthenticationController.php index 5288866907..99fa8d8050 100644 --- a/protected/humhub/modules/admin/controllers/AuthenticationController.php +++ b/protected/humhub/modules/admin/controllers/AuthenticationController.php @@ -1,15 +1,23 @@ subLayout = '@admin/views/layouts/user'; - - return parent::init(); + + return parent::init(); } /** @@ -44,38 +52,48 @@ class AuthenticationController extends Controller public function getAccessRules() { return [ - ['permissions' => \humhub\modules\admin\permissions\ManageSettings::className()] + ['permissions' => ManageSettings::className()] ]; } /** * Returns a List of Users + * @return string */ public function actionIndex() { - $form = new \humhub\modules\admin\models\forms\AuthenticationSettingsForm; + $form = new AuthenticationSettingsForm; if ($form->load(Yii::$app->request->post()) && $form->validate() && $form->save()) { $this->view->saved(); } // Build Group Dropdown - $groups = []; - $groups[''] = Yii::t('AdminModule.controllers_SettingController', 'None - shows dropdown in user registration.'); - foreach (\humhub\modules\user\models\Group::find()->all() as $group) { + $groups = [ + '' => Yii::t( + 'AdminModule.controllers_SettingController', + 'None - shows dropdown in user registration.' + ) + ]; + + foreach (Group::find()->all() as $group) { if (!$group->is_admin_group) { $groups[$group->id] = $group->name; } } return $this->render('authentication', [ - 'model' => $form, - 'groups' => $groups - ]); + 'model' => $form, + 'groups' => $groups + ]); } + /** + * Configure Ldap authentication + * @return string + */ public function actionAuthenticationLdap() { - $form = new \humhub\modules\admin\models\forms\AuthenticationLdapSettingsForm; + $form = new AuthenticationLdapSettingsForm; if ($form->load(Yii::$app->request->post()) && $form->validate() && $form->save()) { $this->view->saved(); return $this->redirect(['/admin/authentication/authentication-ldap']); @@ -88,16 +106,16 @@ class AuthenticationController extends Controller if (Yii::$app->getModule('user')->settings->get('auth.ldap.enabled')) { $enabled = true; try { - $ldapAuthClient = new \humhub\modules\user\authclient\ZendLdapClient(); + $ldapAuthClient = new ZendLdapClient(); $ldap = $ldapAuthClient->getLdap(); $userCount = $ldap->count( Yii::$app->getModule('user')->settings->get('auth.ldap.userFilter'), Yii::$app->getModule('user')->settings->get('auth.ldap.baseDn'), - \Zend\Ldap\Ldap::SEARCH_SCOPE_SUB + Ldap::SEARCH_SCOPE_SUB ); - } catch (\Zend\Ldap\Exception\LdapException $ex) { + } catch (LdapException $ex) { $errorMessage = $ex->getMessage(); - } catch (\Exception $ex) { + } catch (Exception $ex) { $errorMessage = $ex->getMessage(); } } diff --git a/protected/humhub/modules/admin/views/authentication/authentication_ldap.php b/protected/humhub/modules/admin/views/authentication/authentication_ldap.php index 24da91a577..a916917520 100644 --- a/protected/humhub/modules/admin/views/authentication/authentication_ldap.php +++ b/protected/humhub/modules/admin/views/authentication/authentication_ldap.php @@ -1,5 +1,4 @@ beginContent('@admin/views/authentication/_authenticationLayout.php') ?>
- +

- -
$errorMessage]); ?>
+ +
+ $errorMessage] + ) ?> +
-
+
+ +
-
$userCount]); ?>
+
+ $userCount] + ) ?> +
- 'authentication-settings-form']); ?> + 'authentication-settings-form', + 'fieldConfig' => function ($model, $attribute) { + return [ + 'inputOptions' => [ + 'class' => 'form-control', + 'readonly' => Setting::IsFixed('auth.ldap.' . $attribute, 'user') + ], + ]; + } + ]) ?> - field($model, 'enabled')->checkbox(['readonly' => Setting::IsFixed('auth.ldap.enabled', 'user')]); ?> + field($model, 'enabled')->checkbox() ?>
- field($model, 'hostname')->textInput(['readonly' => Setting::IsFixed('auth.ldap.hostname', 'user')]); ?> - field($model, 'port')->textInput(['readonly' => Setting::IsFixed('auth.ldap.port', 'user')]); ?> - field($model, 'encryption')->dropDownList($model->encryptionTypes, ['readonly' => Setting::IsFixed('auth.ldap.encryption', 'user')]); ?> - field($model, 'username')->textInput(['readonly' => Setting::IsFixed('auth.ldap.username', 'user')]); ?> - field($model, 'password')->passwordInput(['readonly' => Setting::IsFixed('auth.ldap.password', 'user')]); ?> - field($model, 'baseDn')->textInput(['readonly' => Setting::IsFixed('auth.ldap.baseDn', 'user')]); ?> - field($model, 'loginFilter')->textArea(['readonly' => Setting::IsFixed('auth.ldap.loginFilter', 'user')]); ?> - field($model, 'userFilter')->textArea(['readonly' => Setting::IsFixed('auth.ldap.userFilter', 'user')]); ?> - field($model, 'usernameAttribute')->textInput(['readonly' => Setting::IsFixed('auth.ldap.usernameAttribute', 'user')]); ?> - field($model, 'emailAttribute')->textInput(['readonly' => Setting::IsFixed('auth.ldap.emailAttribute', 'user')]); ?> - field($model, 'idAttribute')->textInput(['readonly' => Setting::IsFixed('auth.ldap.idAttribute', 'user')]); ?> - field($model, 'refreshUsers')->checkbox(['readonly' => Setting::IsFixed('auth.ldap.refreshUsers', 'user')]); ?> - + field($model, 'hostname')->textInput() ?> + field($model, 'port')->textInput() ?> + field($model, 'encryption')->dropDownList($model->encryptionTypes) ?> + field($model, 'username')->textInput() ?> + field($model, 'password')->passwordInput() ?> + field($model, 'baseDn')->textInput() ?> + field($model, 'loginFilter')->textArea() ?> + field($model, 'userFilter')->textArea() ?> + field($model, 'usernameAttribute')->textInput() ?> + field($model, 'emailAttribute')->textInput() ?> + field($model, 'idAttribute')->textInput() ?> + field($model, 'refreshUsers')->checkbox() ?>
- 'btn btn-primary', 'data-ui-loader' => ""]); ?> - - + 'btn btn-primary', 'data-ui-loader' => ''] + ) ?> + + +
-endContent(); ?> \ No newline at end of file +endContent() ?> \ No newline at end of file