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

2.4 restore does not honor site's quiz overduehandling setting when restoring a 2.2 backup

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.4.6, 2.5
          • Fix Version/s: 2.4.7, 2.5.3, 2.6
          • Component/s: Quiz
          • Labels:
          • Testing Instructions:
            Hide

            1. Get a backup of a quiz from Moodle 2.2 or earlier.

            2. Go to Admin -> Plugins -> Activity modules -> Quiz and set When time expires to Open attempts are submitted automatically.

            3. Restore the backup. Make sure that, in the settings for the restored quiz, When time expires is set to Open attempts are submitted automatically.

            4. Create a quiz in your up-to-date Moodle, with When time expires is set to Attempts must be submitted before time expires, or they are not counted.

            5. Backup that quiz, and restore it, and verify that the When time expires settings has been copied accurately.

            Show
            1. Get a backup of a quiz from Moodle 2.2 or earlier. 2. Go to Admin -> Plugins -> Activity modules -> Quiz and set When time expires to Open attempts are submitted automatically. 3. Restore the backup. Make sure that, in the settings for the restored quiz, When time expires is set to Open attempts are submitted automatically. 4. Create a quiz in your up-to-date Moodle, with When time expires is set to Attempts must be submitted before time expires, or they are not counted. 5. Backup that quiz, and restore it, and verify that the When time expires settings has been copied accurately.
          • Affected Branches:
            MOODLE_24_STABLE, MOODLE_25_STABLE
          • Fixed Branches:
            MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE
          • Pull from Repository:

            Description

            Since 2.2 does not have a quiz overduehandling setting, 2.2 course backups do not contain the setting. When restoring a 2.2 course backup into a 2.4 instance, the restore process should apply the site-level default for the overduehandling setting, but it does not. Instead, it always leaves it at the database default, which is to abandon late submissions.

            I will attach a patch shortly.

              Gliffy Diagrams

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  timhunt Tim Hunt added a comment -

                  Peer review comments.

                  1. It seems that get_config now has caching, so calling it repeatedly is not too bad.

                  2. So, the patch looks fine, but the commit comment is not good. See http://docs.moodle.org/dev/Coding_style#Git_commits.

                  3. Also, this needs to be back-ported to stable branches, since it is a bug fix.

                  Can you fix that up, then this is ready for integration.

                  Show
                  timhunt Tim Hunt added a comment - Peer review comments. 1. It seems that get_config now has caching, so calling it repeatedly is not too bad. 2. So, the patch looks fine, but the commit comment is not good. See http://docs.moodle.org/dev/Coding_style#Git_commits . 3. Also, this needs to be back-ported to stable branches, since it is a bug fix. Can you fix that up, then this is ready for integration.
                  Hide
                  Permalink
                  colin Colin Campbell added a comment -

                  I fixed the commit comment and added a branch for 2.5. Besides 2.4 and 2.5, I believe there are no other stable branches.

                  Show
                  colin Colin Campbell added a comment - I fixed the commit comment and added a branch for 2.5. Besides 2.4 and 2.5, I believe there are no other stable branches.
                  Hide
                  Permalink
                  timhunt Tim Hunt added a comment -

                  That looks gread now. Just two things:

                  • We need some testing instructions. I just wrote those, but feel free to edit them if you think they can be improved.
                  • You should have made a branch based on master. Shall we rely on integrators cherry-picking it? However, it would be good if you could create such a branch before the integrators get here, and please remember this for future bug fixes.

                  Thanks. Submitting for integration now.

                  To INTEGRATORS: Please cherry pick to master form the 2.5 branch, if necessary.

                  Show
                  timhunt Tim Hunt added a comment - That looks gread now. Just two things: We need some testing instructions. I just wrote those, but feel free to edit them if you think they can be improved. You should have made a branch based on master. Shall we rely on integrators cherry-picking it? However, it would be good if you could create such a branch before the integrators get here, and please remember this for future bug fixes. Thanks. Submitting for integration now. To INTEGRATORS: Please cherry pick to master form the 2.5 branch, if necessary.
                  Hide
                  Permalink
                  marina Marina Glancy added a comment -

                  Thanks Colin and Tim, integrated in 2.4, 2.5 and 2.6

                  P.S. There is no caching in get_config() in 2.4 but I think it's still better than errors during restore

                  Show
                  marina Marina Glancy added a comment - Thanks Colin and Tim, integrated in 2.4, 2.5 and 2.6 P.S. There is no caching in get_config() in 2.4 but I think it's still better than errors during restore
                  Hide
                  Permalink
                  marina Marina Glancy added a comment -

                  All good on 2.4, 2.5 and 2.6

                  Show
                  marina Marina Glancy added a comment - All good on 2.4, 2.5 and 2.6
                  Hide
                  Permalink
                  damyon Damyon Wiese added a comment -

                  Here lies 52 bugs.
                  All fixed or swept under a rug.
                  If they come back one day,
                  To our dismay,
                  We all will feel quite un-smug.

                  Thanks for the reporting/fixing/testing on this issue. It has been sent upstream.

                  Show
                  damyon Damyon Wiese added a comment - Here lies 52 bugs. All fixed or swept under a rug. If they come back one day, To our dismay, We all will feel quite un-smug. Thanks for the reporting/fixing/testing on this issue. It has been sent upstream.

                    People

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

                      Dates

                      • Created:
                        Updated:
                        Resolved:
                        Fix Release Date:
                        11/Nov/13