From 1b71f69cfe3a3e23f8bf959a20e278084ecf1b82 Mon Sep 17 00:00:00 2001
From: Dani Palou <dani@moodle.com>
Date: Tue, 30 Apr 2024 15:53:36 +0200
Subject: [PATCH] MOBILE-4560 colors: Use CoreIonicColorNames in missing places

---
 .../competency/classes/competency-plans-source.ts     |  7 ++++---
 .../mod/assign/components/submission/submission.ts    |  3 ++-
 .../features/question/services/question-helper.ts     | 11 ++++++-----
 3 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/src/addons/competency/classes/competency-plans-source.ts b/src/addons/competency/classes/competency-plans-source.ts
index 471db5ba5..53b7f3c09 100644
--- a/src/addons/competency/classes/competency-plans-source.ts
+++ b/src/addons/competency/classes/competency-plans-source.ts
@@ -17,6 +17,7 @@ import { CoreRoutedItemsManagerSource } from '@classes/items-management/routed-i
 import { ADDON_COMPETENCY_COMPETENCIES_PAGE } from '../competency.module';
 import { AddonCompetency, AddonCompetencyPlan, AddonCompetencyProvider } from '../services/competency';
 import { AddonCompetencyHelper } from '../services/competency-helper';
+import { CoreIonicColorNames } from '@singletons/colors';
 
 /**
  * Provides a collection of learning plans.
@@ -73,13 +74,13 @@ export class AddonCompetencyPlansSource extends CoreRoutedItemsManagerSource<Add
             plan.statusname = AddonCompetencyHelper.getPlanStatusName(plan.status);
             switch (plan.status) {
                 case AddonCompetencyProvider.STATUS_ACTIVE:
-                    plan.statuscolor = 'success';
+                    plan.statuscolor = CoreIonicColorNames.SUCCESS;
                     break;
                 case AddonCompetencyProvider.STATUS_COMPLETE:
-                    plan.statuscolor = 'danger';
+                    plan.statuscolor = CoreIonicColorNames.DANGER;
                     break;
                 default:
-                    plan.statuscolor = 'warning';
+                    plan.statuscolor = CoreIonicColorNames.WARNING;
                     break;
             }
         });
diff --git a/src/addons/mod/assign/components/submission/submission.ts b/src/addons/mod/assign/components/submission/submission.ts
index 9cd46da9f..fb3522d0e 100644
--- a/src/addons/mod/assign/components/submission/submission.ts
+++ b/src/addons/mod/assign/components/submission/submission.ts
@@ -60,6 +60,7 @@ import { AddonModAssignModuleHandlerService } from '../../services/handlers/modu
 import { CanLeave } from '@guards/can-leave';
 import { CoreTime } from '@singletons/time';
 import { isSafeNumber, SafeNumber } from '@/core/utils/types';
+import { CoreIonicColorNames } from '@singletons/colors';
 
 /**
  * Component that displays an assignment submission.
@@ -815,7 +816,7 @@ export class AddonModAssignSubmissionComponent implements OnInit, OnDestroy, Can
         if (this.hasOffline || this.submittedOffline) {
             // Offline data.
             this.statusTranslated = Translate.instant('core.notsent');
-            this.statusColor = 'warning';
+            this.statusColor = CoreIonicColorNames.WARNING;
         } else if (!this.assign.teamsubmission) {
 
             // Single submission.
diff --git a/src/core/features/question/services/question-helper.ts b/src/core/features/question/services/question-helper.ts
index 28ab7e1fc..4df7bf94b 100644
--- a/src/core/features/question/services/question-helper.ts
+++ b/src/core/features/question/services/question-helper.ts
@@ -29,6 +29,7 @@ import { CoreQuestionDelegate } from './question-delegate';
 import { CoreIcons } from '@singletons/icons';
 import { CoreUrlUtils } from '@services/utils/url';
 import { ContextLevel } from '@/core/constants';
+import { CoreIonicColorNames } from '@singletons/colors';
 
 /**
  * Service with some common functions to handle questions.
@@ -806,21 +807,21 @@ export class CoreQuestionHelperProvider {
             if ('src' in icon) {
                 if ((icon as HTMLImageElement).src.indexOf('correct') >= 0) {
                     iconName = 'check';
-                    color = 'success';
+                    color = CoreIonicColorNames.SUCCESS;
                 } else if ((icon as HTMLImageElement).src.indexOf('incorrect') >= 0 ) {
                     iconName = 'xmark';
-                    color = 'danger';
+                    color = CoreIonicColorNames.DANGER;
                 }
             } else {
                 if (icon.classList.contains('fa-check-square')) {
                     iconName = 'square-check';
-                    color = 'warning';
+                    color = CoreIonicColorNames.WARNING;
                 } else if (icon.classList.contains('fa-check')) {
                     iconName = 'check';
-                    color = 'success';
+                    color = CoreIonicColorNames.SUCCESS;
                 } else if (icon.classList.contains('fa-xmark') || icon.classList.contains('fa-remove')) {
                     iconName = 'xmark';
-                    color = 'danger';
+                    color = CoreIonicColorNames.DANGER;
                 }
             }