INFRA-132 - CRM/ACL - phpcbf
authorTim Otten <totten@civicrm.org>
Tue, 6 Jan 2015 21:09:18 +0000 (13:09 -0800)
committerTim Otten <totten@civicrm.org>
Tue, 6 Jan 2015 23:04:25 +0000 (15:04 -0800)
CRM/ACL/API.php
CRM/ACL/BAO/ACL.php
CRM/ACL/Form/ACL.php
CRM/ACL/Form/ACLBasic.php
CRM/ACL/Form/EntityRole.php
CRM/ACL/Form/WordPress/Permissions.php
CRM/ACL/Page/ACL.php
CRM/ACL/Page/ACLBasic.php
CRM/ACL/Page/EntityRole.php

index 7ada9d6f609d5cd3517ef7feefebae37a0aabfbf..0d3c6925eea26aa56ea8907ccbff65ca56fca7fc 100644 (file)
@@ -93,8 +93,8 @@ class CRM_ACL_API {
   public static function whereClause($type,
     &$tables,
     &$whereTables,
-    $contactID        = NULL,
-    $onlyDeleted      = FALSE,
+    $contactID = NULL,
+    $onlyDeleted = FALSE,
     $skipDeleteClause = FALSE
   ) {
     // the default value which is valid for rhe final AND
@@ -156,9 +156,9 @@ class CRM_ACL_API {
    */
   public static function group(
     $type,
-    $contactID      = NULL,
-    $tableName      = 'civicrm_saved_search',
-    $allGroups      = NULL,
+    $contactID = NULL,
+    $tableName = 'civicrm_saved_search',
+    $allGroups = NULL,
     $includedGroups = NULL
   ) {
     if ($contactID == NULL) {
@@ -193,9 +193,9 @@ class CRM_ACL_API {
   public static function groupPermission(
     $type,
     $groupID,
-    $contactID      = NULL,
-    $tableName      = 'civicrm_saved_search',
-    $allGroups      = NULL,
+    $contactID = NULL,
+    $tableName = 'civicrm_saved_search',
+    $allGroups = NULL,
     $includedGroups = NULL,
     $flush = FALSE
   ) {
index 3c6c05298c57446d6c6ac78427892039c07bb9d1..378579bc22a4ed5dcbe10235ad4a146b2c7b0c7e 100644 (file)
@@ -130,7 +130,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     /* Include clause if we're looking for a specific table/id permission */
 
-
     if (!empty($object_table)) {
       $where .= " AND ( {$t['ACL']}.object_table IS null
                          OR ({$t['ACL']}.object_table   = '" . CRM_Utils_Type::escape($object_table, 'String') . "'";
@@ -143,7 +142,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     /* Include clause if we're granting an ACL or ACL Role */
 
-
     if (!empty($acl_id)) {
       $where .= " AND ({$t['ACL']}.acl_id IS null
                         OR {$t['ACL']}.acl_id   = " . CRM_Utils_Type::escape($acl_id, 'Integer') . ')';
@@ -159,7 +157,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     /* Query for permissions granted to all contacts in the domain */
 
-
     $query[] = "SELECT      {$t['ACL']}.*, 0 as override
                     FROM        {$t['ACL']}
 
@@ -168,7 +165,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     /* Query for permissions granted to all contacts through an ACL group */
 
-
     $query[] = "SELECT      {$t['ACL']}.*, 0 as override
                     FROM        {$t['ACL']}
 
@@ -188,7 +184,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     /* Query for permissions granted directly to the contact */
 
-
     $query[] = "SELECT      {$t['ACL']}.*, 1 as override
                     FROM        {$t['ACL']}
 
@@ -201,7 +196,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     /* Query for permissions granted to the contact through an ACL group */
 
-
     $query[] = "SELECT      {$t['ACL']}.*, 1 as override
                     FROM        {$t['ACL']}
 
@@ -221,7 +215,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     /* Query for permissions granted to the contact through a group */
 
-
     $query[] = "SELECT      {$t['ACL']}.*, 0 as override
                     FROM        {$t['ACL']}
 
@@ -234,11 +227,9 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
                         AND     {$t['GroupContact']}.contact_id = $contact_id
                         AND     {$t['GroupContact']}.status     = 'Added')";
 
-
     /* Query for permissions granted through an ACL group to a Contact
          * group */
 
-
     $query[] = "SELECT      {$t['ACL']}.*, 0 as override
                     FROM        {$t['ACL']}
 
@@ -276,7 +267,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
              * 3) the rule governs the specific object we want
              */
 
-
       if (empty($dao->object_table) ||
         ($dao->object_table == $object_table
           && (empty($dao->object_id)
@@ -289,7 +279,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
       else {
         /* Otherwise try to generate a clause for this rule */
 
-
         $clause = self::getClause(
           $dao->object_table, $dao->object_id, $tables
         );
@@ -298,7 +287,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
                  * (id is null) on a table other than the one we're interested
                  * in.  So skip it. */
 
-
         if (empty($clause)) {
           continue;
         }
@@ -307,7 +295,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
       /* Now we figure out if this is an allow or deny rule, and possibly
              * a contact-level override */
 
-
       if ($dao->deny) {
         $deny[] = $clause;
       }
@@ -373,7 +360,7 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
    *
    * @return array    - Assoc. array of the ACL rule's properties
    */
-  public function toArray($format = '%s', $hideEmpty = false) {
+  public function toArray($format = '%s', $hideEmpty = FALSE) {
     $result = array();
 
     if (!self::$_fieldKeys) {
@@ -417,7 +404,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     $rule = new CRM_ACL_BAO_ACL();
 
-
     $acl     = self::getTableName();
     $contact = CRM_Contact_BAO_Contact::getTableName();
     $c2g     = CRM_Contact_BAO_GroupContact::getTableName();
@@ -540,7 +526,6 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
 
     $rule = new CRM_ACL_BAO_ACL();
 
-
     $acl     = self::getTableName();
     $c2g     = CRM_Contact_BAO_GroupContact::getTableName();
     $group   = CRM_Contact_BAO_Group::getTableName();
@@ -588,7 +573,6 @@ SELECT      $acl.*
     $acl = self::getTableName();
     $aclRole = 'civicrm_acl_role';
 
-
     $aclER = CRM_ACL_DAO_EntityRole::getTableName();
     $c2g   = CRM_Contact_BAO_GroupContact::getTableName();
     $group = CRM_Contact_BAO_Group::getTableName();
@@ -720,7 +704,6 @@ SELECT $acl.*
       return FALSE;
     }
 
-
     $query = "
 SELECT count( a.id )
   FROM civicrm_acl_cache c, civicrm_acl a
@@ -869,9 +852,9 @@ SELECT g.*
    * @return array
    */
   public static function group($type,
-    $contactID      = NULL,
-    $tableName      = 'civicrm_saved_search',
-    $allGroups      = NULL,
+    $contactID = NULL,
+    $tableName = 'civicrm_saved_search',
+    $allGroups = NULL,
     $includedGroups = NULL
   ) {
 
index 09585d7bf5816f330e29d1aa51dd30d82dda64a8..cc6c182c2857e1a9aaa36e3a3ca521a76345d088 100644 (file)
@@ -154,7 +154,6 @@ class CRM_ACL_Form_ACL extends CRM_Admin_Form {
       '&nbsp;', TRUE
     );
 
-
     $label = ts('Role');
     $role = array(
       '-1' => ts('- select role -'),
@@ -275,7 +274,6 @@ class CRM_ACL_Form_ACL extends CRM_Admin_Form {
     // note this also resets any ACL cache
     CRM_Core_BAO_Cache::deleteGroup('contact fields');
 
-
     if ($this->_action & CRM_Core_Action::DELETE) {
       CRM_ACL_BAO_ACL::del($this->_id);
       CRM_Core_Session::setStatus(ts('Selected ACL has been deleted.'), ts('Record Deleted'), 'success');
index 294992f9611dcdf493f47dd4ecc4afed991a25b3..1a00bb9103a4d9fd5bd6b57adad31a84fac078dd 100644 (file)
@@ -93,9 +93,9 @@ SELECT object_table
       array('</td><td>', '</td></tr><tr><td>')
     );
 
-
     $label = ts('Role');
-    $role = array('-1' => ts('- select role -'),
+    $role = array(
+    '-1' => ts('- select role -'),
       '0' => ts('Everyone'),
     ) + CRM_Core_OptionGroup::values('acl_role');
     $entityID = &$this->add('select', 'entity_id', $label, $role, TRUE);
index 97bbec7197a67c704f550e1cd2ee70542afaf25b..fe092af132045e0cf09d2f4fbb957fbcc6a99a66 100644 (file)
@@ -61,8 +61,6 @@ class CRM_ACL_Form_EntityRole extends CRM_Admin_Form {
       $aclRoles, TRUE
     );
 
-
-
     $label = ts('Assigned to');
     $group = array('' => ts('- select group -')) + CRM_Core_PseudoConstant::staticGroup(FALSE, 'Access');
     $this->add('select', 'entity_id', $label, $group, TRUE, array('class' => 'crm-select2 huge'));
index 0c962ed90726438ad54a2d125b8bccae63f36444..21ad5f077c8b0cebbb0c0d7ae06783875eb797f1 100644 (file)
@@ -44,7 +44,7 @@ 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' );
 
@@ -63,7 +63,7 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form {
         $roleObj = $wp_roles->get_role($role);
         if (!empty($roleObj->capabilities)) {
           foreach ($roleObj->capabilities as $ckey => $cname) {
-            if (array_key_exists($ckey , $permissionsArray)) {
+            if (array_key_exists($ckey, $permissionsArray)) {
               $elementName = $role.'['.$ckey.']';
               $defaults[$elementName] = 1;
             }
@@ -74,7 +74,7 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form {
         $rolePerms[$role] = $permissionsArray;
         foreach ( $rolePerms[$role] as $key => $value) {
           $elementName = $role.'['.$key.']';
-          $this->add('checkbox' , $elementName , $value);
+          $this->add('checkbox', $elementName, $value);
         }
         $roles[$role] = $name;
       }
@@ -87,11 +87,11 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form {
 
     $this->addButtons(
       array(
-        array (
+        array(
           'type'      => 'next',
           'name'      => ts('Save'),
           'spacing'   => '',
-          'isDefault' => false   ),
+          'isDefault' => FALSE   ),
       )
     );
 
@@ -131,7 +131,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));
index 8bf7a85afcb1e20c39cfe57366451f8dbf830aef..aaa149e23a3931b74697b68e39522ba22b62043f 100644 (file)
@@ -120,7 +120,8 @@ class CRM_ACL_Page_ACL extends CRM_Core_Page_Basic {
     );
 
     // set breadcrumb to append to admin/access
-    $breadCrumb = array(array('title' => ts('Access Control'),
+    $breadCrumb = array(array(
+    'title' => ts('Access Control'),
         'url' => CRM_Utils_System::url('civicrm/admin/access',
           'reset=1'
         ),
@@ -170,17 +171,21 @@ ORDER BY entity_id
 
     $roles = CRM_Core_OptionGroup::values('acl_role');
 
-    $group = array('-1' => ts('- select -'),
+    $group = array(
+    '-1' => ts('- select -'),
       '0' => ts('All Groups'),
     ) + CRM_Core_PseudoConstant::group();
-    $customGroup = array('-1' => ts('- select -'),
+    $customGroup = array(
+    '-1' => ts('- select -'),
       '0' => ts('All Custom Groups'),
     ) + CRM_Core_PseudoConstant::get('CRM_Core_DAO_CustomField', 'custom_group_id');
-    $ufGroup = array('-1' => ts('- select -'),
+    $ufGroup = array(
+    '-1' => ts('- select -'),
       '0' => ts('All Profiles'),
     ) + CRM_Core_PseudoConstant::get('CRM_Core_DAO_UFField', 'uf_group_id');
 
-    $event = array('-1' => ts('- select -'),
+    $event = array(
+    '-1' => ts('- select -'),
       '0' => ts('All Events'),
     ) + CRM_Event_PseudoConstant::event();
 
@@ -194,7 +199,6 @@ ORDER BY entity_id
       $acl[$dao->id]['object_id'] = $dao->object_id;
       $acl[$dao->id]['is_active'] = $dao->is_active;
 
-
       if ($acl[$dao->id]['entity_id']) {
         $acl[$dao->id]['entity'] = $roles[$acl[$dao->id]['entity_id']];
       }
index 82601b0fc6110f182accc0fd6e6ef13caef98edb..afb403ebf8a7e4093fca56ff4008c90e6e32209e 100644 (file)
@@ -108,7 +108,8 @@ class CRM_ACL_Page_ACLBasic extends CRM_Core_Page_Basic {
     );
 
     // set breadcrumb to append to admin/access
-    $breadCrumb = array(array('title' => ts('Access Control'),
+    $breadCrumb = array(array(
+    'title' => ts('Access Control'),
         'url' => CRM_Utils_System::url('civicrm/admin/access', 'reset=1'),
       ));
     CRM_Utils_System::appendBreadCrumb($breadCrumb);
index 66efb44070122b9ba382082910a358687ca4f3d9..b7a0791a2297a5e2ddc939c2b1ecaaa06f9f0cee 100644 (file)
@@ -120,7 +120,8 @@ class CRM_ACL_Page_EntityRole extends CRM_Core_Page_Basic {
     );
 
     // set breadcrumb to append to admin/access
-    $breadCrumb = array(array('title' => ts('Access Control'),
+    $breadCrumb = array(array(
+    'title' => ts('Access Control'),
         'url' => CRM_Utils_System::url('civicrm/admin/access',
           'reset=1'
         ),