Details
Description
Every time a SCORM package is updated a new set of 'scorm_scoes' records are created and the old ones are deleted (after fixing up the 'scorm_scoes_tracking' table to point at the new records). This means that any links to specific pages within the SCORM, in the form: [siteurl]/mod/scorm/player.php?a=[XXX]¤torg=[YYY]&scoid=[ZZZ] will break.
From a brief look through the code, I cannot see a reason why 'update_record' cannot be used instead of 'insert_record' in cases where the SCO identifier already exists - this would not only fix the linking issue, but also remove the need to fix-up the 'scorm_scoes_tracking' records. I note that the importing of AICC content already appears to work this way.