Skip to content

Commit b3199d7

Browse files
committed
Merge pull request #275 from dpalou/MOBILE-1306
MOBILE-1306 courses: Fix missing addons after syncing site
2 parents 9bb28d6 + 2beaf6f commit b3199d7

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

www/core/components/courses/services/courses.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,14 +140,19 @@ angular.module('mm.core.courses')
140140
*/
141141
self.getUserCourse = function(id, preferCache, siteid) {
142142
siteid = siteid || $mmSite.getId();
143+
144+
if (!id) {
145+
return $q.reject();
146+
}
147+
143148
if (typeof preferCache == 'undefined') {
144149
preferCache = false;
145150
}
146151

147152
return self.getUserCourses(preferCache, siteid).then(function(courses) {
148153
var course;
149154
angular.forEach(courses, function(c) {
150-
if (c.id === id) {
155+
if (c.id == id) {
151156
course = c;
152157
}
153158
});

www/core/components/courses/services/delegate.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ angular.module('mm.core.courses')
160160
}).finally(function() {
161161
// Update handlers for all courses.
162162
angular.forEach(coursesHandlers, function(handler, courseId) {
163-
self.updateNavHandlersForCourse(courseId);
163+
self.updateNavHandlersForCourse(parseInt(courseId));
164164
});
165165
});
166166
};

0 commit comments

Comments
 (0)