[prev in list] [next in list] [prev in thread] [next in thread] 

List:       squirrelmail-cvs
Subject:    [SM-CVS] SF.net SVN: squirrelmail:[14971] trunk/squirrelmail/plugins/calendar/ calendar_data.php
From:       pdontthink--- via squirrelmail-cvs <squirrelmail-cvs () lists ! sourceforge ! net>
Date:       2022-06-04 17:50:55
Message-ID: 1654365055.323442.3366 () sfp-scm-4 ! v30 ! lw ! sourceforge ! com
[Download RAW message or body]

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.



-----
squirrelmail-cvs mailing list
List address: squirrelmail-cvs@lists.sourceforge.net
List info (subscribe/unsubscribe/change options): \
                https://lists.sourceforge.net/lists/listinfo/squirrelmail-cvs
Repository: http://squirrelmail.org/svn


[prev in list] [next in list] [prev in thread] [next in thread]