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

Badges upgrade step should not require lib/badgeslib.php

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.7, 3.8
    • 3.7.3
    • Badges
    • MOODLE_37_STABLE, MOODLE_38_STABLE
    • MOODLE_37_STABLE
    • MDL-65515-master
    • Hide

      Verifying upgrade.

      1. Install MOODLE_36_STABLE
      2. Upgrade to MOODLE_37_STABLE

        git checkout MOODLE_37_STABLE && php admin/cli/upgrade.php

      3. Verify there are no errors generated during upgrade.
      4. Repeat steps 2 & 3, bust upgrading to master.

      Verifying install.

      1. Install MOODLE_37_STABLE
      2. Verify there are no errors generated during install.
      3. Repeat steps 1 & 2, bust installing master.
      Show
      Verifying upgrade. Install MOODLE_36_STABLE Upgrade to MOODLE_37_STABLE git checkout MOODLE_37_STABLE && php admin/cli/upgrade.php Verify there are no errors generated during upgrade. Repeat steps 2 & 3, bust upgrading to master. Verifying install. Install MOODLE_37_STABLE Verify there are no errors generated during install. Repeat steps 1 & 2, bust installing master.

    Description

      The one function that is required for new installs and upgrades is badges_install_default_backpacks() which creates the backpacks:

      This function does not do any custom logic, it just makes DB queries to insert the backpacks using constants. Those constants and the function should be split into a separate location so they can be called during install/upgrade without loading the rest of the badges functions / constants.

      Attachments

        Issue Links

          Activity

            People

              damyon Damyon Wiese
              damyon Damyon Wiese
              Mihail Geshoski Mihail Geshoski
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Gladys Basiana Gladys Basiana
              Yuliya Bozhko, Amaia Anabitarte, Carlos Escobedo, Laurent David, Mikel Martín Corrales, Sabina Abellan, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/Nov/19

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 3 hours, 30 minutes
                  3h 30m