MDL-57791 analytics: Add missing 'new' to throw
authorDavid Monllao <davidm@moodle.com>
Fri, 23 Jun 2017 06:30:50 +0000 (08:30 +0200)
committerDavid Monllao <davidm@moodle.com>
Mon, 24 Jul 2017 06:37:01 +0000 (08:37 +0200)
Also other code refinements. Credit to Mark Nelson.

analytics/classes/calculable.php
analytics/classes/manager.php
analytics/classes/model.php

index 72813d7..cda4fc3 100644 (file)
@@ -208,7 +208,7 @@ abstract class calculable {
             $match = false;
 
             if (count($range) != 2) {
-                throw \coding_exception('classify_value() $ranges array param should contain 2 items, the predicate ' .
+                throw new \coding_exception('classify_value() $ranges array param should contain 2 items, the predicate ' .
                     'e.g. greater (gt), lower or equal (le)... and the value.');
             }
 
index afea72d..3bc24bd 100644 (file)
@@ -76,7 +76,7 @@ class manager {
      *
      * @param bool $enabled
      * @param bool $trained
-     * @param \context $predictioncontext
+     * @param \context|false $predictioncontext
      * @return \core_analytics\model[]
      */
     public static function get_all_models($enabled = false, $trained = false, $predictioncontext = false) {
@@ -163,7 +163,7 @@ class manager {
     /**
      * Return all system predictions processors.
      *
-     * @return \core_analytics\predictor
+     * @return \core_analytics\predictor[]
      */
     public static function get_all_prediction_processors() {
 
index 2999767..461daec 100644 (file)
@@ -145,7 +145,6 @@ class model {
         if (!$target) {
             return false;
         }
-        $analyser = $this->get_target();
 
         $classname = $target->get_analyser_class();
         if (!class_exists($classname)) {
@@ -296,7 +295,7 @@ class model {
 
         $classname = $target->get_analyser_class();
         if (!class_exists($classname)) {
-            throw \coding_exception($classname . ' class does not exists');
+            throw new \coding_exception($classname . ' class does not exists');
         }
 
         // Returns a \core_analytics\local\analyser\base class.