Moodle

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

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor 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.

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 - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

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

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: