Ignore:
Timestamp:
12/19/12 16:44:18 (11 years ago)
Author:
douglasz
Message:

Ticket #3236 - Correcoes para Best Practice: Short Open Tag e Best Practice: Always Quote Array Keys.

Location:
trunk/expressoAdmin1_2/inc
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/expressoAdmin1_2/inc/class.boconfiguration.inc.php

    r5144 r7681  
    166166               $params['inputTextMaximumRecipientsGroupModal'] = 0 ; 
    167167 
    168             foreach ($params[selectGroupsInRule] as $groupInRule) 
     168            foreach ($params['selectGroupsInRule'] as $groupInRule) 
    169169            { 
    170170                //Verifica se este usuario Já tem uma regra 
  • trunk/expressoAdmin1_2/inc/class.functions.inc.php

    r7655 r7681  
    120120                        { 
    121121                                case list_users: 
    122                                         if ($array_acl[acl_add_users] || $array_acl[acl_edit_users] || $array_acl[acl_delete_users] || $array_acl[acl_change_users_password] || $array_acl[acl_change_users_quote] || $array_acl[acl_edit_sambausers_attributes] || $array_acl[acl_view_users] || $array_acl[acl_manipulate_corporative_information]) 
     122                                        if ($array_acl['acl_add_users'] || $array_acl['acl_edit_users'] || $array_acl['acl_delete_users'] || $array_acl['acl_change_users_password'] || $array_acl['acl_change_users_quote'] || $array_acl['acl_edit_sambausers_attributes'] || $array_acl['acl_view_users'] || $array_acl['acl_manipulate_corporative_information']) 
    123123                                                return true; 
    124124                                        break; 
    125125                                case add_users: 
    126                                         if ($array_acl[acl_add_users]) 
     126                                        if ($array_acl['acl_add_users']) 
    127127                                                return true; 
    128128                                        break; 
    129129                                case edit_users: 
    130                                         if ($array_acl[acl_edit_users]) 
     130                                        if ($array_acl['acl_edit_users']) 
    131131                                                return true; 
    132132                                        break; 
    133133                                case delete_users: 
    134                                         if ($array_acl[acl_delete_users]) 
     134                                        if ($array_acl['acl_delete_users']) 
    135135                                                return true; 
    136136                                        break; 
    137137                                case rename_users: 
    138                                         if ($array_acl[acl_rename_users]) 
     138                                        if ($array_acl['acl_rename_users']) 
    139139                                                return true; 
    140140                                        break; 
    141141                                case view_users: 
    142                                         if ($array_acl[acl_view_users]) 
     142                                        if ($array_acl['acl_view_users']) 
    143143                                                return true; 
    144144                                        break; 
    145145                                case edit_users_picture: 
    146                                         if ($array_acl[acl_edit_users_picture]) 
     146                                        if ($array_acl['acl_edit_users_picture']) 
    147147                                                return true; 
    148148                                        break; 
    149149                                case manipulate_corporative_information: 
    150                                         if ($array_acl[acl_manipulate_corporative_information]) 
     150                                        if ($array_acl['acl_manipulate_corporative_information']) 
    151151                                                return true; 
    152152                                        break; 
    153153                                case change_users_password: 
    154                                         if ($array_acl[acl_change_users_password]) 
     154                                        if ($array_acl['acl_change_users_password']) 
    155155                                                return true; 
    156156                                        break; 
    157157                                case change_users_quote: 
    158                                         if ($array_acl[acl_change_users_quote]) 
     158                                        if ($array_acl['acl_change_users_quote']) 
    159159                                                return true; 
    160160                                        break; 
    161161                                case set_user_default_password: 
    162                                         if ($array_acl[acl_set_user_default_password]) 
     162                                        if ($array_acl['acl_set_user_default_password']) 
    163163                                                return true; 
    164164                                        break; 
    165165                                case empty_user_inbox: 
    166                                         if (($array_acl[acl_empty_user_inbox]) && ($array_acl[acl_edit_users])) 
     166                                        if (($array_acl['acl_empty_user_inbox']) && ($array_acl['acl_edit_users'])) 
    167167                                                return true; 
    168168                                        break; 
    169169                                case edit_sambausers_attributes:                                case list_maillists: 
    170                                         if ($array_acl[acl_add_maillists] || $array_acl[acl_edit_maillists] || $array_acl[acl_delete_maillists]) 
    171                                                 return true; 
    172                                         break; 
    173  
    174                                         if ($array_acl[acl_edit_sambausers_attributes]) 
     170                                        if ($array_acl['acl_add_maillists'] || $array_acl['acl_edit_maillists'] || $array_acl['acl_delete_maillists']) 
     171                                                return true; 
     172                                        break; 
     173 
     174                                        if ($array_acl['acl_edit_sambausers_attributes']) 
    175175                                                return true; 
    176176                                        break; 
    177177                                case edit_sambadomains: 
    178                                         if ($array_acl[acl_edit_sambadomains]) 
     178                                        if ($array_acl['acl_edit_sambadomains']) 
    179179                                                return true; 
    180180                                        break; 
    181181                                 
    182182                                case list_groups: 
    183                                         if ($array_acl[acl_add_groups] || $array_acl[acl_edit_groups] || $array_acl[acl_delete_groups]) 
     183                                        if ($array_acl['acl_add_groups'] || $array_acl['acl_edit_groups'] || $array_acl['acl_delete_groups']) 
    184184                                                return true; 
    185185                                        break; 
    186186                                case add_groups: 
    187                                         if ($array_acl[acl_add_groups]) 
     187                                        if ($array_acl['acl_add_groups']) 
    188188                                                return true; 
    189189                                        break; 
    190190                                case edit_groups: 
    191                                         if ($array_acl[acl_edit_groups]) 
     191                                        if ($array_acl['acl_edit_groups']) 
    192192                                                return true; 
    193193                                        break; 
    194194                                case delete_groups: 
    195                                         if ($array_acl[acl_delete_groups]) 
     195                                        if ($array_acl['acl_delete_groups']) 
    196196                                                return true; 
    197197                                        break; 
    198198                                case edit_email_groups: 
    199                                         if ($array_acl[acl_edit_email_groups]) 
     199                                        if ($array_acl['acl_edit_email_groups']) 
    200200                                                return true; 
    201201                                        break; 
    202202                                 
    203203                                case list_maillists: 
    204                                         if ($array_acl[acl_add_maillists] || $array_acl[acl_edit_maillists] || $array_acl[acl_delete_maillists]) 
     204                                        if ($array_acl['acl_add_maillists'] || $array_acl['acl_edit_maillists'] || $array_acl['acl_delete_maillists']) 
    205205                                                return true; 
    206206                                        break; 
    207207                                case add_maillists: 
    208                                         if ($array_acl[acl_add_maillists]) 
     208                                        if ($array_acl['acl_add_maillists']) 
    209209                                                return true; 
    210210                                        break; 
    211211                                case edit_maillists: 
    212                                         if ($array_acl[acl_edit_maillists]) 
     212                                        if ($array_acl['acl_edit_maillists']) 
    213213                                                return true; 
    214214                                        break; 
    215215                                case delete_maillists: 
    216                                         if ($array_acl[acl_delete_maillists]) 
     216                                        if ($array_acl['acl_delete_maillists']) 
    217217                                                return true; 
    218218                                        break; 
    219219 
    220220                                case list_sectors: 
    221                                         if ($array_acl[acl_create_sectors] || $array_acl[acl_edit_sectors] || $array_acl[acl_delete_sectors]) 
     221                                        if ($array_acl['acl_create_sectors'] || $array_acl['acl_edit_sectors'] || $array_acl['acl_delete_sectors']) 
    222222                                                return true; 
    223223                                        break; 
    224224                                case create_sectors: 
    225                                         if ($array_acl[acl_create_sectors]) 
     225                                        if ($array_acl['acl_create_sectors']) 
    226226                                                return true; 
    227227                                        break; 
    228228                                case edit_sectors: 
    229                                         if ($array_acl[acl_edit_sectors]) 
     229                                        if ($array_acl['acl_edit_sectors']) 
    230230                                                return true; 
    231231                                        break; 
    232232                                case delete_sectors: 
    233                                         if ($array_acl[acl_delete_sectors]) 
     233                                        if ($array_acl['acl_delete_sectors']) 
    234234                                                return true; 
    235235                                        break; 
    236236 
    237237                                case view_global_sessions: 
    238                                         if ($array_acl[acl_view_global_sessions]) 
     238                                        if ($array_acl['acl_view_global_sessions']) 
    239239                                                return true; 
    240240                                        break; 
    241241 
    242242                                case list_computers: 
    243                                         if ($array_acl[acl_create_computers] || $array_acl[acl_edit_computers] || $array_acl[acl_delete_computers]) 
     243                                        if ($array_acl['acl_create_computers'] || $array_acl['acl_edit_computers'] || $array_acl['acl_delete_computers']) 
    244244                                                return true; 
    245245                                        break; 
    246246                                case create_computers: 
    247                                         if ($array_acl[acl_create_computers]) 
     247                                        if ($array_acl['acl_create_computers']) 
    248248                                                return true; 
    249249                                        break; 
    250250                                case edit_computers: 
    251                                         if ($array_acl[acl_edit_computers]) 
     251                                        if ($array_acl['acl_edit_computers']) 
    252252                                                return true; 
    253253                                        break; 
    254254                                case delete_computers: 
    255                                         if ($array_acl[acl_delete_computers]) 
     255                                        if ($array_acl['acl_delete_computers']) 
    256256                                                return true; 
    257257                                        break; 
    258258 
    259259                                case view_logs: 
    260                                         if ($array_acl[acl_view_logs]) 
     260                                        if ($array_acl['acl_view_logs']) 
    261261                                                return true; 
    262262                                        break; 
    263263                         
    264264                                case display_groups: 
    265                                         if ( $array_acl[acl_edit_users] || $array_acl[acl_view_users] || ($array_acl[acl_edit_sambausers_attributes] && ($this->current_config['expressoAdmin_samba_support'] == 'true')) ) 
     265                                        if ( $array_acl['acl_edit_users'] || $array_acl['acl_view_users'] || ($array_acl['acl_edit_sambausers_attributes'] && ($this->current_config['expressoAdmin_samba_support'] == 'true')) ) 
    266266                                                return true; 
    267267                                        break; 
    268268                                case display_emailconfig: 
    269                                         if ($array_acl[acl_edit_users] || $array_acl[acl_view_users]) 
     269                                        if ($array_acl['acl_edit_users'] || $array_acl['acl_view_users']) 
    270270                                                return true; 
    271271                                        break; 
    272272                                case display_applications: 
    273                                         if ($array_acl[acl_edit_users] || $array_acl[acl_view_users]) 
     273                                        if ($array_acl['acl_edit_users'] || $array_acl['acl_view_users']) 
    274274                                                return true; 
    275275                                        break; 
    276276                                case display_emaillists: 
    277                                         if ($array_acl[acl_edit_users] || $array_acl[acl_view_users]) 
     277                                        if ($array_acl['acl_edit_users'] || $array_acl['acl_view_users']) 
    278278                                                return true; 
    279279                                        break; 
  • trunk/expressoAdmin1_2/inc/class.ldap_functions.inc.php

    r7673 r7681  
    382382                if ($anchor != $dn) 
    383383                { 
    384                         if (!@ldap_rename($this->ldap, $params['anchor'], "uid=$params[uid]", $params['context'], true)) 
     384                        if (!@ldap_rename($this->ldap, $params['anchor'], "uid={$params['uid']}", $params['context'], true)) 
    385385                        { 
    386386                                $result['status'] = false; 
     
    467467                                return $result; 
    468468                        }                                          
    469                         $dn = "uid=$params[uid]," . $params['context']; 
     469                        $dn = "uid={$params['uid']}," . $params['context']; 
    470470                        $filter = "(mail=".$params['mail'].")"; 
    471471                        $justthese = array("cn"); 
  • trunk/expressoAdmin1_2/inc/class.uimanagers.inc.php

    r7655 r7681  
    112112                                        $ldap_search = ldap_search($ldap_conn, $GLOBALS['phpgw_info']['server']['ldap_context'], $filter, $justthese); 
    113113                                        $ldap_result = ldap_get_entries($ldap_conn, $ldap_search); 
    114                                         $p->set_var('manager_lid', $array_managers[manager_lid]); 
     114                                        $p->set_var('manager_lid', $array_managers['manager_lid']); 
    115115                                        $p->set_var('manager_cn', $ldap_result[0]['cn'][0] == '' ? '<font color=red>NAO ENCONTRADO NO LDAP</font>' : $ldap_result[0]['cn'][0]); 
    116116                                        $p->set_var('context', $managers_context); 
    117                                         $p->set_var('link_edit',$this->row_action('edit','edit_managers',$array_managers[manager_lid],$array_managers[context])); 
    118                                         $p->set_var('link_delete',$this->row_action('delete','delete_managers',$array_managers[manager_lid],$array_managers[context])); 
     117                                        $p->set_var('link_edit',$this->row_action('edit','edit_managers',$array_managers['manager_lid'],$array_managers['context'])); 
     118                                        $p->set_var('link_delete',$this->row_action('delete','delete_managers',$array_managers['manager_lid'],$array_managers['context'])); 
    119119                                        $p->set_var('link_copy',"<a href='#' onClick='javascript:copy_manager(\"".$array_managers['manager_lid']."\");'>Copiar</a>"); 
    120120                                        $p->fp('rows','row',True); 
Note: See TracChangeset for help on using the changeset viewer.