Changeset 7673 for trunk/workflow


Ignore:
Timestamp:
12/18/12 16:15:29 (11 years ago)
Author:
douglasz
Message:

Ticket #3236 - Correcoes para Performance: Function Within Loop Declaration.

Location:
trunk/workflow/inc
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • trunk/workflow/inc/class.BrowserInfo.inc.php

    r7655 r7673  
    304304                $required = explode('.', $version); 
    305305                $current = explode('.', $this->getBrowserVersion()); 
    306                 for ($i = 0; $i < count($required); ++$i) 
     306        $required_count = count($required); 
     307                for ($i = 0; $i < $required_count; ++$i) 
    307308                { 
    308309                        $subRequired = (int) $required[$i]; 

  • trunk/workflow/inc/class.Paging.inc.php

    r7655 r7673  
    224224                        $links[] = $tmp; 
    225225                } 
    226  
    227                 for ($i = 0; $i < count($paginationLinks); ++$i) 
     226        $paginationLinks_count = count($paginationLinks); 
     227                for ($i = 0; $i < $paginationLinks_count; ++$i) 
    228228                { 
    229229                        $tmp = $paginationLinks[$i]; 

  • trunk/workflow/inc/class.bo_userinterface.inc.php

    r7655 r7673  
    846846 
    847847                $areas = $this->so->getAreaList($this->organizationInfo['organizacao_id']); 
    848                 for ($i = 0; $i < count($areas); ++$i) 
     848        $areas_count = count($areas); 
     849                for ($i = 0; $i < $areas_count; ++$i) 
    849850                { 
    850851                        $areas[$i]['children'] = false; 

  • trunk/workflow/inc/class.so_external_applications.inc.php

    r7655 r7673  
    148148 
    149149                $output = $result->GetArray(-1); 
    150                 for ($i = 0; $i < count($output); ++$i) 
     150        $output_count = count($output); 
     151                for ($i = 0; $i < $output_count; ++$i) 
    151152                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    152153                                unset($output[$i][$j]); 
    … …  
    170171 
    171172                $output = $result->GetArray(-1); 
    172                 for ($i = 0; $i < count($output); ++$i) 
     173        $output_count = count($output); 
     174                for ($i = 0; $i < $output_count; ++$i) 
    173175                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    174176                                unset($output[$i][$j]); 

  • trunk/workflow/inc/class.so_orgchart.inc.php

    r7655 r7673  
    152152 
    153153                $output = $result->GetArray(-1); 
    154                 for ($i = 0; $i < count($output); ++$i) 
     154        $output_count = count($output); 
     155                for ($i = 0; $i < $output_count; ++$i) 
    155156                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    156157                                unset($output[$i][$j]); 
    … …  
    385386 
    386387                $output = $result->GetArray(-1); 
    387                 for ($i = 0; $i < count($output); ++$i) 
     388        $output_count = count($output); 
     389                for ($i = 0; $i < $output_count; ++$i) 
    388390                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    389391                                unset($output[$i][$j]); 
    … …  
    462464 
    463465                $output = $result->GetArray(-1); 
    464                 for ($i = 0; $i < count($output); ++$i) 
     466        $output_count = count($output); 
     467                for ($i = 0; $i < $output_count; ++$i) 
    465468                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    466469                                unset($output[$i][$j]); 
    … …  
    541544 
    542545                $output = $result->GetArray(-1); 
    543                 for ($i = 0; $i < count($output); ++$i) 
     546        $output_count = count($output); 
     547                for ($i = 0; $i < $output_count; ++$i) 
    544548                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    545549                                unset($output[$i][$j]); 
    … …  
    623627 
    624628                $output = $result->GetArray(-1); 
    625                 for ($i = 0; $i < count($output); ++$i){ 
     629        $output_count = count($output); 
     630                for ($i = 0; $i < $output_count; ++$i){ 
    626631                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    627632                                unset($output[$i][$j]); 
    … …  
    714719                $cachedLDAP = Factory::newInstance('CachedLDAP'); 
    715720                $cachedLDAP->setOperationMode($cachedLDAP->OPERATION_MODE_LDAP_DATABASE); 
    716                 for ($i = 0; $i < count($output); ++$i) 
     721        $output_count = count($output); 
     722                for ($i = 0; $i < $output_count; ++$i) 
    717723                { 
    718724                        $output[$i]['funcionario_id_desc'] = ''; 
    … …  
    790796 
    791797                $output = $result->GetArray(-1); 
    792                 for ($i = 0; $i < count($output); ++$i) 
     798        $output_count = count($output); 
     799                for ($i = 0; $i < $output_count; ++$i) 
    793800                { 
    794801                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    … …  
    10031010 
    10041011                $output = $result->GetArray(-1); 
    1005                 for ($i = 0; $i < count($output); ++$i) 
     1012        $output_count = count($output); 
     1013                for ($i = 0; $i < $output_count; ++$i) 
    10061014                { 
    10071015                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    … …  
    13841392 
    13851393                $output = $result->GetArray(-1); 
    1386                 for ($i = 0; $i < count($output); ++$i) 
     1394        $output_count = count($output); 
     1395                for ($i = 0; $i < $output_count; ++$i) 
    13871396                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    13881397                                unset($output[$i][$j]); 
    … …  
    15861595 
    15871596                $output = $result->GetArray(-1); 
    1588                 for ($i = 0; $i < count($output); ++$i) { 
     1597        $output_count = count($output); 
     1598                for ($i = 0; $i < $output_count; ++$i) { 
    15891599                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    15901600                                unset($output[$i][$j]); 

  • trunk/workflow/inc/class.so_userinterface.inc.php

    r7655 r7673  
    123123 
    124124                        /* keep only associative elments and check if the user can access an intranet application */ 
    125                         for ($i = 0; $i < count($preOutput); ++$i) 
     125            $preOutput_count = count($preOutput); 
     126                        for ($i = 0; $i < $preOutput_count; ++$i) 
    126127                        { 
    127128                                if (($preOutput[$i]['intranet_only'] == '1') && (!$showIntranetApplications)) 
    … …  
    169170 
    170171                $output = $result->GetArray(-1); 
    171                 for ($i = 0; $i < count($output); ++$i) 
     172        $output_count = count($output); 
     173                for ($i = 0; $i < $output_count; ++$i) 
    172174                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    173175                                unset($output[$i][$j]); 
    … …  
    196198 
    197199                $output = $result->GetArray(-1); 
    198                 for ($i = 0; $i < count($output); ++$i) 
     200        $output_count = count($output); 
     201                for ($i = 0; $i < $output_count; ++$i) 
    199202                { 
    200203                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    … …  
    218221 
    219222                $output = $result->GetArray(-1); 
    220                 for ($i = 0; $i < count($output); ++$i) 
     223        $output_count = count($output); 
     224                for ($i = 0; $i < $output_count; ++$i) 
    221225                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    222226                                unset($output[$i][$j]); 
    … …  
    296300                $cachedLDAP->setOperationMode($cachedLDAP->OPERATION_MODE_NORMAL); 
    297301                $categoriesCount = array(); 
    298                 for ($i = 0; $i < count($employees); ++$i) 
     302        $employees_count = count($employees); 
     303                for ($i = 0; $i < $employees_count; ++$i) 
    299304                { 
    300305                        /* remove numeric fields */ 
    … …  
    396401                $cachedLDAP = Factory::newInstance('CachedLDAP'); 
    397402                $cachedLDAP->setOperationMode($cachedLDAP->OPERATION_MODE_NORMAL); 
    398                 for ($i = 0; $i < count($employees); ++$i) 
     403        $employees_count = count($employees); 
     404                for ($i = 0; $i < $employees_count; ++$i) 
    399405                { 
    400406                        /* remove numeric fields */ 
    … …  
    434440 
    435441                if (count($searchTermExploded) > 0){ 
    436                         for ($i=1; $i<count($searchTermExploded); ++$i) { 
     442            $searchTermExploded_count = count($searchTermExploded); 
     443                        for ($i=1; $i<$searchTermExploded_count; ++$i) { 
    437444                                if (strlen($searchTermExploded[$i]) > 2) { 
    438445                                        $fullSearch = true; 
    … …  
    481488 
    482489                /* filling return array with employee's information */ 
    483                 for ($i = 0; $i < count($result); ++$i) { 
     490        $result_count = count($result); 
     491                for ($i = 0; $i < $result_count; ++$i) { 
    484492                        $employees []= array( 
    485493                                        'area'                          => $result[$i]['area'], 
    … …  
    536544                /* creating the ldap query */ 
    537545                $ldap_query = '(&(|'; 
    538                 for ($i = 0; $i < count($result); ++$i) { 
     546        $result_count = count($result); 
     547                for ($i = 0; $i < $result_count; ++$i) { 
    539548                        $ldap_query .= '(uidNumber=' . $result[$i]['funcionario_id'] . ')'; 
    540549                } 
    … …  
    556565                /* filling return array with employee's information */ 
    557566                $employees = array(); 
    558                 for ($i = 0; $i < count($result); ++$i) { 
     567        $result_count = count($result); 
     568                for ($i = 0; $i < $result_count; ++$i) { 
    559569 
    560570                        $employee = array(); 
    … …  
    642652 
    643653                /* filling return array with employee's information */ 
    644                 for ($i = 0; $i < count($result); ++$i) { 
     654        $result_count = count($result); 
     655                for ($i = 0; $i < $result_count; ++$i) { 
    645656                        $employees []= array( 
    646657                                        'area'                          => $result[$i]['area'], 
    … …  
    707718 
    708719                $output = $result->GetArray(-1); 
    709                 for ($i = 0; $i < count($output); ++$i) 
     720        $output_count = count($output); 
     721                for ($i = 0; $i < $output_count; ++$i) 
    710722                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    711723                                unset($output[$i][$j]); 
    … …  
    728740 
    729741                $output = $result->GetArray(-1); 
    730                 for ( $i = 0; $i < count($output); ++$i ) 
     742        $output_count = count($output); 
     743                for ( $i = 0; $i < $output_count; ++$i ) 
    731744                { 
    732745                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    … …  
    759772 
    760773                $output = $result->GetArray( -1 ); 
    761                 for ( $i = 0; $i < count($output); ++$i ) 
     774        $output_count = count($output); 
     775                for ( $i = 0; $i < $output_count; ++$i ) 
    762776                        for ($j = 0; $j < $result->_numOfFields; ++$j) 
    763777                                unset($output[$i][$j]); 
    … …  
    795809 
    796810                $categoriesCount = array( ); 
    797                 for ( $i = 0; $i < count( $employees ); ++$i ) 
     811        $employees_count = count( $employees ); 
     812                for ( $i = 0; $i < $employees_count; ++$i ) 
    798813                { 
    799814                        // remove numeric fields 
    … …  
    867882 
    868883                $categoriesCount = array( ); 
    869                 for ( $i = 0; $i < count( $employees ); ++$i ) 
     884        $employees_count = count( $employees ); 
     885                for ( $i = 0; $i < $employees_count; ++$i ) 
    870886                { 
    871887                        // remove numeric fields 
    … …  
    893909 
    894910                // count the number of employees in each category 
    895                 for ( $i = 0; $i < count( $employees ); ++$i ) 
     911        $employees_count = count( $employees ); 
     912                for ( $i = 0; $i < $employees_count; ++$i ) 
    896913                { 
    897914                        $categoryID = $employees[ $i ][ 'funcionario_categoria_id' ]; 

  • trunk/workflow/inc/class.ui_orgchart.inc.php

    r7655 r7673  
    152152 
    153153                        $ldap = &Factory::getInstance('WorkflowLDAP'); 
    154                         for ($i = 0; $i < count($output); ++$i) 
     154            $output_count = count($output); 
     155                        for ($i = 0; $i < $output_count; ++$i) 
    155156                        { 
    156157                                for ($j = 0; $j < $result->_numOfFields; ++$j) 
    … …  
    162163                                        $supervisorArray = explode(' ', $supervisor); 
    163164                                        $supervisorName = ''; 
    164                                         for ($j = 0; $j < count($supervisorArray); ++$j) 
     165                    $supervisorArray_count = count($supervisorArray); 
     166                                        for ($j = 0; $j < $supervisorArray_count; ++$j) 
    165167                                                $supervisorName .= (($j == 0) || ($j == count($supervisorArray) - 1)) ? $supervisorArray[$j] . ' ' : $supervisorArray[$j][0] . '. '; 
    166168                                        $supervisorName = trim(str_replace("Junior", "Jr", $supervisorName)); 

  • trunk/workflow/inc/class.utils.security.php

    r7655 r7673  
    6868        function SecurityUtils($tagsArray = array(), $attrArray = array(), $tagsMethod = 0, $attrMethod = 0, $xssAuto = 1) { 
    6969                // make sure user defined arrays are in lowercase 
    70                 for ($i = 0; $i < count($tagsArray); ++$i) $tagsArray[$i] = strtolower($tagsArray[$i]); 
    71                 for ($i = 0; $i < count($attrArray); ++$i) $attrArray[$i] = strtolower($attrArray[$i]); 
     70        $tagsArray_count = count($tagsArray); 
     71                for ($i = 0; $i < $tagsArray_count; ++$i) 
     72            $tagsArray[$i] = strtolower($tagsArray[$i]); 
     73        $attrArray_count = count($attrArray); 
     74                for ($i = 0; $i < $attrArray_count; ++$i) 
     75            $attrArray[$i] = strtolower($attrArray[$i]); 
    7276                // assign to member vars 
    7377                $this->tagsArray = (array) $tagsArray; 
    … …  
    135139                $newSet = array(); 
    136140                // process attributes 
    137                 for ($i = 0; $i <count($attrSet); ++$i) { 
     141        $attrSet_count = count($attrSet); 
     142                for ($i = 0; $i < $attrSet_count; ++$i) { 
    138143                        // skip blank spaces in tag 
    139144                        if (!$attrSet[$i]) continue; 
Note: See TracChangeset for help on using the changeset viewer.