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

new objectives report for SCORM

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      Use a SCORM package that reports objective data - (contact Dan to get a private package that does this)
      Enter the SCORM package and interact with it to generate the objective related data
      View the SCORM reports page - select Objective report and test.

      Export the data to xls/odt etc and play with the settings at the bottom of the page to display various information.

      Show
      Use a SCORM package that reports objective data - (contact Dan to get a private package that does this) Enter the SCORM package and interact with it to generate the objective related data View the SCORM reports page - select Objective report and test. Export the data to xls/odt etc and play with the settings at the bottom of the page to display various information.
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull Master Branch:
      master_MDL-39926

      Description

      Some SCORM pacakges use objective reporting - we need to create a new SCORM report that allows this information to be displayed and exported

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            danmarsden Dan Marsden added a comment -

            bouncing up for peer review as it would be nice to get someone elses eyes on this as well.

            This code is based on some of the interactions report code - so any issues found with code structure etc may need to be fixed in the interactions report as well. Thanks!

            Show
            danmarsden Dan Marsden added a comment - bouncing up for peer review as it would be nice to get someone elses eyes on this as well. This code is based on some of the interactions report code - so any issues found with code structure etc may need to be fixed in the interactions report as well. Thanks!
            Hide
            danmarsden Dan Marsden added a comment -

            I think this is ready for integration - please bounce it back if I've missed stuff.

            Show
            danmarsden Dan Marsden added a comment - I think this is ready for integration - please bounce it back if I've missed stuff.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            All looks ok but it seems to be missing the new user phonetic fields for the listing.

            So I'm adding a commit on top, applying the same changes done to other reports by:

            https://github.com/moodle/moodle/commit/a327f25ef#L25L272

            FYI, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - All looks ok but it seems to be missing the new user phonetic fields for the listing. So I'm adding a commit on top, applying the same changes done to other reports by: https://github.com/moodle/moodle/commit/a327f25ef#L25L272 FYI, ciao
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment - - edited

            Integrated (master only), thanks!

            For reference, this is the extra commit added:

            http://git.moodle.org/gw?p=integration.git;a=commitdiff;h=4c8bba6686e4ddf2393cfd8d353bc8a3b28de1a7

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - - edited Integrated (master only), thanks! For reference, this is the extra commit added: http://git.moodle.org/gw?p=integration.git;a=commitdiff;h=4c8bba6686e4ddf2393cfd8d353bc8a3b28de1a7
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Everything looks to be working as they should. Status were set correctly based on the tracks, download worked as expected. Various settings worked as far as I tested.
            Passing thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - Everything looks to be working as they should. Status were set correctly based on the tracks, download worked as expected. Various settings worked as far as I tested. Passing thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Just adding an extra commit to let moodle know this is now a core/standard plugin and not a 3rd part add-on:

            http://git.moodle.org/gw?p=integration.git;a=commitdiff;h=07e045b8445d82801ab59921e19882d16b188f21

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Just adding an extra commit to let moodle know this is now a core/standard plugin and not a 3rd part add-on: http://git.moodle.org/gw?p=integration.git;a=commitdiff;h=07e045b8445d82801ab59921e19882d16b188f21 Ciao
            Hide
            danmarsden Dan Marsden added a comment -

            thanks Eloy - I always forget about that one!

            Show
            danmarsden Dan Marsden added a comment - thanks Eloy - I always forget about that one!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            hehe, me2, hence DanP has created MDLSITE-2348.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - hehe, me2, hence DanP has created MDLSITE-2348 .
            Hide
            damyon Damyon Wiese added a comment -

            Moodle has many old functions,
            And although they cause no malfunction,
            There comes a day,
            When they get deprecated away,
            And get and put on the list for expulsion.

            Thanks for all the reports/testing/fixes this week. This issue has been sent upstream.

            Show
            damyon Damyon Wiese added a comment - Moodle has many old functions, And although they cause no malfunction, There comes a day, When they get deprecated away, And get and put on the list for expulsion. Thanks for all the reports/testing/fixes this week. This issue has been sent upstream.
            Hide
            marycooch Mary Cooch added a comment -

            Removing the docs_required label as this improvement is now documented in 2.6, 2.7 and 2.8 https://docs.moodle.org/en/Using_SCORM
            (Note that I haven't added screenshots as I don't have an appropriate SCORM package)
            If you notice that the documentation can be improved, please feel free to log in to the wiki and edit the page. Help in keeping Moodle documentation accurate and up-to-date is much appreciated.

            Show
            marycooch Mary Cooch added a comment - Removing the docs_required label as this improvement is now documented in 2.6, 2.7 and 2.8 https://docs.moodle.org/en/Using_SCORM (Note that I haven't added screenshots as I don't have an appropriate SCORM package) If you notice that the documentation can be improved, please feel free to log in to the wiki and edit the page. Help in keeping Moodle documentation accurate and up-to-date is much appreciated.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  18/Nov/13