Skip to content
Browse files

MDL-35055 question import: slight error with the Match grades option.

Even in the 'Error if grade not listed case', it was applying a small
tolerance. In the case of a fuzzy match, it was returning the inexact
grade from the import file, rather than the precise grade that Moodle
was expecting.

That causes problems when the editing form is displayed, because the
value from the database does not match any of the available options, so
the grade is changed to 0%.
  • Loading branch information...
1 parent 4ff0f02 commit 170130d7565a3036b4b9cc82332561892fb9fbd1 @timhunt timhunt committed Aug 24, 2012
Showing with 23 additions and 18 deletions.
  1. +23 −18 lib/questionlib.php
View
41 lib/questionlib.php
@@ -209,39 +209,44 @@ function get_grade_options() {
}
/**
- * match grade options
- * if no match return error or match nearest
+ * Check whether a given grade is one of a list of allowed options. If not,
+ * depending on $matchgrades, either return the nearest match, or return false
+ * to signal an error.
* @param array $gradeoptionsfull list of valid options
* @param int $grade grade to be tested
* @param string $matchgrades 'error' or 'nearest'
- * @return mixed either 'fixed' value or false if erro
+ * @return mixed either 'fixed' value or false if error.
*/
-function match_grade_options($gradeoptionsfull, $grade, $matchgrades='error') {
+function match_grade_options($gradeoptionsfull, $grade, $matchgrades = 'error') {
+
if ($matchgrades == 'error') {
- // if we just need an error...
+ // (Almost) exact match, or an error.
foreach ($gradeoptionsfull as $value => $option) {
- // slightly fuzzy test, never check floats for equality :-)
+ // Slightly fuzzy test, never check floats for equality.
if (abs($grade - $value) < 0.00001) {
- return $grade;
+ return $value; // Be sure the return the proper value.
}
}
- // didn't find a match so that's an error
+ // Didn't find a match so that's an error.
return false;
+
} else if ($matchgrades == 'nearest') {
- // work out nearest value
- $hownear = array();
+ // Work out nearest value
+ $best = false;
+ $bestmismatch = 2;
foreach ($gradeoptionsfull as $value => $option) {
- if ($grade==$value) {
- return $grade;
+ $newmismatch = abs($grade - $value);
+ if ($newmismatch < $bestmismatch) {
+ $best = $value;
+ $bestmismatch = $newmismatch;
}
- $hownear[ $value ] = abs( $grade - $value );
}
- // reverse sort list of deltas and grab the last (smallest)
- asort( $hownear, SORT_NUMERIC );
- reset( $hownear );
- return key( $hownear );
+ return $best;
+
} else {
- return false;
+ // Unknow option passed.
+ throw new coding_exception('Unknown $matchgrades ' . $matchgrades .
+ ' passed to match_grade_options');
}
}

0 comments on commit 170130d

Please sign in to comment.
Something went wrong with that request. Please try again.