From 6b1e7b70d1e1b228c3862cb1519a579bd3474942 Mon Sep 17 00:00:00 2001
From: Dani Palou <dani@moodle.com>
Date: Mon, 20 Jul 2020 10:03:19 +0200
Subject: [PATCH] MOBILE-3493 course: Don't display selector if course only has
 one section

---
 .../course/components/format/core-course-format.html   |  2 +-
 src/core/course/components/format/format.ts            | 10 +++++++++-
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/core/course/components/format/core-course-format.html b/src/core/course/components/format/core-course-format.html
index 453928f72..1956e9ad6 100644
--- a/src/core/course/components/format/core-course-format.html
+++ b/src/core/course/components/format/core-course-format.html
@@ -11,7 +11,7 @@
         <core-loading [hideUntil]="loaded">
             <!-- Section selector. -->
             <core-dynamic-component [component]="sectionSelectorComponent" [data]="data">
-                <div text-wrap *ngIf="displaySectionSelector && sections && sections.length" padding class="clearfix" ion-row justify-content-between class="safe-padding-horizontal core-button-selector-row" [class.core-section-download]="downloadEnabled">
+                <div text-wrap *ngIf="displaySectionSelector && sections && hasSeveralSections" padding class="clearfix" ion-row justify-content-between class="safe-padding-horizontal core-button-selector-row" [class.core-section-download]="downloadEnabled">
                     <button float-start ion-button icon-start icon-end (click)="showSectionSelector($event)" color="light" class="core-button-select button-no-uppercase" ion-col [attr.aria-label]="('core.course.sections' | translate) + ': ' + (selectedSection && (selectedSection.formattedName || selectedSection.name))" aria-haspopup="true" [attr.aria-expanded]="sectionSelectorExpanded" aria-controls="core-course-section-selector" id="core-course-section-button">
                         <core-icon name="fa-folder"></core-icon>
                         <span class="core-button-select-text">{{selectedSection && (selectedSection.formattedName || selectedSection.name) || 'core.course.sections' | translate }}</span>
diff --git a/src/core/course/components/format/format.ts b/src/core/course/components/format/format.ts
index a439bc728..a5e46d2b5 100644
--- a/src/core/course/components/format/format.ts
+++ b/src/core/course/components/format/format.ts
@@ -76,6 +76,7 @@ export class CoreCourseFormatComponent implements OnInit, OnChanges, OnDestroy {
     stealthModulesSectionId: number = CoreCourseProvider.STEALTH_MODULES_SECTION_ID;
     selectOptions: any = {};
     loaded: boolean;
+    hasSeveralSections: boolean;
 
     protected sectionStatusObserver;
     protected selectTabObserver;
@@ -171,9 +172,16 @@ export class CoreCourseFormatComponent implements OnInit, OnChanges, OnDestroy {
         }
 
         if (changes.sections && this.sections) {
+            const hasAllSections = this.sections[0].id == CoreCourseProvider.ALL_SECTIONS_ID;
+            this.hasSeveralSections = this.sections.length > 2 || (this.sections.length == 2 && !hasAllSections);
+
             if (!this.selectedSection) {
                 // There is no selected section yet, calculate which one to load.
-                if (this.initialSectionId || this.initialSectionNumber) {
+                if (!this.hasSeveralSections) {
+                    // Always load "All sections" to display the section title. If it isn't there just load the section.
+                    this.loaded = true;
+                    this.sectionChanged(this.sections[0]);
+                } else if (this.initialSectionId || this.initialSectionNumber) {
                     // We have an input indicating the section ID to load. Search the section.
                     for (let i = 0; i < this.sections.length; i++) {
                         const section = this.sections[i];