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

incorrect param passed to get_grading_manager from navigationlib

    XMLWordPrintable

Details

    • 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.

    Description

      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

      Attachments

        Issue Links

          Activity

            People

              danmarsden Dan Marsden
              danmarsden Dan Marsden
              Marina Glancy Marina Glancy
              Damyon Wiese Damyon Wiese
              Damyon Wiese Damyon Wiese
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Sujith Haridasan, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Sujith Haridasan
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                10/Nov/14