Eloy's finding: The name and description fields were not initialized when

going into the "new event" form, so they printed warnings INSIDE the fields
with error_reporting(E_ALL). Added initialization as empty strings.

Now I 'm beginning to understand why Martin calls him "Eagle Eyes" :)

diff --git a/calendar/event.php b/calendar/event.php
index 4fa0a14f1e5516c30fa3e7b94f76968ce2f32cd2..a9d40c3fa770dfedfaf1a9c45fd873993479192e 100644 (file)
--- a/calendar/event.php
+++ b/calendar/event.php
@@ -330,6 +330,8 @@
 
             switch($_REQUEST['type']) {
                 case 'user':
+                    $form->name = '';
+                    $form->description = '';
                     $form->courseid = 0;
                     $form->groupid = 0;
                     $form->userid = $USER->id;
@@ -348,6 +350,8 @@
                         $_REQUEST['type'] = 'select';
                     }
                     else {
+                        $form->name = '';
+                        $form->description = '';
                         $form->courseid = $group->courseid;
                         $form->groupid = $group->id;
                         $form->userid = $USER->id;
@@ -367,6 +371,8 @@
                         $_REQUEST['type'] = 'select';
                     }
                     else {
+                        $form->name = '';
+                        $form->description = '';
                         $form->courseid = $courseid;
                         $form->groupid = 0;
                         $form->userid = $USER->id;
@@ -379,6 +385,8 @@
                     }
                 break;
                 case 'site':
+                    $form->name = '';
+                    $form->description = '';
                     $form->courseid = 1;
                     $form->groupid = 0;
                     $form->userid = $USER->id;