Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-47779

Review TODO with associated MDL's

    XMLWordPrintable

Details

    • Epic
    • Resolution: Unresolved
    • Minor
    • BACKEND
    • 2.8, 2.9.4, 3.0.2
    • General
    • MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE
    • TODOs in core

    Description

      This issue has children or is linked to all issues in Moodle tracker that have TODO or @todo in Moodle core pointing to them. Having them linked here will both prevent those issues from being closed as "Inactive" and remind developers to search for remaining TODOs when working on them.

      Please note that some issues were already closed without removing the TODOs. Such issues are linked here with link type "causing regression". Separate issues must be created to finally get rid of TODOs in core.

      ------------
      original description:

      A good starting place for this issue:
      Grep the code base for MDL-21120 - a vague issue that has been closed due to inactivity (rightly so).
      Really each use should be reviewed and if not required any more the TODO removed, or more likely a specific MDL issue created and the code updated.

      By my count there are 918 easily found todo's in code and only 89 with associated MDL's.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              samhemelryk Sam Hemelryk
              Votes:
              2 Vote for this issue
              Watchers:
              13 Start watching this issue

              Dates

                Created:
                Updated: