Moodle
  1. Moodle
  2. MDL-28507

Bad string in the upgrade of very old quiz attempts to 2.1

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.1
        • Fix Version/s: 2.1.2, 2.2
        • Component/s: Quiz
        • Labels:
          None
        • Rank:
          18166

          Description

          Split off from MDL-28127. See screenshot there.

            Issue Links

            has a non-specific relationship to

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-28127 Bad string in quiz upgrade from 2.0 to 2.1

            • Minor - Minor loss of function where workaround is possible
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Sam Hemelryk added a comment -

              Hi Tim,
              A quick glance and I've notice $a->todo is still being referenced.
              Depending upon Petr's super man abilities I don't think we will get through this sprint so I've left this issue included in the integration run so that if you get time to fix it it's automatically back in.

              Cheers
              Sam

              Show
              Sam Hemelryk added a comment - Hi Tim, A quick glance and I've notice $a->todo is still being referenced. Depending upon Petr's super man abilities I don't think we will get through this sprint so I've left this issue included in the integration run so that if you get time to fix it it's automatically back in. Cheers Sam
              Hide
              Permalink
              Tim Hunt added a comment -

              Doh! I have now done a proper global search for ->todo and fixed all occurences. Commits on both branches amended.

              Show
              Tim Hunt added a comment - Doh! I have now done a proper global search for ->todo and fixed all occurences. Commits on both branches amended.
              Hide
              Permalink
              Petr Škoda added a comment -

              Integrated, thank you both.

              Show
              Petr Škoda added a comment - Integrated, thank you both.
              Hide
              Permalink
              Rajesh Taneja added a comment -

              Works Great...
              Thanks for providing the patch Tim

              Show
              Rajesh Taneja added a comment - Works Great... Thanks for providing the patch Tim

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: