Fixed a whole bunch of warnings and minor bugs, which all shared a co…

…mmon cause: The preferences screens weren't handling filter changes correctly.
safatshahin · Apr 20, 2004 · 053c98c · 053c98c
1 parent 5627277
commit 053c98c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/calendar/lib.php b/calendar/lib.php
@@ -567,6 +567,9 @@ function calendar_filter_controls($type) {
         case 'upcoming':
             $getvars = '';
         break;
+        case 'prefs':
+            $getvars = '&amp;from=prefs&amp;pref='.$_GET['edit'];
+        break;
         case 'event':
             global $day, $mon, $yr;
             $getvars = '&amp;from=event&amp;id='.$_GET['id'];

diff --git a/calendar/preferences.php b/calendar/preferences.php
@@ -258,7 +258,7 @@
     list($prevmon, $prevyr) = calendar_sub_month($mon, $yr);
     list($nextmon, $nextyr) = calendar_add_month($mon, $yr);
 
-    echo calendar_filter_controls($_GET['view']);
+    echo calendar_filter_controls('prefs');
     echo '<p>';
     echo calendar_top_controls('display', array('m' => $prevmon, 'y' => $prevyr));
     echo calendar_get_mini($courses, $groups, $users, $prevmon, $prevyr);

diff --git a/calendar/set.php b/calendar/set.php
@@ -89,8 +89,11 @@
 	}
 
 	switch($_GET['from']) {
+	    case 'prefs':
+	    redirect($CFG->wwwroot.'/calendar/preferences.php?edit='.$_GET['pref']);
+	    break;
 		case 'month':
-	redirect($CFG->wwwroot.'/calendar/view.php?view=month&cal_d='.$_GET['cal_d'].'&cal_m='.$_GET['cal_m'].'&cal_y='.$_GET['cal_y']);
+	    redirect($CFG->wwwroot.'/calendar/view.php?view=month&cal_d='.$_GET['cal_d'].'&cal_m='.$_GET['cal_m'].'&cal_y='.$_GET['cal_y']);
 		break;
 		case 'upcoming':
 		redirect($CFG->wwwroot.'/calendar/view.php?view=upcoming&cal_d='.$_GET['cal_d'].'&cal_m='.$_GET['cal_m'].'&cal_y='.$_GET['cal_y']);