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

      Description

      function scorm_report_list in reportlib.php needs to be documented

        Gliffy Diagrams

          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: