EVOgeek/Vmeda.Online
1
0
Fork 1
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #3768 from alfonso-salces/MOBILE-4191

Browse Source 
MOBILE-4191 scorm: Make options field optional
This commit is contained in:
Dani Palou 2023-08-02 08:20:23 +02:00 committed by GitHub
commit 284111865f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/addons/mod/scorm/services/scorm.ts
View File

@ -1124,10 +1124,12 @@ export class AddonModScormProvider {
currentScorm.warningMessage = warning?.message;
}
const scormOptions = CoreUtils.objectToKeyValueMap(response.options, 'name', 'value');
if (response.options) {
const scormOptions = CoreUtils.objectToKeyValueMap(response.options, 'name', 'value');
if (scormOptions.scormstandard) {
currentScorm.scormStandard = Number(scormOptions.scormstandard);
if (scormOptions.scormstandard) {
currentScorm.scormStandard = Number(scormOptions.scormstandard);
}
}
currentScorm.moduleurl = options.moduleUrl;
@ -1871,7 +1873,7 @@ export type AddonModScormGetScormsByCoursesWSParams = {
* Data returned by mod_scorm_get_scorms_by_courses WS.
*/
export type AddonModScormGetScormsByCoursesWSResponse = {
options: AddonModScormOptions[];
options?: AddonModScormOptions[]; // @since v4.3. Scorm options
scorms: AddonModScormScormWSData[];
warnings?: CoreWSExternalWarning[];
};