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

incorrect param passed to get_grading_manager from navigationlib

XMLWordPrintable

    • MOODLE_27_STABLE
    • MOODLE_26_STABLE, MOODLE_27_STABLE
    • master_MDL-47431
    • Hide

      Add a new assignment to a course and set an advanced grading method to be used (rubrics or marking guide)

      Check to make sure the link to "Advanced grading" appears in the admin block when viewing the assignment.

      Show
      Add a new assignment to a course and set an advanced grading method to be used (rubrics or marking guide) Check to make sure the link to "Advanced grading" appears in the admin block when viewing the assignment.

      in lib/navigationlib.php in function load_activity()

      the following call is made:
      $gradingman = get_grading_manager($cm->context, $cm->modname);

      in load_module_settings() the following call is made:
      $gradingman = get_grading_manager($this->page->cm->context, $this->page->activityname);

      All other uses of get_grading_manager (including the info in it's phpdoc) specifies that the frankenstyle name (mod_assign) should be passed instead.

      I haven't seen an issue with the assign module but I'm working on adding support for advanced grading to the forum module and I noticed this variance.

      We need to fix this to use the correct/consistent use of frankenstyle naming and check to make sure this doesn't cause issues with settings appearing in the navigation block

            danmarsden Dan Marsden
            danmarsden Dan Marsden
            Marina Glancy Marina Glancy
            Damyon Wiese Damyon Wiese
            Damyon Wiese Damyon Wiese
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

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