After agreeing to site policy, you get redirected back to where you

were going. Also, guests get asked to agree for EVERY SESSION.
moodle · Nov 29, 2004 · 957b519 · 957b519
1 parent 561c0c4
commit 957b519
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/lib/moodlelib.php b/lib/moodlelib.php
@@ -788,6 +788,7 @@ function require_login($courseid=0, $autologinguest=true) {
     // Check that the user has agreed to a site policy if there is one
     if (!empty($CFG->sitepolicy)) {
         if (!$USER->policyagreed) {
+            $SESSION->wantsurl = $FULLME;
             redirect($CFG->wwwroot .'/user/policy.php');
             die;
         }

diff --git a/user/policy.php b/user/policy.php
@@ -10,11 +10,20 @@
     }
 
     if ($agree == 1 and confirm_sesskey()) {        // User has agreed
-        if (!set_field('user', 'policyagreed', 1, 'id', $USER->id)) {
-            error('Could not save your agreement');
+        if ($USER->username != 'guest') {           // Don't remember guests
+            if (!set_field('user', 'policyagreed', 1, 'id', $USER->id)) {
+                error('Could not save your agreement');
+            }
         }
         $USER->policyagreed = 1;
-        redirect($CFG->wwwroot);
+
+        if (!empty($SESSION->wantsurl)) {
+            $wantsurl = $SESSION->wantsurl;
+            unset($SESSION->wantsurl);
+            redirect($wantsurl);
+        } else {
+            redirect($CFG->wwwroot.'/');
+        }
         exit;
     }