diff --git a/src/core/courses/pages/my-courses/my-courses.ts b/src/core/courses/pages/my-courses/my-courses.ts index cd14a0d56..47c28dec8 100644 --- a/src/core/courses/pages/my-courses/my-courses.ts +++ b/src/core/courses/pages/my-courses/my-courses.ts @@ -96,17 +96,20 @@ export class CoreCoursesMyCoursesPage implements OnDestroy { this.courseIds = courseIds.join(','); - // Load course image of all the courses. - promises.push(this.coursesProvider.getCoursesByField('ids', this.courseIds).then((coursesInfo) => { - coursesInfo = this.utils.arrayToObject(coursesInfo, 'id'); - courses.forEach((course) => { - if (coursesInfo[course.id] && coursesInfo[course.id].overviewfiles && coursesInfo[course.id].overviewfiles[0]) { - course.imageThumb = coursesInfo[course.id].overviewfiles[0].fileurl; - } else { - course.imageThumb = false; - } - }); - })); + if (this.courseIds) { + // Load course image of all the courses. + promises.push(this.coursesProvider.getCoursesByField('ids', this.courseIds).then((coursesInfo) => { + coursesInfo = this.utils.arrayToObject(coursesInfo, 'id'); + courses.forEach((course) => { + if (coursesInfo[course.id] && coursesInfo[course.id].overviewfiles && + coursesInfo[course.id].overviewfiles[0]) { + course.imageThumb = coursesInfo[course.id].overviewfiles[0].fileurl; + } else { + course.imageThumb = false; + } + }); + })); + } promises.push(this.coursesProvider.getCoursesAdminAndNavOptions(courseIds).then((options) => { courses.forEach((course) => { @@ -137,7 +140,9 @@ export class CoreCoursesMyCoursesPage implements OnDestroy { promises.push(this.coursesProvider.invalidateUserCourses()); promises.push(this.courseOptionsDelegate.clearAndInvalidateCoursesOptions()); - promises.push(this.coursesProvider.invalidateCoursesByField('ids', this.courseIds)); + if (this.courseIds) { + promises.push(this.coursesProvider.invalidateCoursesByField('ids', this.courseIds)); + } Promise.all(promises).finally(() => { diff --git a/src/core/courses/pages/my-overview/my-overview.ts b/src/core/courses/pages/my-overview/my-overview.ts index fd67329cd..7a35b75d1 100644 --- a/src/core/courses/pages/my-overview/my-overview.ts +++ b/src/core/courses/pages/my-overview/my-overview.ts @@ -217,17 +217,20 @@ export class CoreCoursesMyOverviewPage implements OnDestroy { this.courseIds = courseIds.join(','); - // Load course image of all the courses. - promises.push(this.coursesProvider.getCoursesByField('ids', this.courseIds).then((coursesInfo) => { - coursesInfo = this.utils.arrayToObject(coursesInfo, 'id'); - courses.forEach((course) => { - if (coursesInfo[course.id] && coursesInfo[course.id].overviewfiles && coursesInfo[course.id].overviewfiles[0]) { - course.imageThumb = coursesInfo[course.id].overviewfiles[0].fileurl; - } else { - course.imageThumb = false; - } - }); - })); + if (this.courseIds) { + // Load course image of all the courses. + promises.push(this.coursesProvider.getCoursesByField('ids', this.courseIds).then((coursesInfo) => { + coursesInfo = this.utils.arrayToObject(coursesInfo, 'id'); + courses.forEach((course) => { + if (coursesInfo[course.id] && coursesInfo[course.id].overviewfiles && + coursesInfo[course.id].overviewfiles[0]) { + course.imageThumb = coursesInfo[course.id].overviewfiles[0].fileurl; + } else { + course.imageThumb = false; + } + }); + })); + } return Promise.all(promises).then(() => { return courses.sort((a, b) => { @@ -286,7 +289,9 @@ export class CoreCoursesMyOverviewPage implements OnDestroy { promises.push(this.coursesProvider.invalidateUserCourses()); promises.push(this.courseOptionsDelegate.clearAndInvalidateCoursesOptions()); - promises.push(this.coursesProvider.invalidateCoursesByField('ids', this.courseIds)); + if (this.courseIds) { + promises.push(this.coursesProvider.invalidateCoursesByField('ids', this.courseIds)); + } return Promise.all(promises).finally(() => { switch (this.tabShown) {