| @@ -4,8 +4,9 @@ | ||
| 4 | 4 | |
| 5 | 5 | namespace UserAccessManager\Controller\Backend; |
| 6 | 6 | |
| 7 | 7 | use UserAccessManager\Config\WordpressConfig; |
| 8 | +use UserAccessManager\Controller\Backend\Administration\SetupController; | |
| 8 | 9 | use UserAccessManager\Controller\Controller; |
| 9 | 10 | use UserAccessManager\Setup\Database\MissingColumnsException; |
| 10 | 11 | use UserAccessManager\Setup\SetupHandler; |
| 11 | 12 | use UserAccessManager\User\UserHandler; |
| @@ -18,10 +19,15 @@ | ||
| 18 | 19 | public const HANDLE_STYLE_ADMIN = 'UserAccessManagerAdmin'; |
| 19 | 20 | public const HANDLE_SCRIPT_GROUP_SUGGEST = 'UserAccessManagerGroupSuggest'; |
| 20 | 21 | public const HANDLE_SCRIPT_TIME_INPUT = 'UserAccessManagerTimeInput'; |
| 21 | 22 | public const HANDLE_SCRIPT_ADMIN = 'UserAccessManagerFunctions'; |
| 22 | - public const UAM_ERRORS = 'UAM_ERRORS'; | |
| 23 | 23 | |
| 24 | + private const ADMIN_SCRIPTS = [ | |
| 25 | + self::HANDLE_SCRIPT_GROUP_SUGGEST => 'assets/js/jquery.uam-group-suggest.js', | |
| 26 | + self::HANDLE_SCRIPT_TIME_INPUT => 'assets/js/jquery.uam-time-input.js', | |
| 27 | + self::HANDLE_SCRIPT_ADMIN => 'assets/js/functions.js' | |
| 28 | + ]; | |
| 29 | + | |
| 24 | 30 | private string $notice = ''; |
| 25 | 31 | |
| 26 | 32 | public function __construct( |
| 27 | 33 | Php $php, |
| @@ -37,9 +43,9 @@ | ||
| 37 | 43 | * @throws MissingColumnsException |
| 38 | 44 | */ |
| 39 | 45 | public function showAdminNotice(): void |
| 40 | 46 | { |
| 41 | - $messages = isset($_SESSION[self::UAM_ERRORS]) === true ? $_SESSION[self::UAM_ERRORS] : []; | |
| 47 | + $messages = $_SESSION[self::UAM_ERRORS] ?? []; | |
| 42 | 48 | $updateAction = $this->getRequestParameter('uam_update_db'); |
| 43 | 49 | |
| 44 | 50 | if ($this->setupHandler->getDatabaseHandler()->isDatabaseUpdateNecessary() === true |
| 45 | 51 | && $updateAction !== SetupController::UPDATE_BLOG |
| @@ -70,28 +76,16 @@ | ||
| 70 | 76 | UserAccessManager::VERSION, |
| 71 | 77 | 'screen' |
| 72 | 78 | ); |
| 73 | 79 | |
| 74 | - $this->wordpress->registerScript( | |
| 75 | - self::HANDLE_SCRIPT_GROUP_SUGGEST, | |
| 76 | - $urlPath . 'assets/js/jquery.uam-group-suggest.js', | |
| 77 | - ['jquery'], | |
| 78 | - UserAccessManager::VERSION | |
| 79 | - ); | |
| 80 | - | |
| 81 | - $this->wordpress->registerScript( | |
| 82 | - self::HANDLE_SCRIPT_TIME_INPUT, | |
| 83 | - $urlPath . 'assets/js/jquery.uam-time-input.js', | |
| 84 | - ['jquery'], | |
| 85 | - UserAccessManager::VERSION | |
| 86 | - ); | |
| 87 | - | |
| 88 | - $this->wordpress->registerScript( | |
| 89 | - self::HANDLE_SCRIPT_ADMIN, | |
| 90 | - $urlPath . 'assets/js/functions.js', | |
| 91 | - ['jquery'], | |
| 92 | - UserAccessManager::VERSION | |
| 93 | - ); | |
| 80 | + foreach (self::ADMIN_SCRIPTS as $handle => $scriptPath) { | |
| 81 | + $this->wordpress->registerScript( | |
| 82 | + $handle, | |
| 83 | + $urlPath . $scriptPath, | |
| 84 | + ['jquery'], | |
| 85 | + UserAccessManager::VERSION | |
| 86 | + ); | |
| 87 | + } | |
| 94 | 88 | } |
| 95 | 89 | |
| 96 | 90 | public function enqueueStylesAndScripts(): void |
| 97 | 91 | { |
| @@ -96,11 +90,12 @@ | ||
| 96 | 90 | public function enqueueStylesAndScripts(): void |
| 97 | 91 | { |
| 98 | 92 | $this->registerStylesAndScripts(); |
| 99 | 93 | $this->wordpress->enqueueStyle(self::HANDLE_STYLE_ADMIN); |
| 100 | - $this->wordpress->enqueueScript(self::HANDLE_SCRIPT_GROUP_SUGGEST); | |
| 101 | - $this->wordpress->enqueueScript(self::HANDLE_SCRIPT_TIME_INPUT); | |
| 102 | - $this->wordpress->enqueueScript(self::HANDLE_SCRIPT_ADMIN); | |
| 94 | + | |
| 95 | + foreach (array_keys(self::ADMIN_SCRIPTS) as $handle) { | |
| 96 | + $this->wordpress->enqueueScript($handle); | |
| 97 | + } | |
| 103 | 98 | } |
| 104 | 99 | |
| 105 | 100 | public function setupAdminDashboard(): void |
| 106 | 101 | { |
