Moodle

ADL Scorm 2004 LMS Conformance Test

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9
  • Fix Version/s: 1.8.5, 1.9.1, 2.0
  • Component/s: SCORM
  • Labels:
    None
  • Environment:
    Windows xp sp2, ADL Test Suite Scorm 2004 3rd edition v1.0.2
  • Database:
    MySQL
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

Description

I tried to tried to evaluate moodle 1.9 for Scorm 2004 with the ADL test suite and the lms test packages
In test package DMI are generated many errors in SCOs 3 and 4
and in package DMB i cant get to the second SCO of the package
I have attached the detailed log with the errors
And searching in moodle forums and in general i dont understand if moodle is Scorm 2004 conformant?

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jennifer Brooks added a comment -

Moodle is not yet SCORM 2004 conformant. I have also found that the second SCO in an aggregation will not launch; neither will the 4th if there are 5 SCOs and the 6th if there are 7 SCOs. I am currently fixing this bug. There are other issues that will also need to be fixed before Moodle will be SCORM 2004 conformant.

Show
Jennifer Brooks added a comment - Moodle is not yet SCORM 2004 conformant. I have also found that the second SCO in an aggregation will not launch; neither will the 4th if there are 5 SCOs and the 6th if there are 7 SCOs. I am currently fixing this bug. There are other issues that will also need to be fixed before Moodle will be SCORM 2004 conformant.
Hide
Permalink
Jennifer Brooks added a comment -

Clarification: This bug is limited to the use of the "continue" navigation request. That is, it is still possible to launch the SCOs by clicking on them in the TOC when "choice" is enabled.

Show
Jennifer Brooks added a comment - Clarification: This bug is limited to the use of the "continue" navigation request. That is, it is still possible to launch the SCOs by clicking on them in the TOC when "choice" is enabled.
Hide
Permalink
Jennifer Brooks added a comment -

Ana Cláudia (in MDL-13887) reported:
When I use version 5.2.5 of the PHP, to sail in the Scorm for the buttons prev/next, the pages jumps of 2 in 2 pages, or either has access for example page 1, later the 3 and the 5. E when I install version 5.2.0 of the PHP, the navigation function perfectly.

Show
Jennifer Brooks added a comment - Ana Cláudia (in MDL-13887 ) reported: When I use version 5.2.5 of the PHP, to sail in the Scorm for the buttons prev/next, the pages jumps of 2 in 2 pages, or either has access for example page 1, later the 3 and the 5. E when I install version 5.2.0 of the PHP, the navigation function perfectly.
Hide
Permalink
Dan Marsden added a comment -

Hi jennifer - that's related to MDL-13093 - Petr put a fix together that seems to fix it - I'm waiting to hear from Kimberly Prewitt to make sure it works before I commit the patch to 1.9stable/HEAD

Dan

Show
Dan Marsden added a comment - Hi jennifer - that's related to MDL-13093 - Petr put a fix together that seems to fix it - I'm waiting to hear from Kimberly Prewitt to make sure it works before I commit the patch to 1.9stable/HEAD Dan
Hide
Permalink
Jennifer Brooks added a comment -

Funny, I fixed it using code very similar to Kristof's. If Kimberly is still having trouble with Petr's patch, we can try adding the following lines in SCORM_13lib.php:

$nextsco = current($scoes);
if($nextsco->id == $sco->id)

{ $nextsco = next($scoes); }

This should replace the one line: $nextsco = next($scoes);

Although very similar, I think it's different enough from Kristof's fix to possibly work for Kimberly.

Show
Jennifer Brooks added a comment - Funny, I fixed it using code very similar to Kristof's. If Kimberly is still having trouble with Petr's patch, we can try adding the following lines in SCORM_13lib.php: $nextsco = current($scoes); if($nextsco->id == $sco->id) { $nextsco = next($scoes); } This should replace the one line: $nextsco = next($scoes); Although very similar, I think it's different enough from Kristof's fix to possibly work for Kimberly.
Hide
Permalink
Dan Marsden added a comment -

hah! - classic! - welcome to the scorm module!

haven't tested your fix yet, but personally I like Petr's fix better as it fixes the bad logic used in the foreach, whereas yours potentially "work's around" the bad logic...

fyi that foreach logic appears in lib.php, scorm12lib.php, scorm13lib.php and the aicc lib. (which petr's patch fixes)

Dan

Show
Dan Marsden added a comment - hah! - classic! - welcome to the scorm module! haven't tested your fix yet, but personally I like Petr's fix better as it fixes the bad logic used in the foreach, whereas yours potentially "work's around" the bad logic... fyi that foreach logic appears in lib.php, scorm12lib.php, scorm13lib.php and the aicc lib. (which petr's patch fixes) Dan
Hide
Permalink
Mike Churchward added a comment -

I just committed a change to 'datamodle.php' that corrected a mistyped function call. In our tests, this was causing navigation errors. Fixing the function call has fixed our navigation issues.

Show
Mike Churchward added a comment - I just committed a change to 'datamodle.php' that corrected a mistyped function call. In our tests, this was causing navigation errors. Fixing the function call has fixed our navigation issues.
Hide
Permalink
Petr Škoda added a comment -

thanks

Show
Petr Škoda added a comment - thanks
Hide
Permalink
Petr Škoda added a comment -

Hello Mike, please tag your commits as merged next time

Show
Petr Škoda added a comment - Hello Mike, please tag your commits as merged next time

People

Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: