MDL-27625 rating - consider RATING_UNSET_RATING a valid rating value

This commit is contained in:
Eloy Lafuente (stronk7) 2011-05-25 12:00:16 +02:00
parent d04f7fb4a8
commit 5693d56cd8
3 changed files with 21 additions and 6 deletions

View file

@ -1420,9 +1420,14 @@ function data_rating_validate($params) {
} }
//check that the submitted rating is valid for the scale //check that the submitted rating is valid for the scale
if ($params['rating'] < 0) {
// lower limit
if ($params['rating'] < 0 && $params['rating'] != RATING_UNSET_RATING) {
throw new rating_exception('invalidnum'); throw new rating_exception('invalidnum');
} else if ($info->scale < 0) { }
// upper limit
if ($info->scale < 0) {
//its a custom scale //its a custom scale
$scalerecord = $DB->get_record('scale', array('id' => -$params['scaleid'])); $scalerecord = $DB->get_record('scale', array('id' => -$params['scaleid']));
if ($scalerecord) { if ($scalerecord) {

View file

@ -3513,9 +3513,14 @@ function forum_rating_validate($params) {
} }
//check that the submitted rating is valid for the scale //check that the submitted rating is valid for the scale
if ($params['rating'] < 0) {
// lower limit
if ($params['rating'] < 0 && $params['rating'] != RATING_UNSET_RATING) {
throw new rating_exception('invalidnum'); throw new rating_exception('invalidnum');
} else if ($info->scale < 0) { }
// upper limit
if ($info->scale < 0) {
//its a custom scale //its a custom scale
$scalerecord = $DB->get_record('scale', array('id' => -$params['scaleid'])); $scalerecord = $DB->get_record('scale', array('id' => -$params['scaleid']));
if ($scalerecord) { if ($scalerecord) {

View file

@ -526,9 +526,14 @@ function glossary_rating_validate($params) {
} }
//check that the submitted rating is valid for the scale //check that the submitted rating is valid for the scale
if ($params['rating'] < 0) {
// lower limit
if ($params['rating'] < 0 && $params['rating'] != RATING_UNSET_RATING) {
throw new rating_exception('invalidnum'); throw new rating_exception('invalidnum');
} else if ($info->scale < 0) { }
// upper limit
if ($info->scale < 0) {
//its a custom scale //its a custom scale
$scalerecord = $DB->get_record('scale', array('id' => -$params['scaleid'])); $scalerecord = $DB->get_record('scale', array('id' => -$params['scaleid']));
if ($scalerecord) { if ($scalerecord) {