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

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

    Details

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

      Description

      Split off from MDL-28127. See screenshot there.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            samhemelryk 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
            samhemelryk 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
            timhunt 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
            timhunt 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
            skodak Petr Skoda added a comment -

            Integrated, thank you both.

            Show
            skodak Petr Skoda added a comment - Integrated, thank you both.
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Works Great...
            Thanks for providing the patch Tim

            Show
            rajeshtaneja 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:
                  Fix Release Date:
                  10/Oct/11