diff --git a/mod/feedback/complete.php b/mod/feedback/complete.php index 7f81b9b9a04..726c08db886 100644 --- a/mod/feedback/complete.php +++ b/mod/feedback/complete.php @@ -422,7 +422,8 @@ //get the value $frmvaluename = $feedbackitem->typ . '_'. $feedbackitem->id; if(isset($savereturn)) { - $value = isset($formdata->{$frmvaluename})?$formdata->{$frmvaluename}:NULL; + $value = isset($formdata->{$frmvaluename})?$formdata->{$frmvaluename}:NULL; + $value = feedback_clean_input_value($feedbackitem, $value); }else { if(isset($feedbackcompletedtmp->id)) { $value = feedback_get_item_value($feedbackcompletedtmp->id, $feedbackitem->id, true); diff --git a/mod/feedback/complete_guest.php b/mod/feedback/complete_guest.php index a25946d2920..ed7f054ba99 100644 --- a/mod/feedback/complete_guest.php +++ b/mod/feedback/complete_guest.php @@ -54,7 +54,7 @@ //if the use hit enter into a textfield so the form should not submit if(isset($formdata->sesskey) AND !isset($formdata->savevalues) AND !isset($formdata->gonextpage) AND !isset($formdata->gopreviouspage)) { - $gopage = $formdata->lastpage; + $gopage = (int)$formdata->lastpage; } if(isset($formdata->savevalues)) { $savevalues = true; @@ -372,7 +372,8 @@ //get the value $frmvaluename = $feedbackitem->typ . '_'. $feedbackitem->id; if(isset($savereturn)) { - $value = isset($formdata->{$frmvaluename})?$formdata->{$frmvaluename}:NULL; + $value = isset($formdata->{$frmvaluename})?$formdata->{$frmvaluename}:NULL; + $value = feedback_clean_input_value($feedbackitem, $value); }else { if(isset($feedbackcompletedtmp->id)) { $value = feedback_get_item_value($feedbackcompletedtmp->id, $feedbackitem->id, sesskey()); diff --git a/mod/feedback/item/captcha/lib.php b/mod/feedback/item/captcha/lib.php index 0dcca48e483..8f6ade08ac2 100644 --- a/mod/feedback/item/captcha/lib.php +++ b/mod/feedback/item/captcha/lib.php @@ -280,4 +280,8 @@ class feedback_item_captcha extends feedback_item_base { function can_switch_require() { return false; } + + function clean_input_value($value) { + return clean_param($value, PARAM_RAW); + } } diff --git a/mod/feedback/item/feedback_item_class.php b/mod/feedback/item/feedback_item_class.php index ddcfcd71496..437c82066a8 100644 --- a/mod/feedback/item/feedback_item_class.php +++ b/mod/feedback/item/feedback_item_class.php @@ -104,6 +104,14 @@ abstract class feedback_item_base { */ abstract function print_item_show_value($item, $value = ''); + /** + * cleans the userinput while submitting the form + * + * @param mixed $value + * @return mixed + */ + abstract function clean_input_value($value); + } //a dummy class to realize pagebreaks @@ -129,6 +137,7 @@ class feedback_item_pagebreak extends feedback_item_base { function print_item_complete($item, $value = '', $highlightrequire = false) {} function print_item_show_value($item, $value = '') {} function can_switch_require(){} + function clean_input_value($value){} } diff --git a/mod/feedback/item/info/lib.php b/mod/feedback/item/info/lib.php index b6311a84eb3..32ecbff0b16 100644 --- a/mod/feedback/item/info/lib.php +++ b/mod/feedback/item/info/lib.php @@ -321,4 +321,8 @@ class feedback_item_info extends feedback_item_base { function can_switch_require() { return false; } + + function clean_input_value($value) { + return clean_param($value, PARAM_INT); + } } diff --git a/mod/feedback/item/label/lib.php b/mod/feedback/item/label/lib.php index 5e5df4c6478..1c09e01d2df 100644 --- a/mod/feedback/item/label/lib.php +++ b/mod/feedback/item/label/lib.php @@ -231,4 +231,8 @@ class feedback_item_label extends feedback_item_base { function print_analysed($item, $itemnr = '', $groupid = false, $courseid = false) {} function get_printval($item, $value) {} function get_analysed($item, $groupid = false, $courseid = false) {} + + function clean_input_value($value) { + return ''; + } } diff --git a/mod/feedback/item/multichoice/lib.php b/mod/feedback/item/multichoice/lib.php index 30402b26ecc..7f097baf613 100644 --- a/mod/feedback/item/multichoice/lib.php +++ b/mod/feedback/item/multichoice/lib.php @@ -709,8 +709,11 @@ class feedback_item_multichoice extends feedback_item_base { return false; } - function can_switch_require() { return true; } + + function clean_input_value($value) { + return clean_param($value, PARAM_INT); + } } diff --git a/mod/feedback/item/multichoicerated/lib.php b/mod/feedback/item/multichoicerated/lib.php index 41764e9a463..885a0f07153 100644 --- a/mod/feedback/item/multichoicerated/lib.php +++ b/mod/feedback/item/multichoicerated/lib.php @@ -588,4 +588,7 @@ class feedback_item_multichoicerated extends feedback_item_base { return true; } + function clean_input_value($value) { + return clean_param($value, PARAM_INT); + } } diff --git a/mod/feedback/item/numeric/lib.php b/mod/feedback/item/numeric/lib.php index cf646d1348d..168fb31dc2b 100644 --- a/mod/feedback/item/numeric/lib.php +++ b/mod/feedback/item/numeric/lib.php @@ -282,16 +282,16 @@ class feedback_item_numeric extends feedback_item_base { } echo ''; echo ''; - + //print the presentation echo '
'; echo ''; - echo ''; + echo ''; echo ''; echo '
'; } - /** + /** * print the item at the complete-page of feedback * * @global object @@ -416,8 +416,15 @@ class feedback_item_numeric extends feedback_item_base { function get_hasvalue() { return 1; } - + function can_switch_require() { return true; } + + function clean_input_value($value) { + if (!is_numeric($value)) { + return null; + } + return clean_param($value, PARAM_FLOAT); + } } diff --git a/mod/feedback/item/textarea/lib.php b/mod/feedback/item/textarea/lib.php index 1b4cd134423..6c26ef6bf95 100644 --- a/mod/feedback/item/textarea/lib.php +++ b/mod/feedback/item/textarea/lib.php @@ -215,7 +215,7 @@ class feedback_item_textarea extends feedback_item_base { //print the presentation echo '
'; echo ''; - echo ''; + echo ''; echo ''; echo '
'; } @@ -255,7 +255,7 @@ class feedback_item_textarea extends feedback_item_base { } function create_value($data) { - $data = clean_text($data); + $data = s($data); return $data; } @@ -280,4 +280,8 @@ class feedback_item_textarea extends feedback_item_base { function can_switch_require() { return true; } + + function clean_input_value($value) { + return s($value); + } } diff --git a/mod/feedback/item/textfield/lib.php b/mod/feedback/item/textfield/lib.php index 74f9b88657f..efb844b7d4d 100644 --- a/mod/feedback/item/textfield/lib.php +++ b/mod/feedback/item/textfield/lib.php @@ -211,7 +211,7 @@ class feedback_item_textfield extends feedback_item_base { //print the presentation echo '
'; echo ''; - echo ''; + echo ''; echo ''; echo '
'; } @@ -249,7 +249,7 @@ class feedback_item_textfield extends feedback_item_base { } function create_value($data) { - $data = clean_text($data); + $data = s($data); return $data; } @@ -274,4 +274,8 @@ class feedback_item_textfield extends feedback_item_base { function can_switch_require() { return true; } + + function clean_input_value($value) { + return s($value); + } } diff --git a/mod/feedback/lib.php b/mod/feedback/lib.php index 59c39b62d19..5ed9bc2b241 100644 --- a/mod/feedback/lib.php +++ b/mod/feedback/lib.php @@ -1841,6 +1841,17 @@ function feedback_get_page_to_continue($feedbackid, $courseid = false, $guestid //functions to handle the values //////////////////////////////////////////////// +/** + * cleans the userinput while submitting the form. + * + * @param mixed $value + * @return mixed + */ +function feedback_clean_input_value($item, $value) { + $itemobj = feedback_get_item_class($item->typ); + return $itemobj->clean_input_value($value); +} + /** * this saves the values of an completed. * if the param $tmp is set true so the values are saved temporary in table feedback_valuetmp.