From 2beaf6f8be18f52bd1656aa0d05826bfec849ab4 Mon Sep 17 00:00:00 2001 From: Dani Palou Date: Thu, 5 Nov 2015 15:25:21 +0100 Subject: [PATCH] MOBILE-1306 courses: Fix missing addons after syncing site --- www/core/components/courses/services/courses.js | 7 ++++++- www/core/components/courses/services/delegate.js | 2 +- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/www/core/components/courses/services/courses.js b/www/core/components/courses/services/courses.js index cac9d39a452..36854296d9e 100644 --- a/www/core/components/courses/services/courses.js +++ b/www/core/components/courses/services/courses.js @@ -140,6 +140,11 @@ angular.module('mm.core.courses') */ self.getUserCourse = function(id, preferCache, siteid) { siteid = siteid || $mmSite.getId(); + + if (!id) { + return $q.reject(); + } + if (typeof preferCache == 'undefined') { preferCache = false; } @@ -147,7 +152,7 @@ angular.module('mm.core.courses') return self.getUserCourses(preferCache, siteid).then(function(courses) { var course; angular.forEach(courses, function(c) { - if (c.id === id) { + if (c.id == id) { course = c; } }); diff --git a/www/core/components/courses/services/delegate.js b/www/core/components/courses/services/delegate.js index 0cf07255a69..dd37748ec9f 100644 --- a/www/core/components/courses/services/delegate.js +++ b/www/core/components/courses/services/delegate.js @@ -160,7 +160,7 @@ angular.module('mm.core.courses') }).finally(function() { // Update handlers for all courses. angular.forEach(coursesHandlers, function(handler, courseId) { - self.updateNavHandlersForCourse(courseId); + self.updateNavHandlersForCourse(parseInt(courseId)); }); }); };