Issue Details (XML | Word | Printable)

Key: MDL-17294
Type: Sub-task Sub-task
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Tim Hunt
Reporter: Tim Hunt
Votes: 0
Watchers: 1
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle
MDL-17284

Review and eliminate all TODOs

Created: 19/Nov/08 03:41 PM   Updated: 05/Jan/09 02:28 PM
Return to search
Component/s: Quiz
Affects Version/s: 2.0
Fix Version/s: 2.0

Participants: Olli Savolainen and Tim Hunt
Security Level: None
Resolved date: 05/Jan/09
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
MDL-17293 comments on a couple of them.

the one in mod/quiz/tabs.php - //TODO: ensure that removing get_query_string here won't hurt
It won't hurt anything.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Olli Savolainen committed 1 file to 'Moodle CVS' - 26/Nov/08 10:16 PM
quiz editing: MDL-17294 added back (removed comments around) random question preview link
MODIFY mod/quiz/editlib.php   Rev. 1.75    (+3 -4 lines)
Olli Savolainen committed 1 file to 'Moodle CVS' - 29/Nov/08 01:47 AM
quiz editing: MDL-17294 fixed a TODO, added the url to usability testing results
MODIFY theme/standard/styles_color.css   Rev. 1.194    (+1 -1 lines)
Olli Savolainen committed 3 files to 'Moodle CVS' - 29/Nov/08 02:29 AM
quiz editing: MDL-17294 changed quiz_process_randomquestion_formdata to always return (instead of sometimes redirecting)
MODIFY mod/quiz/editlib.php   Rev. 1.82    (+10 -18 lines)
MODIFY mod/quiz/addrandom.php   Rev. 1.3    (+6 -3 lines)
MODIFY mod/quiz/edit.php   Rev. 1.131    (+10 -2 lines)
Olli Savolainen committed 2 files to 'Moodle CVS' - 29/Nov/08 03:37 AM
quiz editing: MDL-17294 removed the $showbreaks parameter from quiz_print_question_list
MODIFY mod/quiz/edit.php   Rev. 1.132    (+1 -2 lines)
MODIFY mod/quiz/editlib.php   Rev. 1.83    (+2 -3 lines)
Olli Savolainen committed 1 file to 'Moodle CVS' - 29/Nov/08 03:47 AM
quiz editing: MDL-17294 added a call to quiz_question_tostring()
MODIFY mod/quiz/editlib.php   Rev. 1.84    (+2 -5 lines)
Olli Savolainen committed 1 file to 'Moodle CVS' - 29/Nov/08 03:57 AM
quiz editing: MDL-17294 removed unnecessary comments
MODIFY mod/quiz/editlib.php   Rev. 1.85    (+1 -7 lines)
Olli Savolainen committed 1 file to 'Moodle CVS' - 29/Nov/08 03:58 AM
quiz editing: MDL-17294 removed unnecessary comment
MODIFY mod/quiz/edit.php   Rev. 1.133    (+1 -5 lines)
Olli Savolainen committed 1 file to 'Moodle CVS' - 29/Nov/08 04:43 AM
quiz editing: MDL-17294 improved the error situation handling in process_randomquestion_formdata
MODIFY mod/quiz/editlib.php   Rev. 1.86    (+5 -5 lines)
Olli Savolainen committed 1 file to 'Moodle CVS' - 29/Nov/08 04:51 AM
quiz editing: MDL-17294 removed unnecessary comment
MODIFY mod/quiz/addrandom.php   Rev. 1.4    (+1 -6 lines)
tjhunt committed 1 file to 'Moodle CVS' - 05/Jan/09 02:26 PM
MDL-17294 Eliminate some commented out code, and clean up the 'no questions in quiz' special case.
MODIFY mod/quiz/editlib.php   Rev. 1.97    (+7 -40 lines)