diff options
author | Thomas Steur <thomas.steur@googlemail.com> | 2014-07-22 10:49:54 +0400 |
---|---|---|
committer | Thomas Steur <thomas.steur@googlemail.com> | 2014-07-22 10:51:00 +0400 |
commit | f6d87baf3833a00c3747cd590d83d0e499d7c94e (patch) | |
tree | 15311b6188b358427e443176b6b6694e823e70f8 /plugins/UserCountry/Menu.php | |
parent | 2949359cfb2e2718aa50236148e9f82e4787d0de (diff) |
refs #5192 simplified admin menu since it is very confusing for developers to find the correct translation key to use for menuName. Results in lots of dulicated code as well (same translation keys are mentioned everywhere). Also removed the $displayedForCurrentUsers in those methods since boolean parameters are bad practice and it did result in duplicated code as well as you had to call the same function or pass the same variable in each add() call. Moved the VisitorGenerator in a development section since it is not really diagnostic. There will be another development menu item with the next commit. Need to update ui tests
Diffstat (limited to 'plugins/UserCountry/Menu.php')
-rw-r--r-- | plugins/UserCountry/Menu.php | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/plugins/UserCountry/Menu.php b/plugins/UserCountry/Menu.php index dca2245723..505ae68c97 100644 --- a/plugins/UserCountry/Menu.php +++ b/plugins/UserCountry/Menu.php @@ -16,11 +16,10 @@ class Menu extends \Piwik\Plugin\Menu { public function configureAdminMenu(MenuAdmin $menu) { - if (UserCountry::isGeoLocationAdminEnabled()) { - $menu->add('General_Settings', 'UserCountry_Geolocation', - array('module' => 'UserCountry', 'action' => 'adminIndex'), - Piwik::hasUserSuperUserAccess(), - $order = 8); + if (UserCountry::isGeoLocationAdminEnabled() && Piwik::hasUserSuperUserAccess()) { + $menu->addSettingsItem('UserCountry_Geolocation', + array('module' => 'UserCountry', 'action' => 'adminIndex'), + $order = 8); } } |