Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-71973

Exception thrown when evaluating disabled models from the CLI

XMLWordPrintable

    • MOODLE_310_STABLE, MOODLE_311_STABLE, MOODLE_400_STABLE
    • MOODLE_310_STABLE, MOODLE_311_STABLE
    • Hide
      1. You must have a moodle instance with some data.
      2. By running the below command choose a model that is disabled. (We choose "Students at risk of dropping out" id=1 for example) 

        php admin/tool/analytics/cli/evaluate_model.php --list

      3. Evaluate it by running command below. 

        php admin/tool/analytics/cli/evaluate_model.php --modelid=1

        After evaluating, the cli asks to specify the time splitting (like below)

      4. We can go to the next step by typing 

        \core\analytics\time_splitting\single_range

      5. If the patch applied we must not see the below output 

        : \core\analytics\time_splitting\single_range
        		 
        !!! Exception - Call to a mamber function based_on_assumptions() on bool!!!

      6. If exception occurs it means either the patch isn't useful or doesn't apply properly.*
      Show
      You must have a moodle instance with some data. By running the below command choose a model that is disabled. (We choose "Students at risk of dropping out" id=1 for example) php admin/tool/analytics/cli/evaluate_model.php --list Evaluate it by running command below. php admin/tool/analytics/cli/evaluate_model.php --modelid= 1 After evaluating, the cli asks to specify the time splitting (like below) We can go to the next step by typing \core\analytics\time_splitting\single_range If the patch applied we must not see the below output : \core\analytics\time_splitting\single_range !!! Exception - Call to a mamber function based_on_assumptions() on bool!!! If exception occurs it means either the patch isn't useful or doesn't apply properly.*

      Bug reported by mohalisad in MDL-67249 comments.

      When you run model evaluation script from CLI and the model is disabled it should interactively enables your model.

      But it has a bug and an exception occurs in process:

      !!! Exception - Call to a member function based_on_assumptions() on bool !!!

      It happens in here (analytics/classes/model.php: line 1204):

      The get_target method returns false and that causes this exception.

      This problem is due to the fact that in evaluate_model.php cli file we have an undefined variable(admin/tool/analytics/cli/evaluate_model.php):

      The $modelobj is never referenced before this line therefore the $model constructed wrongly in line 140 and in line 143 that causes an exception.

      Stacktrace:

       

      Steps to reproduce:

      1. Go to Analytics model page and choose a model than could be evaluated e.g. 'Students at risk of dropping out'. Remember the id of the model and disable it.
      2. Run 'php admin/tool/analytics/cli/evaluate_model.php --modelid=xx' using modelid of the previous model.
      3. Use a valid subclass of '\core_analytics\local\time_splitting\base' as an analysis interval. e.g. '\core\analytics\time_splitting\single_range'
      4. Script should finish with no error

        1. image-2021-06-18-16-03-58-862.png
          image-2021-06-18-16-03-58-862.png
          46 kB
        2. image-2021-06-18-17-39-00-196.png
          image-2021-06-18-17-39-00-196.png
          66 kB
        3. image-2021-06-18-17-39-10-037.png
          image-2021-06-18-17-39-10-037.png
          47 kB
        4. image-2021-06-18-17-39-18-920.png
          image-2021-06-18-17-39-18-920.png
          55 kB
        5. image-2021-06-20-22-55-11-313.png
          image-2021-06-20-22-55-11-313.png
          84 kB
        6. image-2021-06-20-22-56-27-089.png
          image-2021-06-20-22-56-27-089.png
          84 kB
        7. image-2021-06-20-23-01-04-597.png
          image-2021-06-20-23-01-04-597.png
          27 kB
        8. image-2021-06-20-23-10-22-172.png
          image-2021-06-20-23-10-22-172.png
          22 kB
        9. image-2021-06-21-18-56-27-884.png
          image-2021-06-21-18-56-27-884.png
          56 kB

            mohalisad MohammadAli Sadraei
            amaia Amaia Anabitarte
            Simey Lameze Simey Lameze
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 6 hours, 37 minutes
                6h 37m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.