Replace deprecated Addon::callHooks with Hook::callAll

[friendica.git] / mod / settings.php

diff --git a/mod/settings.php b/mod/settings.php
index 1ec3725dc389e5767c9ea5edde993e778b43ae6f..14c0cc64d7f32b09d8f30a04780b7faa9e608a98 100644 (file)
--- a/mod/settings.php
+++ b/mod/settings.php
@@ -8,8 +8,8 @@ use Friendica\BaseModule;
 use Friendica\Content\Feature;
 use Friendica\Content\Nav;
 use Friendica\Core\ACL;
-use Friendica\Core\Addon;
 use Friendica\Core\Config;
+use Friendica\Core\Hook;
 use Friendica\Core\L10n;
 use Friendica\Core\Logger;
 use Friendica\Core\PConfig;
@@ -30,7 +30,7 @@ use Friendica\Util\Temporal;
 
 function get_theme_config_file($theme)
 {
-       $a = get_app();
+       $a = \get_app();
        $base_theme = defaults($a->theme_info, 'extends');
 
        if (file_exists("view/theme/$theme/config.php")) {
@@ -215,7 +215,7 @@ function settings_post(App $a)
        if (($a->argc > 1) && ($a->argv[1] == 'addon')) {
                BaseModule::checkFormSecurityTokenRedirectOnError('/settings/addon', 'settings_addon');
 
-               Addon::callHooks('addon_settings_post', $_POST);
+               Hook::callAll('addon_settings_post', $_POST);
                return;
        }
 
@@ -297,7 +297,7 @@ function settings_post(App $a)
                        }
                }
 
-               Addon::callHooks('connector_settings_post', $_POST);
+               Hook::callAll('connector_settings_post', $_POST);
                return;
        }
 
@@ -372,7 +372,7 @@ function settings_post(App $a)
                                intval(local_user())
                );
 
-               Addon::callHooks('display_settings_post', $_POST);
+               Hook::callAll('display_settings_post', $_POST);
                $a->internalRedirect('settings/display');
                return; // NOTREACHED
        }
@@ -385,41 +385,29 @@ function settings_post(App $a)
                $a->internalRedirect('settings');
        }
 
-       Addon::callHooks('settings_post', $_POST);
+       Hook::callAll('settings_post', $_POST);
 
        if (!empty($_POST['password']) || !empty($_POST['confirm'])) {
                $newpass = $_POST['password'];
                $confirm = $_POST['confirm'];
 
-               $err = false;
-               if ($newpass != $confirm) {
-                       notice(L10n::t('Passwords do not match. Password unchanged.') . EOL);
-                       $err = true;
-               }
-
-               if (empty($newpass) || empty($confirm)) {
-                       notice(L10n::t('Empty passwords are not allowed. Password unchanged.') . EOL);
-                       $err = true;
-               }
-
-               if (!Config::get('system', 'disable_password_exposed', false) && User::isPasswordExposed($newpass)) {
-                       notice(L10n::t('The new password has been exposed in a public data dump, please choose another.') . EOL);
-                       $err = true;
-               }
+               try {
+                       if ($newpass != $confirm) {
+                               throw new Exception(L10n::t('Passwords do not match.'));
+                       }
 
-               //  check if the old password was supplied correctly before changing it to the new value
-               if (!User::authenticate(intval(local_user()), $_POST['opassword'])) {
-                       notice(L10n::t('Wrong password.') . EOL);
-                       $err = true;
-               }
+                       //  check if the old password was supplied correctly before changing it to the new value
+                       User::getIdFromPasswordAuthentication(local_user(), $_POST['opassword']);
 
-               if (!$err) {
                        $result = User::updatePassword(local_user(), $newpass);
-                       if (DBA::isResult($result)) {
-                               info(L10n::t('Password changed.') . EOL);
-                       } else {
-                               notice(L10n::t('Password update failed. Please try again.') . EOL);
+                       if (!DBA::isResult($result)) {
+                               throw new Exception(L10n::t('Password update failed. Please try again.'));
                        }
+
+                       info(L10n::t('Password changed.'));
+               } catch (Exception $e) {
+                       notice($e->getMessage());
+                       notice(L10n::t('Password unchanged.'));
                }
        }
 
@@ -755,7 +743,7 @@ function settings_content(App $a)
                        $settings_addons = L10n::t('No Addon settings configured');
                }
 
-               Addon::callHooks('addon_settings', $settings_addons);
+               Hook::callAll('addon_settings', $settings_addons);
 
 
                $tpl = Renderer::getMarkupTemplate('settings/addons.tpl');
@@ -802,7 +790,7 @@ function settings_content(App $a)
                }
 
                $settings_connectors = '';
-               Addon::callHooks('connector_settings', $settings_connectors);
+               Hook::callAll('connector_settings', $settings_connectors);
 
                if (is_site_admin()) {
                        $diasp_enabled = L10n::t('Built-in support for %s connectivity is %s', L10n::t('Diaspora'), ((Config::get('system', 'diaspora_enabled')) ? L10n::t('enabled') : L10n::t('disabled')));
@@ -875,7 +863,7 @@ function settings_content(App $a)
                        '$submit' => L10n::t('Save Settings'),
                ]);
 
-               Addon::callHooks('display_settings', $o);
+               Hook::callAll('display_settings', $o);
                return $o;
        }
 
@@ -1193,7 +1181,7 @@ function settings_content(App $a)
                '$nickname_block' => $prof_addr,
 
                '$h_pass'       => L10n::t('Password Settings'),
-               '$password1'=> ['password', L10n::t('New Password:'), '', ''],
+               '$password1'=> ['password', L10n::t('New Password:'), '', L10n::t('Allowed characters are a-z, A-Z, 0-9 and special characters except white spaces, accentuated letters and colon (:).')],
                '$password2'=> ['confirm', L10n::t('Confirm:'), '', L10n::t('Leave password fields blank unless changing')],
                '$password3'=> ['opassword', L10n::t('Current Password:'), '', L10n::t('Your current password to confirm the changes')],
                '$password4'=> ['mpassword', L10n::t('Password:'), '', L10n::t('Your current password to confirm the changes')],
@@ -1274,7 +1262,7 @@ function settings_content(App $a)
 
        ]);
 
-       Addon::callHooks('settings_form', $o);
+       Hook::callAll('settings_form', $o);
 
        $o .= '</form>' . "\r\n";