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

          Attachments

            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