0013342: allow to configure default user/admin role names
authorPhilipp Schüle <p.schuele@metaways.de>
Tue, 18 Jul 2017 08:32:25 +0000 (10:32 +0200)
committerPhilipp Schüle <p.schuele@metaways.de>
Tue, 18 Jul 2017 10:43:04 +0000 (12:43 +0200)
* use new configs everywhere

https://forge.tine20.org/view.php?id=13342

Change-Id: I8967430d4b7aa171b3a0c19e8b8cd448c9f8a228
Reviewed-on: http://gerrit.tine20.com/customers/5233
Tested-by: Jenkins CI (http://ci.tine20.com/)
Reviewed-by: Philipp Schüle <p.schuele@metaways.de>
tine20/Admin/Setup/Initialize.php
tine20/Setup/Frontend/Cli.php
tine20/Setup/Initialize.php
tine20/Tinebase/Acl/Roles.php

index 4f60c09..2df0204 100644 (file)
@@ -20,8 +20,6 @@ class Admin_Setup_Initialize extends Setup_Initialize
     /**
      * Override method because admin app requires special rights
      * @see tine20/Setup/Setup_Initialize::createInitialRights($_application)
-     * 
-     * @todo make hard coded role name ('admin role') configurable
      */
     public static function createInitialRights(Tinebase_Model_Application $_application)
     {
@@ -31,7 +29,8 @@ class Admin_Setup_Initialize extends Setup_Initialize
         $oldNotesValue = $roles->useNotes(false);
         $oldModLogValue = $roles->modlogActive(false);
 
-        $adminRole = $roles->getRoleByName('admin role');
+        $adminRoleName = Tinebase_Config::getInstance()->get(Tinebase_Config::DEFAULT_ADMIN_ROLE_NAME);
+        $adminRole = $roles->getRoleByName($adminRoleName);
         $allRights = Tinebase_Application::getInstance()->getAllRights($_application->getId());
         foreach ( $allRights as $right ) {
             $roles->addSingleRight(
index c8fb487..00c4b85 100644 (file)
@@ -367,7 +367,8 @@ class Setup_Frontend_Cli
         }
         
         //get set rights
-        $users = Tinebase_Acl_Roles::getInstance()->getRoleByName('user role');
+        $userRoleName = Tinebase_Config::getInstance()->get(Tinebase_Config::DEFAULT_USER_ROLE_NAME);
+        $users = Tinebase_Acl_Roles::getInstance()->getRoleByName($userRoleName);
         $rights = Tinebase_Acl_Roles::getInstance()->getRoleRights($users->getId());
         
         //Uninstall Applications
index 1c2a0a2..291c596 100644 (file)
@@ -112,8 +112,6 @@ class Setup_Initialize
     /**
      * create inital rights
      * 
-     * @todo make hard coded role names ('user role' and 'admin role') configurable
-     * 
      * @param Tinebase_Application $application
      * @return void
      */
index 803533b..adc9bde 100644 (file)
@@ -779,9 +779,7 @@ class Tinebase_Acl_Roles extends Tinebase_Controller_Record_Abstract
     
     /**
      * Create initial Roles
-     * 
-     * @todo make hard coded role names ('user role' and 'admin role') configurable
-     * 
+     *
      * @return void
      */
     public function createInitialRoles()
@@ -798,10 +796,13 @@ class Tinebase_Acl_Roles extends Tinebase_Controller_Record_Abstract
         $oldSetNotes = $this->_setNotes;
         $this->_omitModLog = true;
         $this->_setNotes = false;
+
+        $userRoleName = Tinebase_Config::getInstance()->get(Tinebase_Config::DEFAULT_USER_ROLE_NAME);
+        $adminRoleName = Tinebase_Config::getInstance()->get(Tinebase_Config::DEFAULT_ADMIN_ROLE_NAME);
         
         // add roles and add the groups to the roles
         $adminRole = new Tinebase_Model_Role(array(
-            'name'                  => 'admin role',
+            'name'                  => $adminRoleName,
             'description'           => 'admin role for tine. this role has all rights per default.',
         ));
         $adminRole = $this->createRole($adminRole);
@@ -813,7 +814,7 @@ class Tinebase_Acl_Roles extends Tinebase_Controller_Record_Abstract
         ));
         
         $userRole = new Tinebase_Model_Role(array(
-            'name'                  => 'user role',
+            'name'                  => $userRoleName,
             'description'           => 'userrole for tine. this role has only the run rights for all applications per default.',
         ));
         $userRole = $this->createRole($userRole);