From 89ed649a074acd55b6046b8bd1e00fe2ea69c2ee Mon Sep 17 00:00:00 2001
From: Dani Palou <dani@moodle.com>
Date: Mon, 2 Jul 2018 14:51:11 +0200
Subject: [PATCH] MOBILE-2440 course: Display all sections

---
 src/core/course/pages/section-selector/section-selector.html | 2 +-
 src/core/course/pages/section-selector/section-selector.ts   | 3 ---
 src/core/course/pages/section/section.ts                     | 3 ++-
 src/core/course/providers/helper.ts                          | 2 +-
 4 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/core/course/pages/section-selector/section-selector.html b/src/core/course/pages/section-selector/section-selector.html
index 3f24ebe7d..eee9fab1a 100644
--- a/src/core/course/pages/section-selector/section-selector.html
+++ b/src/core/course/pages/section-selector/section-selector.html
@@ -10,7 +10,7 @@
 </ion-header>
 <ion-content>
     <ng-container *ngFor="let section of sections">
-        <a ion-item *ngIf="sectionHasContent(section)" text-wrap (click)="selectSection(section)" [class.core-primary-item]="selected.id == section.id" [class.item-dimmed]="section.visible === 0 || section.uservisible === false" detail-none>
+        <a ion-item *ngIf="section.hasContent" text-wrap (click)="selectSection(section)" [class.core-primary-item]="selected.id == section.id" [class.item-dimmed]="section.visible === 0 || section.uservisible === false" detail-none>
             <h2><core-format-text [text]="section.formattedName || section.name"></core-format-text></h2>
             <ion-badge color="secondary" *ngIf="section.visible === 0">{{ 'core.course.nocontentavailable' | translate }}</ion-badge>
             <ion-badge color="secondary" *ngIf="section.availabilityinfo"><core-format-text  [text]=" section.availabilityinfo"></core-format-text></ion-badge>
diff --git a/src/core/course/pages/section-selector/section-selector.ts b/src/core/course/pages/section-selector/section-selector.ts
index 0f1f23933..ff76889fc 100644
--- a/src/core/course/pages/section-selector/section-selector.ts
+++ b/src/core/course/pages/section-selector/section-selector.ts
@@ -28,13 +28,10 @@ export class CoreCourseSectionSelectorPage {
 
     sections: any;
     selected: number;
-    sectionHasContent: any;
 
     constructor(navParams: NavParams, courseHelper: CoreCourseHelperProvider, private viewCtrl: ViewController) {
         this.sections = navParams.get('sections');
         this.selected = navParams.get('selected');
-
-        this.sectionHasContent = courseHelper.sectionHasContent;
     }
 
     /**
diff --git a/src/core/course/pages/section/section.ts b/src/core/course/pages/section/section.ts
index 9c3bb2e47..f236ae11f 100644
--- a/src/core/course/pages/section/section.ts
+++ b/src/core/course/pages/section/section.ts
@@ -194,7 +194,8 @@ export class CoreCourseSectionPage implements OnDestroy {
                         // Add a fake first section (all sections).
                         this.sections.unshift({
                             name: this.translate.instant('core.course.allsections'),
-                            id: CoreCourseProvider.ALL_SECTIONS_ID
+                            id: CoreCourseProvider.ALL_SECTIONS_ID,
+                            hasContent: true
                         });
                     }
 
diff --git a/src/core/course/providers/helper.ts b/src/core/course/providers/helper.ts
index 46f026e67..af4aeea19 100644
--- a/src/core/course/providers/helper.ts
+++ b/src/core/course/providers/helper.ts
@@ -1168,7 +1168,7 @@ export class CoreCourseHelperProvider {
      * @return {boolean} Whether the section has content.
      */
     sectionHasContent(section: any): boolean {
-        if (section.id == CoreCourseProvider.ALL_SECTIONS_ID || section.hiddenbynumsections) {
+        if (section.hiddenbynumsections) {
             return false;
         }