Moodle
  1. Moodle
  2. MDL-27256 Improve SCORM quiz reporting
  3. MDL-30044

Add documentation to the function scorm_report_list in reportlib.php

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2
    • Component/s: SCORM
    • Labels:
      None
    • Rank:
      24452

      Description

      function scorm_report_list in reportlib.php needs to be documented

        Activity

        Hide
        Dan Marsden added a comment -

        "avaialable" -> "available"
        "sub packages" -> "sub-plugins"

        Show
        Dan Marsden added a comment - "avaialable" -> "available" "sub packages" -> "sub-plugins"
        Hide
        Ankit Agarwal added a comment - - edited

        LOL...damn I always commit typos! grrr!
        anyways updated patch!

        Show
        Ankit Agarwal added a comment - - edited LOL...damn I always commit typos! grrr! anyways updated patch!
        Hide
        Dan Marsden added a comment -

        Looks good - go and integrate!

        Show
        Dan Marsden added a comment - Looks good - go and integrate!
        Hide
        Eloy Lafuente (stronk7) added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Ankit Agarwal added a comment -

        rebased!

        Note to integrators: Master only!

        Thanks

        Show
        Ankit Agarwal added a comment - rebased! Note to integrators: Master only! Thanks
        Hide
        Sam Hemelryk added a comment -

        Thanks Ankit + Dan, this has been integrated now

        Show
        Sam Hemelryk added a comment - Thanks Ankit + Dan, this has been integrated now
        Hide
        Sam Hemelryk added a comment -

        No testing required this is purely documentation change

        Show
        Sam Hemelryk added a comment - No testing required this is purely documentation change
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Yes, you got this finally upstream, just in time for Moodle 2.2beta. Congrats and thanks!

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Yes, you got this finally upstream, just in time for Moodle 2.2beta. Congrats and thanks! Ciao

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: