X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=CRM%2FACL%2FForm%2FWordPress%2FPermissions.php;h=f4c3e14fd095938f394dcd3cffbb9e8f72869143;hb=4de9f9328a1b4a7daf9802bdb6a6d9989b554fe8;hp=0c962ed90726438ad54a2d125b8bccae63f36444;hpb=9d29f5f8596366df6a6ae99b97029da639e03004;p=civicrm-core.git diff --git a/CRM/ACL/Form/WordPress/Permissions.php b/CRM/ACL/Form/WordPress/Permissions.php index 0c962ed907..f4c3e14fd0 100644 --- a/CRM/ACL/Form/WordPress/Permissions.php +++ b/CRM/ACL/Form/WordPress/Permissions.php @@ -44,9 +44,9 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { * * @return void */ - public function buildQuickForm( ) { + public function buildQuickForm() { - CRM_Utils_System::setTitle( 'Wordpress Access Control' ); + CRM_Utils_System::setTitle('Wordpress Access Control'); // Get the core permissions array $permissionsArray = self::getPermissionArray(); @@ -57,14 +57,14 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { if (!isset($wp_roles)) { $wp_roles = new WP_Roles(); } - foreach ( $wp_roles->role_names as $role => $name ) { + foreach ($wp_roles->role_names as $role => $name) { // Dont show the permissions options for administrator, as they have all permissions if ($role !== 'administrator') { $roleObj = $wp_roles->get_role($role); if (!empty($roleObj->capabilities)) { foreach ($roleObj->capabilities as $ckey => $cname) { - if (array_key_exists($ckey , $permissionsArray)) { - $elementName = $role.'['.$ckey.']'; + if (array_key_exists($ckey, $permissionsArray)) { + $elementName = $role . '[' . $ckey . ']'; $defaults[$elementName] = 1; } } @@ -72,9 +72,9 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { // Compose the checkbox array for each role, to assign to form $rolePerms[$role] = $permissionsArray; - foreach ( $rolePerms[$role] as $key => $value) { - $elementName = $role.'['.$key.']'; - $this->add('checkbox' , $elementName , $value); + foreach ($rolePerms[$role] as $key => $value) { + $elementName = $role . '[' . $key . ']'; + $this->add('checkbox', $elementName, $value); } $roles[$role] = $name; } @@ -87,11 +87,12 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { $this->addButtons( array( - array ( - 'type' => 'next', - 'name' => ts('Save'), - 'spacing' => '', - 'isDefault' => false ), + array( + 'type' => 'next', + 'name' => ts('Save'), + 'spacing' => '', + 'isDefault' => FALSE + ), ) ); @@ -112,18 +113,18 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { if (!isset($wp_roles)) { $wp_roles = new WP_Roles(); } - foreach ( $wp_roles->role_names as $role => $name ) { + foreach ($wp_roles->role_names as $role => $name) { $roleObj = $wp_roles->get_role($role); //Remove all civicrm capabilities for the role, as there may be some capabilities checkbox unticked - foreach ($permissionsArray as $key => $capability){ + foreach ($permissionsArray as $key => $capability) { $roleObj->remove_cap($key); } //Add the selected wordpress capabilities for the role $rolePermissions = $params[$role]; if (!empty($rolePermissions)) { - foreach ( $rolePermissions as $key => $capability ) { + foreach ($rolePermissions as $key => $capability) { $roleObj->add_cap($key); } } @@ -131,7 +132,7 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { if ($role == 'anonymous_user') { // Get the permissions into a format that matches what we get from WP $allWarningPermissions = CRM_Core_Permission::getAnonymousPermissionsWarnings(); - foreach ($allWarningPermissions as $key => $permission) { + foreach ($allWarningPermissions as $key => $permission) { $allWarningPermissions[$key] = CRM_utils_String::munge(strtolower($permission)); } $warningPermissions = array_intersect($allWarningPermissions, array_keys($rolePermissions)); @@ -141,7 +142,10 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { } if (!empty($warningPermissionNames)) { CRM_Core_Session::setStatus( - ts('The %1 role was assigned one or more permissions that may prove dangerous for users of that role to have. Please reconsider assigning %2 to them.', array( 1 => $wp_roles->role_names[$role], 2 => implode(', ', $warningPermissionNames))), + ts('The %1 role was assigned one or more permissions that may prove dangerous for users of that role to have. Please reconsider assigning %2 to them.', array( + 1 => $wp_roles->role_names[$role], + 2 => implode(', ', $warningPermissionNames) + )), ts('Unsafe Permission Settings') ); } @@ -159,7 +163,7 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { CRM_Core_Session::setStatus("", ts('Wordpress Access Control Updated'), "success"); // rebuild the menus to comply with the new permisssions/capabilites - CRM_Core_Invoke::rebuildMenuAndCaches( ); + CRM_Core_Invoke::rebuildMenuAndCaches(); CRM_Utils_System::redirect('admin.php?page=CiviCRM&q=civicrm/admin/access&reset=1'); CRM_Utils_System::civiExit(); @@ -170,9 +174,10 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form { * This function should be shared from a similar one in * distmaker/utils/joomlaxml.php * - * @return array civicrm permissions + * @return array + * civicrm permissions */ - public static function getPermissionArray(){ + public static function getPermissionArray() { global $civicrm_root; $permissions = CRM_Core_Permission::basicPermissions();