Merge pull request #4606 from alfonso-salces/MOBILE-4921

MOBILE-4921 courses: Add course category id and course id in data attr

Author: dpalou

src/addons/block/myoverview/components/myoverview/addon-block-myoverview.html

@@ -164,7 +164,7 @@ <h3 class="item-heading">
             <ion-grid class="ion-no-padding" [class.core-no-grid]="layout !== 'card'" [class.list-item-limited-width]="layout !== 'card'">
                 <ion-row class="ion-no-padding">
                     <ion-col *ngFor="let course of filteredCourses" class="ion-no-padding" size="12" size-sm="6" size-md="6" size-lg="4"
-                        size-xl="3" [attr.data-course-id]="course.id">
+                        size-xl="3" [attr.data-course-id]="course.id" [attr.data-category-id]="course.categoryid">
                         <core-courses-course-list-item [course]="course" class="core-courseoverview" [showDownload]="downloadCourseEnabled"
                             [layout]="layout" />
                     </ion-col>

src/addons/block/timeline/components/events/events.ts

@@ -34,6 +34,7 @@ import { CoreContentLinksHelper } from '@features/contentlinks/services/contentl
     ],
     host: {
         '[attr.data-course-id]': 'course()?.id ?? null',
+        '[attr.data-category-id]': 'course()?.categoryid ?? null',
     },
 })
 export class AddonBlockTimelineEventsComponent implements OnInit {

src/core/features/course/components/course-index/course-index.ts

@@ -57,6 +57,10 @@ export class CoreCourseCourseIndexComponent implements OnInit {
         return this.course?.id ?? null;
     }
 
+    @HostBinding('attr.data-category-id') protected get courseCategoryId(): number | null {
+        return this.course?.categoryid ?? null;
+    }
+
     /**
      * @inheritdoc
      */

src/core/features/course/pages/contents/contents.ts

@@ -97,6 +97,10 @@ export default class CoreCourseContentsPage implements OnInit, OnDestroy, CoreRe
         return this.course?.id ?? null;
     }
 
+    @HostBinding('attr.data-category-id') protected get courseCategoryId(): number | null {
+        return this.course?.categoryid ?? null;
+    }
+
     /**
      * @inheritdoc
      */

src/core/features/course/pages/course-summary/course-summary.ts

@@ -91,6 +91,10 @@ export default class CoreCourseSummaryPage implements OnInit, OnDestroy {
         return this.course?.id ?? null;
     }
 
+    @HostBinding('attr.data-category-id') protected get courseCategoryId(): number | null {
+        return this.course?.categoryid ?? null;
+    }
+
     constructor() {
         // Refresh the view when the app is resumed.
         this.appResumeSubscription = CorePlatform.resume.subscribe(() => {

src/core/features/course/pages/index/index.ts

@@ -80,6 +80,10 @@ export default class CoreCourseIndexPage implements OnInit, OnDestroy {
         return this.course?.id ?? null;
     }
 
+    @HostBinding('attr.data-category-id') protected get courseCategoryId(): number | null {
+        return this.course?.categoryid ?? null;
+    }
+
     constructor() {
         this.selectTabObserver = CoreEvents.on(CORE_COURSE_SELECT_TAB, (data) => {
             const index = data.selectedTab

src/core/features/courses/components/course-list-item/course-list-item.ts

@@ -63,6 +63,7 @@ import { CoreEnrolInfoIcon } from '@features/enrol/services/enrol-delegate';
     ],
     host: {
         '[attr.data-course-id]': 'course.id',
+        '[attr.data-category-id]': 'course.categoryid',
     },
 })
 export class CoreCoursesCourseListItemComponent implements OnInit, OnDestroy, OnChanges {

src/core/features/grades/pages/course/course.ts

@@ -47,6 +47,9 @@ import { CoreSharedModule } from '@/core/shared.module';
     imports: [
         CoreSharedModule,
     ],
+    host: {
+        '[attr.data-course-id]': 'courseId ?? null',
+    },
 })
 export default class CoreGradesCoursePage implements AfterViewInit, OnDestroy {
 

src/core/features/grades/pages/courses/courses.html

@@ -20,7 +20,8 @@ <h1>{{ 'core.grades.grades' | translate }}</h1>
                 <ion-list>
                     @for (course of courses.items; track course.courseid) {
                         <ion-item [attr.aria-label]="course.courseFullName" [attr.aria-current]="courses.getItemAriaCurrent(course)"
-                            class="ion-text-wrap" button [detail]="true" (click)="courses.select(course)">
+                            [attr.data-course-id]="course.courseid" class="ion-text-wrap" button [detail]="true"
+                            (click)="courses.select(course)">
                             <ion-label>
                                 <p class="item-heading">
                                     <core-format-text [text]="course.courseFullName" [contextInstanceId]="course.courseid"