From 10d2b5864b401b6e9c7ee7143da7fe053970a583 Mon Sep 17 00:00:00 2001
From: David Carrillo <davidmc@moodle.com>
Date: Tue, 12 Dec 2023 17:47:29 +0100
Subject: [PATCH] MDL-80325 reportbuilder: Update 'Delete' links to red color

- Update all 'Delete' links on RB system report actions to red color
---
 .../classes/reportbuilder/local/systemreports/badges.php   | 2 +-
 .../classes/reportbuilder/local/systemreports/cohorts.php  | 2 +-
 .../classes/reportbuilder/local/systemreports/comments.php | 2 +-
 .../local/systemreports/course_external_tools_list.php     | 3 ++-
 .../classes/local/systemreports/report_schedules.php       | 7 ++++++-
 reportbuilder/classes/local/systemreports/reports_list.php | 7 ++++++-
 6 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/badges/classes/reportbuilder/local/systemreports/badges.php b/badges/classes/reportbuilder/local/systemreports/badges.php
index aaec07927b7..29ffda2c249 100644
--- a/badges/classes/reportbuilder/local/systemreports/badges.php
+++ b/badges/classes/reportbuilder/local/systemreports/badges.php
@@ -258,7 +258,7 @@ class badges extends system_report {
                 'id' => ':courseid',
             ]),
             new pix_icon('t/delete', '', 'core'),
-            [],
+            ['class' => 'text-danger'],
             false,
             new lang_string('delete', 'core')
         ))->add_callback(static function(stdclass $row): bool {
diff --git a/cohort/classes/reportbuilder/local/systemreports/cohorts.php b/cohort/classes/reportbuilder/local/systemreports/cohorts.php
index e764c7c15d3..305e1a86466 100644
--- a/cohort/classes/reportbuilder/local/systemreports/cohorts.php
+++ b/cohort/classes/reportbuilder/local/systemreports/cohorts.php
@@ -239,7 +239,7 @@ class cohorts extends system_report {
         $this->add_action((new action(
             new moodle_url('/cohort/edit.php', ['id' => ':id', 'delete' => 1, 'returnurl' => $returnurl]),
             new pix_icon('t/delete', '', 'core'),
-            [],
+            ['class' => 'text-danger'],
             false,
             new lang_string('delete')
         ))->add_callback(function(stdClass $row): bool {
diff --git a/comment/classes/reportbuilder/local/systemreports/comments.php b/comment/classes/reportbuilder/local/systemreports/comments.php
index da41670e6f8..e4040516145 100644
--- a/comment/classes/reportbuilder/local/systemreports/comments.php
+++ b/comment/classes/reportbuilder/local/systemreports/comments.php
@@ -116,7 +116,7 @@ class comments extends system_report {
         $this->add_action(new action(
             new moodle_url('#'),
             new pix_icon('t/delete', ''),
-            ['data-action' => 'comment-delete', 'data-comment-id' => ':id'],
+            ['data-action' => 'comment-delete', 'data-comment-id' => ':id', 'class' => 'text-danger'],
             false,
             new lang_string('delete')
         ));
diff --git a/mod/lti/classes/reportbuilder/local/systemreports/course_external_tools_list.php b/mod/lti/classes/reportbuilder/local/systemreports/course_external_tools_list.php
index 99feda7af16..5f8751c467e 100644
--- a/mod/lti/classes/reportbuilder/local/systemreports/course_external_tools_list.php
+++ b/mod/lti/classes/reportbuilder/local/systemreports/course_external_tools_list.php
@@ -237,7 +237,8 @@ class course_external_tools_list extends system_report {
                         'data-action' => 'course-tool-delete',
                         'data-course-tool-id' => $row->id,
                         'data-course-tool-name' => $row->name,
-                        'data-course-tool-usage' => $this->perrowtoolusage
+                        'data-course-tool-usage' => $this->perrowtoolusage,
+                        'class' => 'text-danger',
                     ],
                 ));
 
diff --git a/reportbuilder/classes/local/systemreports/report_schedules.php b/reportbuilder/classes/local/systemreports/report_schedules.php
index b25b564c193..f71afe7b51d 100644
--- a/reportbuilder/classes/local/systemreports/report_schedules.php
+++ b/reportbuilder/classes/local/systemreports/report_schedules.php
@@ -307,7 +307,12 @@ class report_schedules extends system_report {
         $this->add_action((new action(
             new moodle_url('#'),
             new pix_icon('t/delete', ''),
-            ['data-action' => 'schedule-delete', 'data-schedule-id' => ':id', 'data-schedule-name' => ':name'],
+            [
+                'data-action' => 'schedule-delete',
+                'data-schedule-id' => ':id',
+                'data-schedule-name' => ':name',
+                'class' => 'text-danger',
+            ],
             false,
             new lang_string('deleteschedule', 'core_reportbuilder')
         ))
diff --git a/reportbuilder/classes/local/systemreports/reports_list.php b/reportbuilder/classes/local/systemreports/reports_list.php
index acbdfd03f2f..17bd95515d5 100644
--- a/reportbuilder/classes/local/systemreports/reports_list.php
+++ b/reportbuilder/classes/local/systemreports/reports_list.php
@@ -281,7 +281,12 @@ class reports_list extends system_report {
         $this->add_action((new action(
             new moodle_url('#'),
             new pix_icon('t/delete', ''),
-            ['data-action' => 'report-delete', 'data-report-id' => ':id', 'data-report-name' => ':name'],
+            [
+                'data-action' => 'report-delete',
+                'data-report-id' => ':id',
+                'data-report-name' => ':name',
+                'class' => 'text-danger',
+            ],
             false,
             new lang_string('deletereport', 'core_reportbuilder')
         ))