rating MDL-21657 changing from ratings (plural) to rating (singular)
authorAndrew Davis <andrew@affinitysoftware.net>
Tue, 16 Mar 2010 09:37:31 +0000 (09:37 +0000)
committerAndrew Davis <andrew@affinitysoftware.net>
Tue, 16 Mar 2010 09:37:31 +0000 (09:37 +0000)
course/moodleform_mod.php
lang/en_utf8/rating.php
lib/moodlelib.php
lib/outputrenderers.php
mod/data/lib.php
mod/forum/lib.php
mod/glossary/lib.php
rating/index.php

index b525845..10736d3 100644 (file)
@@ -342,7 +342,7 @@ abstract class moodleform_mod extends moodleform {
             }
         }
 
-        if (plugin_supports('mod', $this->_modname, FEATURE_RATINGS, false)) {
+        if (plugin_supports('mod', $this->_modname, FEATURE_RATE, false)) {
             $mform->addElement('header', 'modstandardratings', get_string('ratings', 'ratings'));
 
             //$mform->addElement('checkbox', 'assessed', get_string('allowratings', 'ratings') , get_string('ratingsuse', 'ratings'));
index 21a1046..d082cce 100644 (file)
@@ -1,4 +1,5 @@
 <?php
+$string['rate'] = 'Rate';
 $string['rating'] = 'Rating';
 $string['ratings'] = 'Ratings';
 $string['aggregatetype'] = 'Aggregate type';
index d161ed1..abd25d9 100644 (file)
@@ -364,7 +364,7 @@ define('FEATURE_MODEDIT_DEFAULT_COMPLETION', 'modedit_default_completion');
 
 define('FEATURE_COMMENT', 'comment');
 
-define('FEATURE_RATINGS', 'ratings');
+define('FEATURE_RATE', 'rate');
 
 /** Unspecified module archetype */
 define('MOD_ARCHETYPE_OTHER', 0);
index 0882347..00f3460 100644 (file)
@@ -1244,7 +1244,7 @@ class core_renderer extends renderer_base {
         }
 
         if (empty($strrate)) {
-            $strrate = get_string("rate", "forum");
+            $strrate = get_string("rate", "rating");
         }
 
         $strratings = '';
@@ -1313,13 +1313,16 @@ END;
 
             //generate an array of values for numeric scales
             $scalearray = $rating->settings->scale->scaleitems;
-            if( !is_array($scalearray) && is_int($scalearray) ) {
-                $scalearray = array();
-                for($i=0; $i<=$rating->settings->scale->scaleitems; $i++) {
-                    $scalearray[$i] = $i;
+            if( !is_array($scalearray) ) { //almost certainly a numerical scale
+                $intscalearray = intval($scalearray);//just in case theyve passed "5" instead of 5
+                if( is_int($intscalearray) && $intscalearray>0 ){
+                    $scalearray = array();
+                    for($i=0; $i<=$rating->settings->scale->scaleitems; $i++) {
+                        $scalearray[$i] = $i;
+                    }
                 }
             }
-            
+
             $scalearray = array(RATING_UNSET_RATING => $strrate.'...') + $scalearray;
             $strratings .= html_writer::select($scalearray, 'rating'.$rating->itemid, $rating->rating, false, array('class'=>'postratingmenu ratinginput'));
 
index b88a25a..d86c530 100755 (executable)
@@ -2579,7 +2579,7 @@ function data_supports($feature) {
         case FEATURE_GRADE_HAS_GRADE:         return true;
         case FEATURE_GRADE_OUTCOMES:          return true;
         case FEATURE_MOD_SUBPLUGINS:          return array('datafield'=>'mod/data/field', 'datapreset'=>'mod/data/preset');
-        case FEATURE_RATINGS:                 return true;
+        case FEATURE_RATE:                 return true;
 
         default: return null;
     }
index 3c85af3..f79ba8c 100644 (file)
@@ -276,7 +276,7 @@ function forum_supports($feature) {
         case FEATURE_COMPLETION_HAS_RULES:    return true;
         case FEATURE_GRADE_HAS_GRADE:         return true;
         case FEATURE_GRADE_OUTCOMES:          return true;
-        case FEATURE_RATINGS:                 return true;
+        case FEATURE_RATE:                 return true;
 
         default: return null;
     }
index 2fc8eda..f382b49 100644 (file)
@@ -2667,7 +2667,7 @@ function glossary_supports($feature) {
         case FEATURE_COMPLETION_TRACKS_VIEWS: return true;
         case FEATURE_GRADE_HAS_GRADE:         return true;
         case FEATURE_GRADE_OUTCOMES:          return true;
-        case FEATURE_RATINGS:                 return true;
+        case FEATURE_RATE:                 return true;
 
         default: return null;
     }
index 75e08cf..88a2ce7 100644 (file)
@@ -45,10 +45,10 @@ $PAGE->set_url($url);
 
 $permissions = rating::get_rating_permissions(context);
 if (!$permissions[RATING_VIEW]) {
-    print_error('noviewrate', 'ratings');
+    print_error('noviewrate', 'rating');
 }
 if (!$permissions[RATING_VIEW_ALL] and $USER->id != $item->userid) {
-    print_error('noviewanyrate', 'ratings');
+    print_error('noviewanyrate', 'rating');
 }
 
 switch ($sort) {
@@ -59,8 +59,8 @@ switch ($sort) {
 
 $scalemenu = make_grades_menu($scaleid);
 
-$strratings = get_string('ratings', 'ratings');
-$strrating  = get_string('rating', 'ratings');
+$strratings = get_string('ratings', 'rating');
+$strrating  = get_string('rating', 'rating');
 $strname    = get_string('name');
 $strtime    = get_string('time');