From cf4f0ddaa8a5e7e8380175c7d07402fa6ea3e8c5 Mon Sep 17 00:00:00 2001
From: Moritz Strohm <strohm@data-quest.de>
Date: Mon, 7 Nov 2022 07:06:34 +0000
Subject: [PATCH] course/members: prevent reading the "write to all" link being
read twice, closes #562
Closes #562
Merge request studip/studip!1132
---
app/views/course/members/accepted_list.php | 16 +++++-----------
app/views/course/members/autor_list.php | 16 +++++-----------
app/views/course/members/awaiting_list.php | 6 ++----
app/views/course/members/dozent_list.php | 16 +++++-----------
app/views/course/members/tutor_list.php | 16 +++++-----------
app/views/course/members/user_list.php | 16 +++++-----------
6 files changed, 27 insertions(+), 59 deletions(-)
diff --git a/app/views/course/members/accepted_list.php b/app/views/course/members/accepted_list.php
index 53e69181355..c484e3c4704 100644
--- a/app/views/course/members/accepted_list.php
+++ b/app/views/course/members/accepted_list.php
@@ -12,17 +12,11 @@
'emailrequest' => 1,
'course_id' => $course_id,
'default_subject' => $subject,
- ]) ?>" data-dialog>
- <?= Icon::create(
- 'inbox',
- Icon::ROLE_CLICKABLE,
- [
- 'title' => sprintf(
- _('Nachricht mit Mailweiterleitung an alle %s versenden'),
- _('vorläufig akzeptierten Nutzer/-innen')
- )
- ]
- ) ?>
+ ]) ?>" data-dialog title="<?= sprintf(
+ _('Nachricht mit Mailweiterleitung an alle %s versenden'),
+ _('vorläufig akzeptierten Nutzer/-innen')
+ ) ?>">
+ <?= Icon::create('inbox') ?>
</a>
</span>
<?= _('Vorläufig akzeptierte Teilnehmende') ?>
diff --git a/app/views/course/members/autor_list.php b/app/views/course/members/autor_list.php
index 5c2ca173ad7..180d090719b 100644
--- a/app/views/course/members/autor_list.php
+++ b/app/views/course/members/autor_list.php
@@ -14,17 +14,11 @@
'who' => 'autor',
'course_id' => $course_id,
'default_subject' => $subject,
- ]) ?>" data-dialog>
- <?= Icon::create(
- 'inbox',
- Icon::ROLE_CLICKABLE,
- [
- 'title' => sprintf(
- _('Nachricht mit Mailweiterleitung an alle %s versenden'),
- htmlReady($status_groups['autor']
- )
- )
- ]) ?>
+ ]) ?>" data-dialog title="<?= sprintf(
+ _('Nachricht mit Mailweiterleitung an alle %s versenden'),
+ htmlReady($status_groups['autor'])
+ ) ?>">
+ <?= Icon::create('inbox') ?>
</a>
</span>
<? endif ?>
diff --git a/app/views/course/members/awaiting_list.php b/app/views/course/members/awaiting_list.php
index cb44dea5688..061ddaf164c 100644
--- a/app/views/course/members/awaiting_list.php
+++ b/app/views/course/members/awaiting_list.php
@@ -11,10 +11,8 @@
'emailrequest' => 1,
'course_id' => $course_id,
'default_subject' => $subject,
- ])?>" data-dialog>
- <?= Icon::create('inbox')->asImg([
- 'title' => _('Nachricht mit Mailweiterleitung an alle Wartenden versenden'),
- ]) ?>
+ ])?>" data-dialog title="<?= _('Nachricht mit Mailweiterleitung an alle Wartenden versenden') ?>">
+ <?= Icon::create('inbox') ?>
</a>
</span>
</caption>
diff --git a/app/views/course/members/dozent_list.php b/app/views/course/members/dozent_list.php
index d839b7e4f9b..ab5ae0a0ac9 100644
--- a/app/views/course/members/dozent_list.php
+++ b/app/views/course/members/dozent_list.php
@@ -8,17 +8,11 @@
'who' => 'dozent',
'course_id' => $course_id,
'default_subject' => $subject
- ]) ?>" data-dialog>
- <?= Icon::create(
- 'inbox',
- Icon::ROLE_CLICKABLE,
- [
- 'title' => sprintf(
- _('Nachricht mit Mailweiterleitung an alle %s versenden'),
- htmlReady($status_groups['dozent'])
- )
- ]
- )->asImg() ?>
+ ]) ?>" data-dialog title="<?= sprintf(
+ _('Nachricht mit Mailweiterleitung an alle %s versenden'),
+ htmlReady($status_groups['dozent'])
+ ) ?>">
+ <?= Icon::create('inbox') ?>
</a>
</span>
<? endif ?>
diff --git a/app/views/course/members/tutor_list.php b/app/views/course/members/tutor_list.php
index bb77d3a5e80..cb0405cbdcd 100644
--- a/app/views/course/members/tutor_list.php
+++ b/app/views/course/members/tutor_list.php
@@ -13,17 +13,11 @@
'emailrequest' => 1,
'course_id' => $course_id,
'default_subject' => $subject,
- ]) ?>" data-dialog>
- <?= Icon::create(
- 'inbox',
- Icon::ROLE_CLICKABLE,
- [
- 'title' => sprintf(
- _('Nachricht mit Mailweiterleitung an alle %s versenden'),
- htmlReady( $status_groups['tutor'])
- )
- ]
- ) ?>
+ ]) ?>" data-dialog title="<?= sprintf(
+ _('Nachricht mit Mailweiterleitung an alle %s versenden'),
+ htmlReady( $status_groups['tutor'])
+ ) ?>">
+ <?= Icon::create('inbox') ?>
</a>
</span>
<? endif ?>
diff --git a/app/views/course/members/user_list.php b/app/views/course/members/user_list.php
index c3102ea3a36..fafef93f673 100644
--- a/app/views/course/members/user_list.php
+++ b/app/views/course/members/user_list.php
@@ -26,17 +26,11 @@
'who' => 'user',
'course_id' => $course_id,
'default_subject' => $subject,
- ]) ?>" data-dialog>
- <?= Icon::create(
- 'inbox',
- Icon::ROLE_CLICKABLE,
- [
- 'title' => sprintf(
- _('Nachricht mit Mailweiterleitung an alle %s versenden'),
- htmlReady($status_groups['user'])
- )
- ]
- ) ?>
+ ]) ?>" data-dialog title="<?= sprintf(
+ _('Nachricht mit Mailweiterleitung an alle %s versenden'),
+ htmlReady($status_groups['user'])
+ ) ?>">
+ <?= Icon::create('inbox') ?>
</a>
</span>
<? endif ?>
--
GitLab