Moodle

AICC Import does not handle Core_Vendor

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.6.3
  • Fix Version/s: 1.8.7, 1.9.3, 2.0
  • Component/s: SCORM
  • Labels:
    None
  • Affected Branches:
    MOODLE_16_STABLE
  • Fixed Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

Description

mod/scorm/locallib.php does not process the Core_Vendor section of an AICC au file. This has been checked in current version of 1.7 and the problem persists. Can be fixed by following change to the scorm_parse_aicc() function:
on the line following:

----------------------------CUT---------------------------
$sco->masteryscore = $element->mastery_score;
----------------------------CUT---------------------------

insert:

----------------------------CUT---------------------------
if (!isset($element->core_vendor)) { $element->core_vendor = ''; }
else { $element->core_vendor = eregi_replace('<cr>', "\r\n", $element->core_vendor); }
$sco->datafromlms = $element->core_vendor;
----------------------------CUT---------------------------

This functionality is required for level 1 AICC compliance.

Activity

Hide
Martin Dougiamas added a comment -

Assigning to Sadiel for prioritising and fixing.

Show
Martin Dougiamas added a comment - Assigning to Sadiel for prioritising and fixing.
Hide
Martin Dougiamas added a comment -

Assigning to Jesús Rincón to organise and start working on.

Show
Martin Dougiamas added a comment - Assigning to Jesús Rincón to organise and start working on.
Hide
Jesús Rincón added a comment -

Could you please send an AICC file so we can reproduce this?

Show
Jesús Rincón added a comment - Could you please send an AICC file so we can reproduce this?
Hide
Keith Starling added a comment -

This is an AICC sample lesson with data in the Core_Vendor section, it requires the Macromedia Authorware plugin to run, which can be downloaded from http://www.adobe.com/shockwave/download/alternates/#ap

Show
Keith Starling added a comment - This is an AICC sample lesson with data in the Core_Vendor section, it requires the Macromedia Authorware plugin to run, which can be downloaded from http://www.adobe.com/shockwave/download/alternates/#ap
Hide
Keith Starling added a comment -

Patch to fix this in Trunk

Show
Keith Starling added a comment - Patch to fix this in Trunk
Hide
Dan Marsden added a comment -

Assigning to Piers - Does this get fixed with the patches from Martin Holden?

Dan

Show
Dan Marsden added a comment - Assigning to Piers - Does this get fixed with the patches from Martin Holden? Dan
Hide
Piers Harding added a comment -

Have applied this patch in HEAD, and attempted to test this using the attached package. This package fails to load correctly hanging on the interaction with http://www.aicc.org/pages/samples.html (which opens if I go to it directly from within browser). Would be great if someone could test this and get back to me on their experiences - thanks.

Piers Harding.

Show
Piers Harding added a comment - Have applied this patch in HEAD, and attempted to test this using the attached package. This package fails to load correctly hanging on the interaction with http://www.aicc.org/pages/samples.html (which opens if I go to it directly from within browser). Would be great if someone could test this and get back to me on their experiences - thanks. Piers Harding.
Hide
Dan Marsden added a comment -

Hi Piers - patch looks good to me, can you please now apply this 1.9Stable and 1.8Stable?

thanks!

Dan

Show
Dan Marsden added a comment - Hi Piers - patch looks good to me, can you please now apply this 1.9Stable and 1.8Stable? thanks! Dan
Hide
Piers Harding added a comment -

Applied patch supplied, and propagated changes to 1.8 and 1.9.

Cheers.

Show
Piers Harding added a comment - Applied patch supplied, and propagated changes to 1.8 and 1.9. Cheers.
Hide
Dan Marsden added a comment -

great! - good work Piers!

1 down....76 more to go!

Dan

Show
Dan Marsden added a comment - great! - good work Piers! 1 down....76 more to go! Dan

People

Vote (2)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: