MOBILE-3890 course: Detect language on single activity course

Author: crazyserver

src/addons/mod/quiz/tests/behat/single_activity.feature

@@ -0,0 +1,35 @@
+
+@app_parallel_run_quiz @addon_mod_quiz @app @mod @mod_quiz @javascript @singleactivity @forced_language
+Feature: Test single activity of quiz type in app
+  In order to view a quiz while using the mobile app
+  As a student
+  I need single activity of quiz type functionality to work
+
+  Background:
+    Given the following "users" exist:
+      | username | firstname | lastname |
+      | student1 | First     | Student  |
+    And the following "courses" exist:
+      | fullname                    | shortname | category | format         | activitytype | lang |
+      | Single activity quiz course | C1        | 0        | singleactivity | quiz         | es   |
+    And the following "course enrolments" exist:
+      | user     | course | role    |
+      | student1 | C1     | student |
+    And the following "activities" exist:
+      | activity | name                  | intro                  | course | idnumber | lang |
+      | quiz     | Single activity quiz  | Test quiz description  | C1     | quiz1    | ca   |
+    And the following "language pack" exists:
+      | language | es | ca |
+
+  Scenario: Single activity quiz
+    Given I entered the course "Single activity quiz course" as "student1" in the app
+    Then I should find "Single activity quiz course" in the app
+
+    When I set "page-core-course-index core-course-image" styles to "background" "lightblue"
+    And I set "page-core-course-index core-course-image" styles to "--core-image-visibility" "hidden"
+    Then the UI should match the snapshot
+    And I should find "Encara no s'han afegit preguntes" in the app
+
+    When I press "Participants" in the app
+    Then I should find "Último acceso" in the app
+    And I should find "Participantes" in the app

src/addons/mod/quiz/tests/behat/snapshots/test-single-activity-of-quiz-type-in-app-single-activity-quiz_10.png

@@ -22,6 +22,8 @@ import { CoreCourse } from '@features/course/services/course';
 import type { CoreCourseModuleMainActivityComponent } from '@features/course/classes/main-activity-component';
 import { CoreSharedModule } from '@/core/shared.module';
 import { CoreCourseFormatDynamicComponent } from '@features/course/classes/base-course-format-component';
+import { CoreCourseForceLanguage } from '@features/course/services/course-force-language';
+import { CoreLang } from '@services/lang';
 
 /**
  * Component to display single activity format. It will determine the right component to use and instantiate it.
@@ -60,12 +62,15 @@ export class CoreCourseFormatSingleActivityComponent extends CoreCourseFormatDyn
 
         this.data.courseId = this.course.id;
         this.data.module = module;
+        this.moduleId = module?.id;
 
         if (module && !this.componentClass) {
             // We haven't obtained the class yet. Get it now.
             const component = await CoreCourseModuleDelegate.getMainComponent(this.course, module);
             this.componentClass = component || CoreCourseUnsupportedModuleComponent;
         }
+
+        await this.refreshLanguage();
     }
 
     /**
@@ -93,8 +98,9 @@ export class CoreCourseFormatSingleActivityComponent extends CoreCourseFormatDyn
     /**
      * User entered the page that contains the component.
      */
-    ionViewDidEnter(): void {
+    async ionViewDidEnter(): Promise<void> {
         this.dynamicComponent()?.callComponentMethod('ionViewDidEnter');
+        await this.refreshLanguage();
     }
 
     /**
@@ -104,4 +110,18 @@ export class CoreCourseFormatSingleActivityComponent extends CoreCourseFormatDyn
         this.dynamicComponent()?.callComponentMethod('ionViewDidLeave');
     }
 
+    /**
+     * Refresh the language according to course/module forced language.
+     */
+    async refreshLanguage(): Promise<void> {
+        try {
+            const lang =
+                await CoreCourseForceLanguage.getForcedLanguageFromCourseOrModule(this.course, this.course?.id, this.moduleId);
+
+            await CoreLang.forceContextLanguage(lang);
+        } catch {
+            // Ignore errors.
+        }
+    }
+
 }

src/core/features/course/format/singleactivity/components/singleactivity.ts

