thanks for the report - assigning to David, our Language Guru.

Does this affect any SCORM package played in French? This definitely is JavaScript issue (as it does not escape characters), not the language issue. I can look at it though, of course.

whoops - assigning back to me - for some reason I thought this was an invalid lang issue!

Fouzi - any chance you could be more specific? - which lang string causes the issue?

thanks,

Hi,
Here's an example of error that I got after enabling SCORM debuging:
----------------------------------------------------------------------
Erreur : missing ; before statement
Fichier Source : /mod/scorm/api.php?a=14&scoid=932&currentorg=xp_man0_toc1&attempt=1
Ligne : 636, Colonne : 36
Code Source :
logButton.innerHTML = '-->L'historique API est désactivée';
-----------------------------------------------------------------------
As You may note, It is possible to correct that like this:
logButton.innerHTML = "-->L'historique API est désactivée";
or
logButton.innerHTML = '-->L\'historique API est désactivée';
I think the correction should be done on French translation strings.
Note: It may be not the unique non well formed french string. And maybe such error can occur with other languages.
Regards

here's a untested patch with a fix.

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

rebased

Thanks Dan another spot on patch that has been integrated

Works as expected!
Thanks

Some changes to Moodle should be milestones in the project by themselves.

This is not the case and your fix is not so important, but your collaboration is highly appreciated, thanks!

Closing as fixed, ciao