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

Change upgrade step for 24 and master for MDL-38173

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.4.2, 2.5
    • Fix Version/s: 2.4.3, STABLE backlog
    • Component/s: Course
    • Labels:

      Description

      The upgrade step for MDL-38173 should always be run on master and 24 branches - there is a chance that it would be missed if someone upgraded from 2012062504.08 to an old version of 2.4, then to the latest version of 24 (same for 25).

      An extra patch was added to address this - but missed integration.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            nebgor Aparup Banerjee added a comment -

            so once more (harmless) for 2.4.2 guys : "sections where the sequence is not in sync with the course_modules table" being fixed.

            integrated into 24 and master. thanks! <lol/>

            Show
            nebgor Aparup Banerjee added a comment - so once more (harmless) for 2.4.2 guys : "sections where the sequence is not in sync with the course_modules table" being fixed. integrated into 24 and master. thanks! <lol/>
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success!

            Tested in 2.4 and master.

            There were some unrelated errors that appeared during the upgrade of master.

            Show
            salvetore Michael de Raadt added a comment - Test result: Success! Tested in 2.4 and master. There were some unrelated errors that appeared during the upgrade of master.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

            Thanks!

            PS: Yay, legacy template messages. Yes, you're ok, we don't have CVS anymore!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks! PS: Yay, legacy template messages. Yes, you're ok, we don't have CVS anymore!
            Hide
            marina Marina Glancy added a comment -

            This was not integrated (probably by mistake) in 2.5 and therefore is not present in 2.5 and 2.6 now. This will be dealt with in MDL-38228

            Show
            marina Marina Glancy added a comment - This was not integrated (probably by mistake) in 2.5 and therefore is not present in 2.5 and 2.6 now. This will be dealt with in MDL-38228

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  18/Mar/13