@@ -18,12 +18,12 @@ import { ActivatedRoute, NavigationEnd } from '@angular/router';
 import { CoreSitesReadingStrategy } from '@services/sites';
 import { makeSingleton, Router } from '@singletons';
 import {
+    CoreCourseAnyCourseData,
     CoreCourses,
     CoreCourseSearchedData,
 } from '../../courses/services/courses';
 import { CoreNavigator } from '@services/navigator';
 import { filter } from 'rxjs/operators';
-import { CorePromiseUtils } from '@static/promise-utils';
 import { CoreLang } from '@services/lang';
 import { CoreCourse } from './course';
 import { CoreCourseModuleDelegate } from './module-delegate';
@@ -84,9 +84,30 @@ export class CoreCourseForceLanguageService {
      * @returns Language code if forced.
      */
     protected async getForcedLanguageFromRoute(source: CoreCourseForceLanguageSource): Promise<string | undefined> {
-        let course = CoreNavigator.getRouteParam<CoreCourseSearchedData>('course');
-        let courseId = course?.id ?? CoreNavigator.getRouteNumberParam('courseId');
+        const course = CoreNavigator.getRouteParam<CoreCourseSearchedData>('course');
+        const courseId = course?.id ?? CoreNavigator.getRouteNumberParam('courseId');
 
+        let cmId: number | undefined;
+        if (source === CoreCourseForceLanguageSource.MODULE) {
+            cmId = CoreNavigator.getRouteNumberParam('cmId');
+        }
+
+        return this.getForcedLanguageFromCourseOrModule(course, courseId, cmId);
+    }
+
+    /**
+     * Get forced language from course or module.
+     *
+     * @param course Course object.
+     * @param courseId Course ID.
+     * @param cmId Course module ID.
+     * @returns Language code if forced.
+     */
+    async getForcedLanguageFromCourseOrModule(
+        course?: CoreCourseAnyCourseData,
+        courseId?: number,
+        cmId?: number,
+): Promise<string | undefined> {
         if (!courseId) {
             // Not in a course/module, empty the cache and return.
             this.lastNavigationCheck.courseId = 0;
@@ -97,8 +118,8 @@ export class CoreCourseForceLanguageService {
             return;
         }
 
-        if (source === CoreCourseForceLanguageSource.MODULE) {
-            const modLang = await this.getModuleForcedLangFromRoute(courseId);
+        if (cmId) {
+            const modLang = await this.getModuleForcedLang(courseId, cmId);
             if (modLang) {
                 return modLang;
             }
@@ -114,8 +135,8 @@ export class CoreCourseForceLanguageService {
             lang = course.lang;
         } else {
             try {
-                course = await
-                    CoreCourses.getCourseByField('id', courseId, { readingStrategy: CoreSitesReadingStrategy.PREFER_CACHE });
+                course =
+                    await CoreCourses.getCourseByField('id', courseId, { readingStrategy: CoreSitesReadingStrategy.PREFER_CACHE });
                 lang = course.lang;
             } catch {
                 lang = undefined;
@@ -133,26 +154,24 @@ export class CoreCourseForceLanguageService {
      * Get module forced language from route.
      *
      * @param courseId Course ID of the module.
+     * @param cmId Course module ID.
      * @returns Module forced language if found.
      */
-    protected async getModuleForcedLangFromRoute(courseId: number): Promise<string | undefined> {
-        const cmId = CoreNavigator.getRouteNumberParam('cmId');
+    protected async getModuleForcedLang(courseId: number, cmId: number): Promise<string | undefined> {
         let lang: string | undefined = undefined;
-        if (cmId) {
-            if (this.lastNavigationCheck.cmId === cmId) {
-                lang = this.lastNavigationCheck.cmLang;
-            } else {
+
+        if (this.lastNavigationCheck.cmId === cmId) {
+            lang = this.lastNavigationCheck.cmLang;
+        } else {
+            try {
                 // TODO: In the future this should directly return the module language instead of checking the delegate.
                 // See: MDL-87241
-                const module = await CorePromiseUtils.ignoreErrors(
-                    CoreCourse.getModule(cmId, courseId, undefined, true),
-                );
-
-                if (module) {
-                    lang = await CorePromiseUtils.ignoreErrors(
-                        CoreCourseModuleDelegate.getModuleForcedLang(module),
-                    );
-                }
+                const module = await CoreCourse.getModule(cmId, courseId, undefined, true);
+
+                lang = await CoreCourseModuleDelegate.getModuleForcedLang(module);
+            } catch {
+                lang = undefined;
+                cmId = 0;
             }
         }