Skip to content

Commit 79eb998

Browse files
committed
MOBILE-1987 guest: Fix addons shown in guest courses
1 parent 59f8e58 commit 79eb998

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

www/core/components/courses/controllers/viewresult.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -82,12 +82,12 @@ angular.module('mm.core.courses')
8282
// Success retrieving the course, we can assume the user has permissions to view it.
8383
course.fullname = c.fullname || course.fullname;
8484
course.summary = c.summary || course.summary;
85-
return loadCourseNavHandlers(refresh);
85+
return loadCourseNavHandlers(refresh, false);
8686
}).catch(function() {
8787
// The user is not an admin/manager. Check if we can provide guest access to the course.
8888
return canAccessAsGuest().then(function(passwordRequired) {
8989
if (!passwordRequired) {
90-
return loadCourseNavHandlers(refresh);
90+
return loadCourseNavHandlers(refresh, true);
9191
} else {
9292
course._handlers = [];
9393
$scope.handlersShouldBeShown = false;
@@ -126,7 +126,7 @@ angular.module('mm.core.courses')
126126
}
127127

128128
// Load course nav handlers.
129-
function loadCourseNavHandlers(refresh) {
129+
function loadCourseNavHandlers(refresh, guest) {
130130
var promises = [],
131131
navOptions,
132132
admOptions;
@@ -147,8 +147,8 @@ angular.module('mm.core.courses')
147147
}));
148148

149149
return $q.all(promises).then(function() {
150-
course._handlers = $mmCoursesDelegate.getNavHandlersFor(
151-
course.id, refresh, navOptions[course.id], admOptions[course.id]);
150+
var getHandlersFn = guest ? $mmCoursesDelegate.getNavHandlersForGuest : $mmCoursesDelegate.getNavHandlersFor;
151+
course._handlers = getHandlersFn(course.id, refresh, navOptions[course.id], admOptions[course.id]);
152152
});
153153

154154
}

0 commit comments

Comments
 (0)