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: 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 53e6918..c484e3c 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 5c2ca17..180d090 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 cb44dea..061ddaf 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 d839b7e..ab5ae0a 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 bb77d3a..cb0405c 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 c3102ea..fafef93 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 ?>
-- 
cgit v1.0