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

Upgrade helper sets 'file submission' to yes for online text and offline assignment types

    Details

    • Testing Instructions:
      Hide
      1. Go to Site administration -> Plugins -> Assignment plugins -> Submission plugins -> Manage submission plugins
      2. Set all the submission plugins to visible and enabled by default (in the settings for each plugin).
      3. Enable the "Assignment (2.2)" activity
      4. Create an "Advanced uploading of files" assignment in a course
      5. Use the Assignment upgrade tool to upgrade the new assignment
      6. View the settings for the new assignment
      7. Verify that "Online text" is "No", "File submissions" is "Yes" and "Submission comments" is "Yes"
      Show
      Go to Site administration -> Plugins -> Assignment plugins -> Submission plugins -> Manage submission plugins Set all the submission plugins to visible and enabled by default (in the settings for each plugin). Enable the "Assignment (2.2)" activity Create an "Advanced uploading of files" assignment in a course Use the Assignment upgrade tool to upgrade the new assignment View the settings for the new assignment Verify that "Online text" is "No", "File submissions" is "Yes" and "Submission comments" is "Yes"
    • Workaround:
      Hide

      Change the assignment settings to disable unwanted submission types after it has been upgraded.

      A workaround for a mass upgrade would be to disable all the new assignment plugins types by default, do the mass upgrade, then set them back to whatever they were before.

      Show
      Change the assignment settings to disable unwanted submission types after it has been upgraded. A workaround for a mass upgrade would be to disable all the new assignment plugins types by default, do the mass upgrade, then set them back to whatever they were before.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-34052-master

      Description

      When upgrading assignments from older versions of Moodle, the upgrade helper in Moodle 2.3 sets the setting 'file submissions' setting to "Yes" (plus Max files to 1 and max submission size to course upload limit) for the online text and offline assignment types. Expected behaviour is to upgrade the 'file submission' setting to "No" for both of these assignment types.

      Steps to recreate:

      • create offline or online text assignment (or restore from backup)
      • login as admin and upgrade assignment using upgrade helper
      • view settings of upgraded assignment - scroll to "File submission" setting

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            lienkalenka Lenka Kolesarova added a comment -

            I don´t understand: why is this bug created more than a month ago considered minor? In the Settings mapping section of the documentation for the Upgrade tool (http://docs.moodle.org/23/en/Upgrade_tool#Settings_Mapping) the expected functionality is clearly stated "For a Single File Upload or Advanced Uploading of Files assignment type this setting will upgrade to Yes. For all other subtypes it will upgrade to No." Yet it does not work. For us it is a blocker that makes it impossible to upgrade all the old assignment subtypes and start using the new assignment site-wide.

            Show
            lienkalenka Lenka Kolesarova added a comment - I don´t understand: why is this bug created more than a month ago considered minor? In the Settings mapping section of the documentation for the Upgrade tool ( http://docs.moodle.org/23/en/Upgrade_tool#Settings_Mapping ) the expected functionality is clearly stated "For a Single File Upload or Advanced Uploading of Files assignment type this setting will upgrade to Yes. For all other subtypes it will upgrade to No." Yet it does not work. For us it is a blocker that makes it impossible to upgrade all the old assignment subtypes and start using the new assignment site-wide.
            Hide
            pjfish06 Susan Mangan added a comment -

            Agreed. I just now realized this after upgrading thousands of offline assignments... sigh

            Show
            pjfish06 Susan Mangan added a comment - Agreed. I just now realized this after upgrading thousands of offline assignments... sigh
            Hide
            pjfish06 Susan Mangan added a comment -

            I wonder if there is any way to track these and do a mass update ... anyone?

            Show
            pjfish06 Susan Mangan added a comment - I wonder if there is any way to track these and do a mass update ... anyone?
            Hide
            corleone Raymond Antonio added a comment -

            Hi Damyon,
            There is a patch for this bug on my Github link above ready for peer review. Thanks Damyon

            Show
            corleone Raymond Antonio added a comment - Hi Damyon, There is a patch for this bug on my Github link above ready for peer review. Thanks Damyon
            Hide
            damyon Damyon Wiese added a comment -

            I ammended Lukes commit to fix up the format of the git message and added a 2.4 branch.

            Show
            damyon Damyon Wiese added a comment - I ammended Lukes commit to fix up the format of the git message and added a 2.4 branch.
            Hide
            damyon Damyon Wiese added a comment -

            Thanks Grette, Raymond and Luke - this is ready for integration.

            Show
            damyon Damyon Wiese added a comment - Thanks Grette, Raymond and Luke - this is ready for integration.
            Hide
            damyon Damyon Wiese added a comment -

            This patch disables plugins that do not report that they know how to upgrade a particular type of the old assignment. Previously this was getting left at whatever the site defaults were for that type of assignment plugin.

            Show
            damyon Damyon Wiese added a comment - This patch disables plugins that do not report that they know how to upgrade a particular type of the old assignment. Previously this was getting left at whatever the site defaults were for that type of assignment plugin.
            Hide
            poltawski Dan Poltawski added a comment -

            Integrated to master, 24 and 23, thanks guys

            Show
            poltawski Dan Poltawski added a comment - Integrated to master, 24 and 23, thanks guys
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success!

            Tested in 2.3 and master.

            Show
            salvetore Michael de Raadt added a comment - Test result: Success! Tested in 2.3 and master.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Many thanks for your effort, the whole Moodle Community will be enjoying your great solutions starting now!

            Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Many thanks for your effort, the whole Moodle Community will be enjoying your great solutions starting now! Closing, ciao

              People

              • Votes:
                3 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/Jan/13