Moodle
  1. Moodle
  2. MDL-33702

SCORM 2004 objectives report status field always empty

    Details

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

      Create a SCORM package using the attached zip
      enter the scorm as a student and answer some questions.
      Check the interactions report to see if it displays a valid status for the entries in the objectives table.

      Show
      Create a SCORM package using the attached zip enter the scorm as a student and answer some questions. Check the interactions report to see if it displays a valid status for the entries in the objectives table.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      MDL-33702-master
    • Rank:
      41708

      Description

      In SCORM 1.2 the status of an objective comes from cmi.objectives.n.status. In SCORM 2004 this was split into cmi.objectives.n.success_status and cmi.objectives.n.completion_status, with cmi.objectives.n.progress_measure a useful normalised value.

      The /mod/scorm/userreport.php page currently creates an HTML table for objectives with columns 'identifier', 'status', 'raw', 'min' and 'max'. It seems strange for the raw value to come before the min and max bounds, rather than in between.

      The status column is populated only from the SCORM 1.2 data model location. There needs to be added consideration for SCORM 2004 so that something suitable gets displayed in this column. Perhaps the progress_measure as a percentage could be used until success_status is known? e.g. "Unattempted" > "25% done" > "Passed/Failed", with fallback to completion_status if those properties are undefined.

        Activity

        Hide
        Dan Marsden added a comment -

        thanks Peter - that makes sense, adding Ankit here in the off-chance that he wants to modify the SCORM reports to do this otherwise I'll get to this at some point... patches are always welcome

        Show
        Dan Marsden added a comment - thanks Peter - that makes sense, adding Ankit here in the off-chance that he wants to modify the SCORM reports to do this otherwise I'll get to this at some point... patches are always welcome
        Hide
        Dan Marsden added a comment -

        Peter - if you were able to attach a SCORM package (or link to one) that set those vars it would help a lot for testing and to develop the patch

        Show
        Dan Marsden added a comment - Peter - if you were able to attach a SCORM package (or link to one) that set those vars it would help a lot for testing and to develop the patch
        Hide
        Ankit Agarwal added a comment -

        Will try to have a look on this during next sprint. as Dan said, a sample scorm package will be appreciated.
        Thanks

        Show
        Ankit Agarwal added a comment - Will try to have a look on this during next sprint. as Dan said, a sample scorm package will be appreciated. Thanks
        Hide
        Peter Chamberlin added a comment -

        The “Advanced Run-Time Calls” SCORM 2004 3rd Edition example package at http://scorm.com/scorm-explained/technical-scorm/golf-examples/ can be used to test the data model properties for objective status.

        Show
        Peter Chamberlin added a comment - The “Advanced Run-Time Calls” SCORM 2004 3rd Edition example package at http://scorm.com/scorm-explained/technical-scorm/golf-examples/ can be used to test the data model properties for objective status.
        Hide
        Ankit Agarwal added a comment -

        Estimated development time:- 12-14 hours

        Show
        Ankit Agarwal added a comment - Estimated development time:- 12-14 hours
        Hide
        Ankit Agarwal added a comment -

        Hi Dan,
        I have added some new features and done some refactoring. Still need to find a good package to test it properly. It will great to get your feedback on this.

        Also I noticed we are using interactions_n.id in interactions report, which should probably be interactions.n.id ?

        @ Peter
        I couldn't get the rustic pack working with Moodle.

        Thanks

        Show
        Ankit Agarwal added a comment - Hi Dan, I have added some new features and done some refactoring. Still need to find a good package to test it properly. It will great to get your feedback on this. Also I noticed we are using interactions_n.id in interactions report, which should probably be interactions.n.id ? @ Peter I couldn't get the rustic pack working with Moodle. Thanks
        Hide
        Peter Chamberlin added a comment -

        If you cannot get the Rustici pack to function then there are likely other issues in Moodle that need fixing to support SCORM 2004 to the minimum expected level.

        I have created a demonstration package for you to test with (see attached file).

        Show
        Peter Chamberlin added a comment - If you cannot get the Rustici pack to function then there are likely other issues in Moodle that need fixing to support SCORM 2004 to the minimum expected level. I have created a demonstration package for you to test with (see attached file).
        Hide
        Peter Chamberlin added a comment -

        SCORM 2004 3rd Edition cross-domain package

        Objectives correspond to assessment marks/scores/progress.

        Show
        Peter Chamberlin added a comment - SCORM 2004 3rd Edition cross-domain package Objectives correspond to assessment marks/scores/progress.
        Hide
        Ankit Agarwal added a comment -

        Thanks Peter for the packs, that will help a lot with testing. And yes we do not officially support 2004 SCORM yet.

        Show
        Ankit Agarwal added a comment - Thanks Peter for the packs, that will help a lot with testing. And yes we do not officially support 2004 SCORM yet.
        Hide
        Dan Marsden added a comment -

        looks fine to me - NOTE TO INTEGRATOR - master only - not stable branches please.

        Peter - would be good if you could test this patch if you get a chance and let us know if it displays the information you would like.

        Show
        Dan Marsden added a comment - looks fine to me - NOTE TO INTEGRATOR - master only - not stable branches please. Peter - would be good if you could test this patch if you get a chance and let us know if it displays the information you would like.
        Hide
        Dan Poltawski added a comment -

        Taking integration_held issues out of current integration (whilst we are keeping master and 23_STABLE in sync).

        Show
        Dan Poltawski added a comment - Taking integration_held issues out of current integration (whilst we are keeping master and 23_STABLE in sync).
        Hide
        Peter Chamberlin added a comment -

        Dan - The patched files appear to display the correct results for my SCORM 1.2 and SCORM 2004 packages.

        Show
        Peter Chamberlin added a comment - Dan - The patched files appear to display the correct results for my SCORM 1.2 and SCORM 2004 packages.
        Hide
        Ankit Agarwal added a comment -

        Thanks for the testing Peter.
        Hopefully this will get integrated next week.

        Thanks

        Show
        Ankit Agarwal added a comment - Thanks for the testing Peter. Hopefully this will get integrated next week. Thanks
        Hide
        Ankit Agarwal added a comment -

        Rebased.
        Thanks

        Show
        Ankit Agarwal added a comment - Rebased. Thanks
        Hide
        Sam Hemelryk added a comment -

        Thanks Ankit, this has been integrated now

        Show
        Sam Hemelryk added a comment - Thanks Ankit, this has been integrated now
        Hide
        Rajesh Taneja added a comment -

        Works great,

        Thanks for fixing this Ankit.

        Show
        Rajesh Taneja added a comment - Works great, Thanks for fixing this Ankit.
        Hide
        Dan Poltawski added a comment -

        Congratulations!

        You've made it into the weekly release!

        Thanks for your contribution - here are some random drummers to keep you inspired for the next week!
        http://www.youtube.com/watch?v=_QhpHUmVCmY

        Show
        Dan Poltawski added a comment - Congratulations! You've made it into the weekly release! Thanks for your contribution - here are some random drummers to keep you inspired for the next week! http://www.youtube.com/watch?v=_QhpHUmVCmY

          People

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

            Dates

            • Created:
              Updated:
              Resolved: