Skip to content

Update scorm_api.php async: true, for Scorm API #6312

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: 1.11.x
Choose a base branch
from

Conversation

LudiscapeXApi
Copy link
Contributor

async: false provoque des erreurs ou des plantages. sur certains serveurs. J'ai déplacer le code dans la partie success de l'appel ajax afin de fluidifier le chargement des autres libs js.

async: false provoque des erreurs ou des plantages. sur certains serveurs. J'ai déplacer le code dans la partie success de l'appel ajax afin de fluidifier le chargement des autres libs js.
@ywarnier
Copy link
Member

Alors le async:false a été mis là il y a plus de 12 ans, donc probablement pas tout à fait au goût du jour. Par contre, il faut faire très attention avec l'async:true, parce que du coup les différents messages SCORM peuvent arriver dans le désordre et ça peut avoir un impact sur la cohérence des données (cas vécu, peut être pas avec cet async en particulier, mais dans SCORM en tout cas).

@ywarnier
Copy link
Member

Pour le reste, je suis intéressé par ton feedback. Modifier le code lié à SCORM apporte systématiquement des problèmes.
Par ailleurs, le déplacement du code dans "success" est un légèrement excessif, je pense : Si pas "success", il faudrait au moins logger quelque chose pour le dire, et le code lié au glossaire n'a pas grand chose à voir avec le succès du LMSInitialize (mais il est légitime de discuter de son intérêt si le SCO ne se charge pas)

@ywarnier
Copy link
Member

Je ne l'inclus pas dans 1.11.30 (je suis très frileux par rapport à ça) mais je le garde ici ouvert pour faire des tests à l'occasion quand c'est trop lent

@LudiscapeXApi
Copy link
Contributor Author

Oui, je comprends. J’ai cependant un doute sur le fait d’avoir couvert tous les cas en production. Par ailleurs, l’utilisation de [async: false] pourrait ne plus être prise en charge à l’avenir, notamment avec Chromium qui tend à devenir la référence. Bloquer le chargement d’une page en attendant la résolution d’une promesse peut poser des problèmes de performance, en particulier sur des serveurs peu puissants.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants