From e891fcb1a11ddf2a6ff76346dd425ee7c960b5e8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pau=20Ferrer=20Oca=C3=B1a?= <crazyserver@gmail.com>
Date: Fri, 1 Feb 2019 11:51:36 +0100
Subject: [PATCH] MOBILE-2858 courses: Do not check course options on empty
 list

---
 src/core/courses/providers/courses.ts | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/core/courses/providers/courses.ts b/src/core/courses/providers/courses.ts
index 548069d07..d4e492aa8 100644
--- a/src/core/courses/providers/courses.ts
+++ b/src/core/courses/providers/courses.ts
@@ -555,6 +555,10 @@ export class CoreCoursesProvider {
      * @return {Promise<any>} Promise resolved with administration options for each course.
      */
     getUserAdministrationOptions(courseIds: number[], siteId?: string): Promise<any> {
+        if (!courseIds || courseIds.length == 0) {
+            return Promise.resolve({});
+        }
+
         return this.sitesProvider.getSite(siteId).then((site) => {
             const params = {
                     courseids: courseIds
@@ -597,6 +601,10 @@ export class CoreCoursesProvider {
      * @return {Promise<any>} Promise resolved with navigation options for each course.
      */
     getUserNavigationOptions(courseIds: number[], siteId?: string): Promise<any> {
+        if (!courseIds || courseIds.length == 0) {
+            return Promise.resolve({});
+        }
+
         return this.sitesProvider.getSite(siteId).then((site) => {
             const params = {
                     courseids: courseIds