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

SCORM report - basic plugin needs to be added to standard core list

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2
    • Component/s: SCORM
    • Labels:

      Description

      standard_plugins_list in pluginlib.php

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            skodak Petr Skoda added a comment -

            implemented including the necessary lang strings for Reports

            Show
            skodak Petr Skoda added a comment - implemented including the necessary lang strings for Reports
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Hi Petr, looks like you forgot to push this one to github.
            Stopping review to its up there

            Show
            samhemelryk Sam Hemelryk added a comment - Hi Petr, looks like you forgot to push this one to github. Stopping review to its up there
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks for fixing that up Petr, this has been integrated now

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks for fixing that up Petr, this has been integrated now Cheers Sam
            Hide
            andyjdavis Andrew Davis added a comment - - edited

            I can see the following in my plugin information. Is that what I should see? I'll PM Petr to get the testing instructions clarified.

            SCORM package / Reports
            Basic Report scormreport_basic Standard 2011071100

            Show
            andyjdavis Andrew Davis added a comment - - edited I can see the following in my plugin information. Is that what I should see? I'll PM Petr to get the testing instructions clarified. SCORM package / Reports Basic Report scormreport_basic Standard 2011071100
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Yip thats correct-o-mondo!

            Show
            samhemelryk Sam Hemelryk added a comment - Yip thats correct-o-mondo!
            Hide
            andyjdavis Andrew Davis added a comment -

            ok. passing. Please make testing instructions more explicit in future

            Show
            andyjdavis Andrew Davis added a comment - ok. passing. Please make testing instructions more explicit in future
            Hide
            skodak Petr Skoda added a comment -

            Thanks everybody, this is now part of the weekly build.

            Show
            skodak Petr Skoda added a comment - Thanks everybody, this is now part of the weekly build.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  5/Dec/11