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

Disabling Marking Allocation and Marking Workflow together in assignment settings leaves Marking Allocation enabled

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create an assignment with both Marking Workflow and Marking Allocation to Yes. Save.
      2. Edit the assignment settings again and change both Use Marking Allocation and Use Marking Workflow (in that order) to No. Save.
      3. Edit the assignment settings one more time and expand the Grade section. Note that Use Marking Allocation is No.
      Show
      Create an assignment with both Marking Workflow and Marking Allocation to Yes. Save. Edit the assignment settings again and change both Use Marking Allocation and Use Marking Workflow (in that order) to No. Save. Edit the assignment settings one more time and expand the Grade section. Note that Use Marking Allocation is No.
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      master_MDL-45715
    • Pull Master Diff URL:

      Description

      For an assignment that has both Marking Allocation and Marking Workflow enabled, disabling both at the same time and saving leaves Marking Allocation enabled.

      Steps to reproduce:

      1. Create an assignment with both Marking Workflow and Marking Allocation to Yes. Save.
      2. Edit the assignment settings again and change both Use Marking Allocation and Use Marking Workflow (in that order) to No. Save.
      3. Edit the assignment settings one more time and expand the Grade section. Note that Use Marking Allocation is still Yes.

      Expected: Marking Allocation and Marking Workflow should both be saved as No.

      Actual: Marking Workflow is saved as No but Marking Allocation is still set to Yes.

      This seems to be because the select box for Marking Allocation is disabled on the page when setting Marking Workflow to No, so the new value of the field is not passed in the form.

        Attachments

          Activity

            People

            Assignee:
            danmarsden Dan Marsden
            Reporter:
            cfollin Chris Follin
            Peer reviewer:
            Damyon Wiese
            Integrator:
            Dan Poltawski
            Tester:
            Frédéric Massart
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              8/Sep/14