Skip to content

Commit 644f3c8

Browse files
committed
MOBILE-2915 lesson: Fix 'End of cluster' text displayed
1 parent f969f99 commit 644f3c8

File tree

2 files changed

+13
-5
lines changed

2 files changed

+13
-5
lines changed

src/addon/mod/lesson/pages/player/player.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,14 +32,14 @@
3232
<!-- Page content. -->
3333
<ion-card *ngIf="!eolData && !processData">
3434
<!-- Content page. -->
35-
<ion-item text-wrap *ngIf="!question">
35+
<ion-item text-wrap *ngIf="!question && pageContent">
3636
<core-format-text [component]="component" [componentId]="lesson.coursemodule" [text]="pageContent"></core-format-text>
3737
</ion-item>
3838

3939
<!-- Question page. -->
4040
<!-- We need to set ngIf loaded to make formGroup directive restart every time a page changes, see MOBILE-2540. -->
4141
<form *ngIf="question && loaded" ion-list [formGroup]="questionForm">
42-
<ion-item-divider text-wrap>
42+
<ion-item-divider text-wrap *ngIf="pageContent">
4343
<core-format-text [component]="component" [componentId]="lesson.coursemodule" [text]="pageContent"></core-format-text>
4444
</ion-item-divider>
4545

src/addon/mod/lesson/providers/helper.ts

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ import { AddonModLessonProvider } from './lesson';
2727
export class AddonModLessonHelperProvider {
2828

2929
constructor(private domUtils: CoreDomUtilsProvider, private fb: FormBuilder, private translate: TranslateService,
30-
private textUtils: CoreTextUtilsProvider, private timeUtils: CoreTimeUtilsProvider) { }
30+
private textUtils: CoreTextUtilsProvider, private timeUtils: CoreTimeUtilsProvider,
31+
private lessonProvider: AddonModLessonProvider) { }
3132

3233
/**
3334
* Given the HTML of next activity link, format it to extract the href and the text.
@@ -149,8 +150,15 @@ export class AddonModLessonHelperProvider {
149150
return contents.innerHTML.trim();
150151
}
151152

152-
// Cannot find contents element, return the page.contents (some elements like videos might not work).
153-
return data.page.contents;
153+
// Cannot find contents element.
154+
if (this.lessonProvider.isQuestionPage(data.page.type) ||
155+
data.page.qtype == AddonModLessonProvider.LESSON_PAGE_BRANCHTABLE) {
156+
// Return page.contents to prevent having duplicated elements (some elements like videos might not work).
157+
return data.page.contents;
158+
} else {
159+
// It's an end of cluster, end of branch, etc. Return the whole pagecontent to match what's displayed in web.
160+
return data.pagecontent;
161+
}
154162
}
155163

156164
/**

0 commit comments

Comments
 (0)