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

--filter option in evaluate_model.php is broken

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.7
    • Fix Version/s: 3.7.2
    • Component/s: Analytics
    • Labels:
    • Testing Instructions:
      Hide
      1. Go to CLI and execute php admin/tool/analytics/cli/evaluate_model.php --list
      2. Remember the model id for "Students at risk of dropping out"
      3. Execute php admin/tool/analytics/cli/evaluate_model.php --modelid=THEMODELID --filter=123123123
      4. You SHOULD NOT see any coding_exception (this was the result before this patch) and you SHOULD see a !! No courses to analyse !! message
      Show
      Go to CLI and execute php admin/tool/analytics/cli/evaluate_model.php --list Remember the model id for "Students at risk of dropping out" Execute php admin/tool/analytics/cli/evaluate_model.php --modelid=THEMODELID --filter=123123123 You SHOULD NOT see any coding_exception (this was the result before this patch) and you SHOULD see a !! No courses to analyse !! message
    • Affected Branches:
      MOODLE_37_STABLE
    • Fixed Branches:
      MOODLE_37_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-65993_master
    • Pull Master Diff URL:

      Description

      The --filter option allows users to filter by courseid but it is broken since 3.7.0, it will be replaced by a context filtering in MDL-64739, which is scheduled for 3.8. We can wait for MDL-64739 or fix it before it if people votes for this issue to be resolved.

        Attachments

          Activity

            People

            Assignee:
            dmonllao David Monllaó
            Reporter:
            dmonllao David Monllaó
            Integrator:
            Jun Pataleta
            Tester:
            Janelle Barcega
            Participants:
            Component watchers:
            Elizabeth Dalton, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              9/Sep/19

                Time Tracking

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 40 minutes
                1h 40m