Moodle
  1. Moodle
  2. MDL-27732

Glossary module does not include introformat field in MBZ backup

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.3
    • Fix Version/s: 2.0.4
    • Component/s: Glossary
    • Labels:
    • Testing Instructions:
      Hide
      • In some Moodle 2.0. disable the HTML editor in your profile
      • Create several glossaries with the introduction field in various text formats (Moodle format, HTML, Markdown)
      • Backup and restore these glossaries
      • Edit the glossaries settings
      • TEST: make sure that the field format information is restored correctly
      Show
      In some Moodle 2.0. disable the HTML editor in your profile Create several glossaries with the introduction field in various text formats (Moodle format, HTML, Markdown) Backup and restore these glossaries Edit the glossaries settings TEST: make sure that the field format information is restored correctly
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-27732-glossary-introformat
    • Rank:
      17409

      Description

      Looking at backup_glossary_activity_structure_step::define_structure() it is obvious that the Glossary module does not include the introformat field in the backup file.

        Issue Links

          Activity

          Hide
          David Mudrak added a comment -

          Triaging for STABLE backlog as a critical issue (data loss risk)

          Show
          David Mudrak added a comment - Triaging for STABLE backlog as a critical issue (data loss risk)
          Hide
          David Mudrak added a comment -

          Linking MDL-27729 - note that the converted Moodle 1.9 backup files contain this field.

          Show
          David Mudrak added a comment - Linking MDL-27729 - note that the converted Moodle 1.9 backup files contain this field.
          Hide
          Sam Hemelryk added a comment -

          Looks good thanks David.
          Once I've checked with Eloy that it is OK for things to be sent upstream this can go in.

          Show
          Sam Hemelryk added a comment - Looks good thanks David. Once I've checked with Eloy that it is OK for things to be sent upstream this can go in.
          Hide
          Sam Hemelryk added a comment -

          Thanks David this has been integrated now.

          Show
          Sam Hemelryk added a comment - Thanks David this has been integrated now.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Passing, well spotted David! My fault!

          Show
          Eloy Lafuente (stronk7) added a comment - Passing, well spotted David! My fault!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This is now upstream, yay! Many thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - This is now upstream, yay! Many thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: