[SM-CVS] SF.net SVN: squirrelmail:[14971] trunk/squirrelmail/plugins/calendar/ calendar_data.php

[<pdo...@us...>]

Revision: 14971
          http://sourceforge.net/p/squirrelmail/code/14971
Author:   pdontthink
Date:     2022-06-04 17:50:55 +0000 (Sat, 04 Jun 2022)
Log Message:
-----------
Remove use of each() and make sense of poorly named variables

Modified Paths:
--------------
    trunk/squirrelmail/plugins/calendar/calendar_data.php

Modified: trunk/squirrelmail/plugins/calendar/calendar_data.php
===================================================================
--- trunk/squirrelmail/plugins/calendar/calendar_data.php	2022-06-04 17:20:28 UTC (rev 14970)
+++ trunk/squirrelmail/plugins/calendar/calendar_data.php	2022-06-04 17:50:55 UTC (rev 14971)
@@ -104,22 +104,20 @@
     $filename = getHashedFile($username, $data_dir, "$username.$year.cal");
     $fp = fopen ($filetmp,"w");
     if ($fp) {
-        while ( $calfoo = each ($calendardata)) {
-            while ( $calbar = each ($calfoo['value'])) {
-                $calfoobar = $calendardata[$calfoo['key']][$calbar['key']];
-                array_walk($calfoobar,'calendar_encodedata');
+        foreach ( $calendardata as $datetime => $events ) {
+            foreach ( $events as $time_of_day => $event ) {
+                array_walk($event,'calendar_encodedata');
                 /**
                  * Make sure that reminder field is set. Calendar forms don't implement it, 
                  * but it is still used for calendar data. Backwards compatibility.
                  */ 
-                if (!isset($calfoobar['reminder'])) $calfoobar['reminder']='';
+                if (!isset($event['reminder'])) $event['reminder']='';
 
-                $calstr = "$calfoo[key]|$calbar[key]|$calfoobar[length]|$calfoobar[priority]|$calfoobar[title]|$calfoobar[message]|$calfoobar[reminder]\n";
+                $calstr = "$datetime|$time_of_day|$event[length]|$event[priority]|$event[title]|$event[message]|$event[reminder]\n";
                 if(sq_fwrite($fp, $calstr, 4096) === FALSE) {
                     error_box(_("Could not write calendar file %s", "$username.$year.cal.tmp"));
                 }
             }
-
         }
         fclose ($fp);
         @unlink($filename);

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.