Moodle
  1. Moodle
  2. MDL-38386

Change upgrade step for 24 and master for MDL-38173

    Details

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

      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.

        Issue Links

          Activity

          Hide
          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
          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
          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
          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
          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
          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 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 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: