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

                  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
                    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
                    Permalink
                    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
                    Permalink
                    skodak Petr Skoda added a comment -

                    Integrated, thank you both.

                    Show
                    skodak Petr Skoda added a comment - Integrated, thank you both.
                    Hide
                    Permalink
                    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