From: agrabs <agrabs>
Date: Wed, 16 Sep 2009 12:21:47 +0000 (+0000)
Subject: MDL-19773 - Cannot have code like $SESSION->feedback = new object(); in mod/feedback... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=f1b9e2df43c63a1dcbfacb2f7fc22dea9ad11698;p=moodle.git

MDL-19773 - Cannot have code like $SESSION->feedback = new object(); in mod/feedback/lib.php
---

diff --git a/mod/feedback/complete.php b/mod/feedback/complete.php
index 019eca35de..c0976970cd 100644
--- a/mod/feedback/complete.php
+++ b/mod/feedback/complete.php
@@ -12,6 +12,8 @@
     require_once("lib.php");
     require_once($CFG->libdir . '/completionlib.php');
 
+    feedback_init_feedback_session();
+
     $id = required_param('id', PARAM_INT);
     $completedid = optional_param('completedid', false, PARAM_INT);
     $preservevalues  = optional_param('preservevalues', 0,  PARAM_INT);
diff --git a/mod/feedback/complete_guest.php b/mod/feedback/complete_guest.php
index 0d196d0492..63a9e87b41 100644
--- a/mod/feedback/complete_guest.php
+++ b/mod/feedback/complete_guest.php
@@ -11,6 +11,8 @@
     require_once("../../config.php");
     require_once("lib.php");
 
+    feedback_init_feedback_session();
+
     $id = required_param('id', PARAM_INT);
     $completedid = optional_param('completedid', false, PARAM_INT);
     $preservevalues  = optional_param('preservevalues', 0,  PARAM_INT);
diff --git a/mod/feedback/edit.php b/mod/feedback/edit.php
index ffe41f70e4..c778d6b407 100644
--- a/mod/feedback/edit.php
+++ b/mod/feedback/edit.php
@@ -12,6 +12,8 @@
     require_once("lib.php");
     require_once('edit_form.php');
 
+    feedback_init_feedback_session();
+
     $id = required_param('id', PARAM_INT);
 
     if(($formdata = data_submitted()) AND !confirm_sesskey()) {
diff --git a/mod/feedback/edit_item.php b/mod/feedback/edit_item.php
index 46359c4247..244c7f57be 100644
--- a/mod/feedback/edit_item.php
+++ b/mod/feedback/edit_item.php
@@ -11,6 +11,8 @@
     require_once("../../config.php");
     require_once("lib.php");
 
+    feedback_init_feedback_session();
+
     $id = optional_param('id', NULL, PARAM_INT);
     $typ = optional_param('typ', false, PARAM_ALPHA);
     $itemid = optional_param('itemid', false, PARAM_INT);
diff --git a/mod/feedback/lib.php b/mod/feedback/lib.php
index 2f7b286283..2f6455c418 100644
--- a/mod/feedback/lib.php
+++ b/mod/feedback/lib.php
@@ -38,14 +38,6 @@ define('FEEDBACK_RESETFORM_RESET', 'feedback_reset_data_');
 define('FEEDBACK_RESETFORM_DROP', 'feedback_drop_feedback_');
 define('FEEDBACK_MAX_PIX_LENGTH', '400'); //max. Breite des grafischen Balkens in der Auswertung
 
-//initialize the feedback-Session - not nice at all!!
-global $SESSION;
-if (!empty($SESSION)) {
-    if (!isset($SESSION->feedback) OR !is_object($SESSION->feedback)) {
-        $SESSION->feedback = new object();
-    }
-}
-
 /**
  * @uses FEATURE_GROUPS
  * @uses FEATURE_GROUPINGS
@@ -2299,6 +2291,8 @@ function feedback_print_errors() {
 
     global $SESSION, $OUTPUT;
 
+    feedback_init_feedback_session();
+
     if(empty($SESSION->feedback->errors)) {
 		return;
     }
@@ -2361,4 +2355,14 @@ function feedback_extend_settings_navigation($settings, $module) {
     if (has_capability('moodle/course:manageactivities', $PAGE->cm->context)) {
         $feedbacknav->add(get_string('updatethis', '', get_string('modulename', 'feedback')), new moodle_url($CFG->wwwroot.'/course/mod.php', array('update' => $PAGE->cm->id, 'return' => true, 'sesskey' => sesskey())));
     }
+}
+
+function feedback_init_feedback_session() {
+    //initialize the feedback-Session - not nice at all!!
+    global $SESSION;
+    if (!empty($SESSION)) {
+        if (!isset($SESSION->feedback) OR !is_object($SESSION->feedback)) {
+            $SESSION->feedback = new object();
+        }
+    }
 }
\ No newline at end of file