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

Make sure all functions in lib/deprecatedlib.php have debugging message

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0
    • Fix Version/s: 3.2
    • Component/s: General
    • Labels:
    • Testing Instructions:
      Hide
      1. Check lib/deprecatedlib.php.
        • Confirm that all methods have @deprecated tags.
        • Confirm that debugging() is called for all functions except for the following:
          • Functions that have been finally deprecated (throws).
          • print_textarea()
          • unzip_file()
          • zip_files()
          • events_* functions. (Some call debugging, some do not. It's fine.)
          • Functions that have comments saying not to add debugging
      Show
      Check lib/deprecatedlib.php. Confirm that all methods have @deprecated tags. Confirm that debugging() is called for all functions except for the following: Functions that have been finally deprecated (throws). print_textarea() unzip_file() zip_files() events_* functions. (Some call debugging, some do not. It's fine.) Functions that have comments saying not to add debugging
    • Affected Branches:
      MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Epic Link:
    • Pull from Repository:
    • Pull Master Branch:
      MDL-50051-master
    • Pull Master Diff URL:
    • Story Points:
      2.67
    • Sprint:
      3.2 Sprint 3

      Description

      Go through all functions in lib/deprecatedlib.php and make sure they all:

      • display developer debugging message
      • have @deprecated since X.X phpdocs token

      If some functions do not do any of that, add @deprecated since 3.0 and debugging message.

      AFAIK some of functions are still used in Moodle. Make sure to grep !

        Attachments

          Activity

            People

            Assignee:
            jpataleta Jun Pataleta
            Reporter:
            marina Marina Glancy
            Peer reviewer:
            Jake Dallimore
            Integrator:
            Dan Poltawski
            Tester:
            John Okely
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              5/Dec/16