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

          Attachments

            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