Moodle

Clean up todos

Details

  • Type: Task Task
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 2.0
  • Fix Version/s: 2.0.8
  • Component/s: Other
  • Labels:
    None
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

We just decided all TODOs in the code should be linked to a tracker issue. http://docs.moodle.org/en/Development:Coding_style#TODO_comments_must_link_to_a_tracker_issue.

Of course, most of the ones we have at the moment don't.

I made a little script .../lib/simpletest/todochecker.php which lists all the todos in the code, together with whether they link to a tracker issue as per the guideline.

Before the Moodle 2.0 release, we should use that to try to fix as many todos as possible, and ensure that the rest have corresponding tracker issues filed.

Activity

There are no comments yet on this issue.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: