Merge branch 'MDL-27675_master' of git://github.com/grabs/moodle
authorSam Hemelryk <sam@moodle.com>
Wed, 20 Jun 2012 23:25:07 +0000 (11:25 +1200)
committerSam Hemelryk <sam@moodle.com>
Wed, 20 Jun 2012 23:25:07 +0000 (11:25 +1200)
1  2 
mod/feedback/item/captcha/lib.php
mod/feedback/item/feedback_item_class.php
mod/feedback/item/info/lib.php
mod/feedback/item/multichoicerated/lib.php
mod/feedback/item/numeric/lib.php
mod/feedback/item/textarea/lib.php
mod/feedback/item/textfield/lib.php

@@@ -326,4 -326,12 +326,12 @@@ class feedback_item_captcha extends fee
      public function can_switch_require() {
          return false;
      }
 -    function clean_input_value($value) {
+     public function value_type() {
+         return PARAM_RAW;
+     }
++    public function clean_input_value($value) {
+         return clean_param($value, $this->value_type());
+     }
  }
@@@ -128,6 -125,14 +128,14 @@@ abstract class feedback_item_base 
       */
      abstract public function print_item_show_value($item, $value = '');
  
 -    abstract function clean_input_value($value);
+     /**     
+      * cleans the userinput while submitting the form
+      *
+      * @param mixed $value
+      * @return mixed
+      */
++    abstract public function clean_input_value($value);
  }
  
  //a dummy class to realize pagebreaks
@@@ -388,4 -388,12 +388,12 @@@ class feedback_item_info extends feedba
      public function can_switch_require() {
          return false;
      }
 -    function clean_input_value($value) {
+     public function value_type() {
+         return PARAM_INT;
+     }
++    public function clean_input_value($value) {
+         return clean_param($value, $this->value_type());
+     }
  }
@@@ -678,4 -678,11 +678,11 @@@ class feedback_item_multichoicerated ex
          return true;
      }
  
 -    function clean_input_value($value) {
+     public function value_type() {
+         return PARAM_INT;
+     }
++    public function clean_input_value($value) {
+         return clean_param($value, $this->value_type());
+     }
  }
@@@ -534,4 -534,15 +534,15 @@@ class feedback_item_numeric extends fee
      public function can_switch_require() {
          return true;
      }
 -    function clean_input_value($value) {
+     
+     public function value_type() {
+         return PARAM_FLOAT;
+     }
++    public function clean_input_value($value) {
+         if (!is_numeric($value)) {
+             return null;
+         }
+         return clean_param($value, $this->value_type());
+     }
  }
@@@ -333,4 -333,12 +333,12 @@@ class feedback_item_textarea extends fe
      public function can_switch_require() {
          return true;
      }
 -    function clean_input_value($value) {
+     public function value_type() {
+         return PARAM_RAW;
+     }
++    public function clean_input_value($value) {
+         return s($value);
+     }
  }
@@@ -320,4 -320,12 +320,12 @@@ class feedback_item_textfield extends f
      public function can_switch_require() {
          return true;
      }
 -    function clean_input_value($value) {
+     public function value_type() {
+         return PARAM_RAW;
+     }
++    public function clean_input_value($value) {
+         return s($value);
+     }
  }