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

          Attachments

            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