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

Deleting assignment feedback plugins deletes config for submission plugin instead

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.1
    • Fix Version/s: 2.3.2
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Go to "Site administration" => "Plugins" => "Assignment Plugins" => "Feedback Plugins" => "Manage assignment feedback plugins"
      2. Delete the "Feedback file" plugin
      3. Go to "Site administration" => "Notifications"
      4. Reinstall the plugin
      5. Verify the plugin that gets re-installed is "assignfeedback_file" and not "assignsubmission_file"
      Show
      Go to "Site administration" => "Plugins" => "Assignment Plugins" => "Feedback Plugins" => "Manage assignment feedback plugins" Delete the "Feedback file" plugin Go to "Site administration" => "Notifications" Reinstall the plugin Verify the plugin that gets re-installed is "assignfeedback_file" and not "assignsubmission_file"
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:

      Description

      In the new assignment module you can delete individual feedback plugins. This contains a bug and will delete the corresponding submission plugin instead (e.g. deleting assignfeedback_file deletes the assignsubmission_file plugin instead).

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            poltawski Dan Poltawski added a comment -

            Looks good, +1

            Show
            poltawski Dan Poltawski added a comment - Looks good, +1
            Hide
            poltawski 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
            poltawski 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
            samhemelryk Sam Hemelryk added a comment -

            Changes look spot on thanks Damyon, have integrated this now.

            Show
            samhemelryk Sam Hemelryk added a comment - Changes look spot on thanks Damyon, have integrated this now.
            Hide
            timb Tim Barker added a comment -

            Congrats, the test passed1

            Show
            timb Tim Barker added a comment - Congrats, the test passed1
            Hide
            nebgor Aparup Banerjee added a comment -

            yay, it works!

            This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

            Thank you all for taking the time to get us here.

            cheers!

            Show
            nebgor Aparup Banerjee added a comment - yay, it works! This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week. Thank you all for taking the time to get us here. cheers!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Sep/12