From e33627f1c1e7f4c3862cec82f951e542a4391085 Mon Sep 17 00:00:00 2001
From: Dani Palou <dani@moodle.com>
Date: Thu, 27 Jun 2024 13:09:25 +0200
Subject: [PATCH] MOBILE-4612 course-image: Use signals in course-image
 component

---
 .../components/course-image/course-image.html |  6 +--
 .../components/course-image/course-image.ts   | 44 ++++++++-----------
 2 files changed, 21 insertions(+), 29 deletions(-)

diff --git a/src/core/components/course-image/course-image.html b/src/core/components/course-image/course-image.html
index fee483a9d..adc91885c 100644
--- a/src/core/components/course-image/course-image.html
+++ b/src/core/components/course-image/course-image.html
@@ -1,4 +1,4 @@
-<ion-icon *ngIf="!course.courseimage" name="fas-graduation-cap" slot="start" aria-hidden="true" />
-<ion-avatar *ngIf="course.courseimage" slot="start">
-    <img [url]="course.courseimage" core-external-content alt="" (error)="loadFallbackCourseIcon()" />
+<ion-icon *ngIf="!course().courseimage" name="fas-graduation-cap" slot="start" aria-hidden="true" />
+<ion-avatar *ngIf="course().courseimage" slot="start">
+    <img [url]="course().courseimage" core-external-content alt="" (error)="loadFallbackCourseIcon()" />
 </ion-avatar>
diff --git a/src/core/components/course-image/course-image.ts b/src/core/components/course-image/course-image.ts
index 46ce93d9f..a6b559873 100644
--- a/src/core/components/course-image/course-image.ts
+++ b/src/core/components/course-image/course-image.ts
@@ -13,7 +13,7 @@
 // limitations under the License.
 
 import { toBoolean } from '@/core/transforms/boolean';
-import { Component, Input, ElementRef, OnInit, OnChanges, HostBinding } from '@angular/core';
+import { Component, ElementRef, HostBinding, input, effect } from '@angular/core';
 import { CoreCourseListItem } from '@features/courses/services/courses';
 import { CoreCoursesHelper } from '@features/courses/services/courses-helper';
 import { CoreColors } from '@singletons/colors';
@@ -23,41 +23,31 @@ import { CoreColors } from '@singletons/colors';
     templateUrl: 'course-image.html',
     styleUrls: ['./course-image.scss'],
 })
-export class CoreCourseImageComponent implements OnInit, OnChanges {
+export class CoreCourseImageComponent {
 
-    @Input({ required: true }) course!: CoreCourseListItem;
-    @Input({ transform: toBoolean }) fill = false;
+    course = input.required<CoreCourseListItem>();
+    fill = input(false, { transform: toBoolean });
 
     protected element: HTMLElement;
 
     constructor(element: ElementRef) {
         this.element = element.nativeElement;
+
+        effect(() => {
+            this.setCourseColor();
+        });
     }
 
     @HostBinding('class.fill-container')
     get fillContainer(): boolean {
-        return this.fill;
-    }
-
-    /**
-     * @inheritdoc
-     */
-    ngOnInit(): void {
-        this.setCourseColor();
-    }
-
-    /**
-     * @inheritdoc
-     */
-    ngOnChanges(): void {
-        this.setCourseColor();
+        return this.fill();
     }
 
     /**
      * Removes the course image set because it cannot be loaded and set the fallback icon color.
      */
     loadFallbackCourseIcon(): void {
-        this.course.courseimage = undefined;
+        this.course().courseimage = undefined;
 
         // Set the color because it won't be set at this point.
         this.setCourseColor();
@@ -67,15 +57,17 @@ export class CoreCourseImageComponent implements OnInit, OnChanges {
      * Set course color.
      */
     protected async setCourseColor(): Promise<void> {
-        await CoreCoursesHelper.loadCourseColorAndImage(this.course);
+        const course = this.course();
 
-        if (this.course.color) {
-            this.element.style.setProperty('--course-color', this.course.color);
+        await CoreCoursesHelper.loadCourseColorAndImage(course);
 
-            const tint = CoreColors.lighter(this.course.color, 50);
+        if (course.color) {
+            this.element.style.setProperty('--course-color', course.color);
+
+            const tint = CoreColors.lighter(course.color, 50);
             this.element.style.setProperty('--course-color-tint', tint);
-        } else if(this.course.colorNumber !== undefined) {
-            this.element.classList.add('course-color-' + this.course.colorNumber);
+        } else if(course.colorNumber !== undefined) {
+            this.element.classList.add('course-color-' + course.colorNumber);
         }
     }