Moodle
  1. Moodle
  2. MDL-35662

Grant extension does not work when applied to multiple users in assignment grading interface

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Create an assignment with multiple students and a due date.
      2. Go to the grading table for the assignment, select multiple students and choose Grant extension from the drop down list underneath the table.
      3. Click "Go", accept the confirmation and you will go to the grant extension page.
      4. VALIDATE that the form contains a control for setting the extension date.
      Show
      Create an assignment with multiple students and a due date. Go to the grading table for the assignment, select multiple students and choose Grant extension from the drop down list underneath the table. Click "Go", accept the confirmation and you will go to the grant extension page. VALIDATE that the form contains a control for setting the extension date.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:

      Description

      The batch grant extension feature in the Assignment module is showing the wrong form.

      Replication steps:

      1. Create an assignment with multiple students and a due date.
      2. Go to the grading table for the assignment, select multiple students and choose Grant extension from the drop down list underneath the table.
      3. Click "Go", accept the confirmation and you will go to the grant extension page.

      Expected results: The form contains a control for setting the extension date.

      Actual results: The form contains a drop down list of the grading batch operations.

        Gliffy Diagrams

          Activity

          Hide
          Dan Poltawski added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Hacky, but integrated (master/2.4 only), thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Hacky, but integrated (master/2.4 only), thanks!
          Hide
          Rossiani Wijaya added a comment -

          Tested this issue on 2.4 only.

          It works as expected.

          Test passed.

          Show
          Rossiani Wijaya added a comment - Tested this issue on 2.4 only. It works as expected. Test passed.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Closing as fixed, many thanks for your awesome collaboration.

          Show
          Eloy Lafuente (stronk7) added a comment - Closing as fixed, many thanks for your awesome collaboration.

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: