Uploaded image for project: '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
    • Status: Closed
    • Priority: 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
          danmarsden Dan Marsden added a comment -

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

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

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

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

          Looks good - go and integrate!

          Show
          danmarsden Dan Marsden added a comment - Looks good - go and integrate!
          Hide
          stronk7 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
          stronk7 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_frenz Ankit Agarwal added a comment -

          rebased!

          Note to integrators: Master only!

          Thanks

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

          Thanks Ankit + Dan, this has been integrated now

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

          No testing required this is purely documentation change

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

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

          Ciao

          Show
          stronk7 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:
                Fix Release Date:
                5/Dec/11