Changeset 7673 for trunk/prototype/services/ImapServiceAdapter.php

Timestamp:

12/18/12 16:15:29 (11 years ago)

Author:

douglasz

Message:

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

File:

• trunk/prototype/services/ImapServiceAdapter.php (modified) (3 diffs)

trunk/prototype/services/ImapServiceAdapter.php

@@ -171 +171 @@
                                         if( $filter[0] !== 'msgNumber' )
                                         {
-                                        for( $i = 0; $i < count($filter); ++$i )
-                                        {
-                                                if( count( $filter[$i] ) === 4 )
-                                                $criteria['isExact'] = ( array_shift( $filter[$i] ) === 'AND' );
-
-                                                $criteria[ $filter[$i][0] ] = array( 'criteria' => $filter[$i][2], 'filter' => $filter[$i][1] );
-                                        }
-
-                                        return $this->searchSieveRule($criteria);
+                        $filter_count = count($filter);
+                        for( $i = 0; $i < $filter_count; ++$i )
+                        {
+                            if( count( $filter[$i] ) === 4 )
+                            $criteria['isExact'] = ( array_shift( $filter[$i] ) === 'AND' );
+
+                            $criteria[ $filter[$i][0] ] = array( 'criteria' => $filter[$i][2], 'filter' => $filter[$i][1] );
+                        }
+
+                        return $this->searchSieveRule($criteria);
                                         }
 
@@ -188 +189 @@
 
                                         if( empty( $msgNumber ) )
-                                        return( false );
+                                            return( false );
 
                                         $result = $this->get_info_msgs( array( 'folder' => $folder_name, 
@@ -213 +214 @@
 
                                 $response = array( "page" => $page, "total" => $total_pages, "records" => $count );
-                                
-                                for ($i=0; $i<count($result); ++$i)
+                                $result_count = count($result);
+                                for ($i=0; $i<$result_count; ++$i)
                                 {
                                         $flags_enum = array('Unseen'=> 1,  'Answered'=> 1, 'Forwarded'=> 1, 'Flagged'=> 1, 'Recent'=> 1, 'Draft'=> 1 );