Changeset 7673 for trunk/calendar/inc/class.uicalendar.inc.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/calendar/inc/class.uicalendar.inc.php (modified) (12 diffs)

trunk/calendar/inc/class.uicalendar.inc.php

@@ -328 +328 @@
                                         );
                                 }
-                                for($l=0;$l<count($var);++$l)
+                $var_count = count($var);
+                                for($l=0;$l<$var_count;++$l)
                                 {
                                         $this->output_template_array($mini_cal_tpl,'monthweek_day','mini_day',$var[$l]);
@@ -1331 +1332 @@
                         $str = '';
 
-                        for($i=0;$i<count($event['recur_exception']);++$i)
+            $event_count = count($event['recur_exception']);
+                        for($i=0;$i<$event_count;++$i)
                         {
                                 $str .= '    <option value="'.$i.'">'.$GLOBALS['phpgw']->common->show_date($event['recur_exception'][$i]).'</option>'."\n";
@@ -1457 +1459 @@
                                 // Add participants
                                 $participants = explode(";", $GLOBALS['phpgw']->session->appsession("participants") );
-                                for($_f_part=0; $_f_part<count($participants); ++$_f_part)
+                $participants_count = count($participants);
+                                for($_f_part=0; $_f_part<$participants_count; ++$_f_part)
                                 {
                                         $this->bo->add_attribute('participants','A',$participants[$_f_part]);
@@ -2809 +2812 @@
                         $accounts = $GLOBALS['phpgw']->acl->get_ids_for_location('run',1,'calendar');
                         $users = Array();
-                        for($i=0;$i<count($accounts);++$i)
+            $accounts_count = count($accounts);
+                        for($i=0;$i<$accounts_count;++$i)
                         {
                                 $user = $accounts[$i];
@@ -2820 +2824 @@
                                                 if($group_members != False)
                                                 {
-                                                        for($j=0;$j<count($group_members);++$j)
+                            $group_members_count = count($group_members);
+                                                        for($j=0;$j<$group_members_count;++$j)
                                                         {
                                                                 if(!isset($users[$group_members[$j]]))
@@ -2861 +2866 @@
                         );
 
-                        for($i=0;$i<count($var);++$i)
+            $var_count = count($var);
+                        for($i=0;$i<$var_count;++$i)
                         {
                                 $this->output_template_array($p,'rows','list',$var[$i]);
@@ -3607 +3613 @@
 
                         $description = $this->bo->get_short_field($event,$is_private,'description');
-                        for($i=0;$i<count($picture);++$i)
+            $picture_count = count($picture);
+                        for($i=0;$i<$picture_count;++$i)
                         {
                                 $var = Array(
@@ -3797 +3804 @@
 
                         $description = $this->bo->get_short_field($event,$is_private,'description');
-                        for($i=0;$i<count($picture);++$i)
+            $picture_count = count($picture);
+                        for($i=0;$i<$picture_count;++$i)
                         {
                                 $var = Array(
@@ -3868 +3876 @@
 
                         $overlap = '';
-                        for($i=0;$i<count($overlapping_events);++$i)
+            $overlapping_events_count = count($overlapping_events);
+                        for($i=0;$i<$overlapping_events_count;++$i)
                         {
                                 $overlapped_event = $this->bo->read_entry($overlapping_events[$i],True);
@@ -5734 +5743 @@
                         }
 
-                        for ($i=0; $i<count($control_data['part']); ++$i)
+            $control_data_count = count($control_data['part']);
+                        for ($i=0; $i<$control_data_count; ++$i)
                         {
                                 $id = $control_data['part'][$i];
@@ -5745 +5755 @@
                         if ($control_data['action'] == lang('Delete selected contacts'))
                         {
-                                for ($i=0; $i<count($control_data['delete']); ++$i)
+                $control_data_count = count($control_data['delete']);
+                                for ($i=0; $i<$control_data_count; ++$i)
                                 {
                                         $id = $control_data['delete'][$i];
@@ -5970 +5981 @@
                                 if($holidays)
                                 {
-                                        for($k=0;$k<count($holidays);++$k)
+                    $holidays_count = count($holidays);
+                                        for($k=0;$k<$holidays_count;++$k)
                                         {
                                                 $holiday_name[] = $holidays[$k]['name'];