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

Add more details for third party developers

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: 3.3
    • Component/s: Accessibility, Blocks
    • Labels:
    • Testing Instructions:
      Hide
      1. Check the PHPDocs for mod_assign_core_calendar_provide_event_action in mod/assign/lib.php and confirm they make sense.
      2. Check mod/upgrade.txt and ensure the mention of the deprecation of *_print_overview functions exists.
      3. Check mod/upgrade.txt and ensure that the 3 callbacks introduced by -MDL-55611- are mentioned and a link to the docs exists.
      Show
      Check the PHPDocs for mod_assign_core_calendar_provide_event_action in mod/assign/lib.php and confirm they make sense. Check mod/upgrade.txt and ensure the mention of the deprecation of *_print_overview functions exists. Check mod/upgrade.txt and ensure that the 3 callbacks introduced by - MDL-55611 - are mentioned and a link to the docs exists.
    • Affected Branches:
      MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_33_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-58501_master

      Description

      We should add some details to mod/upgrade.txt about the new callbacks as well as updating the PHPDocs for these functions so they are more descriptive. Eg. 'Handles creating actions for events.' is very vague.

      It should also be listed in the docs at https://docs.moodle.org/dev/.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              markn Mark Nelson
              Reporter:
              markn Mark Nelson
              Peer reviewer:
              cameron1729
              Integrator:
              Andrew Nicols
              Tester:
              John Okely
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, 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:
                15/May/17