From 60327697d864e84e5cac172e2406c862cc5a5b0e Mon Sep 17 00:00:00 2001
From: Elmar Ludwig <elmar.ludwig@uni-osnabrueck.de>
Date: Mon, 21 Nov 2022 09:28:21 +0000
Subject: [PATCH] the Icon and Button APIs already escape attributes, fixes
#1797
Closes #1797
Merge request studip/studip!1179
---
app/controllers/course/ilias_interface.php | 2 +-
app/views/admin/course_wizard_steps/index.php | 2 +-
app/views/admin/ilias_interface/index.php | 4 ++--
app/views/admin/lti/index.php | 2 +-
app/views/admin/specification/index.php | 2 +-
app/views/contact/index.php | 4 ++--
app/views/course/forum/area/add.php | 2 +-
app/views/course/forum/index/_threads.php | 4 ++--
app/views/course/ilias_interface/add_object.php | 2 +-
app/views/course/ilias_interface/index.php | 2 +-
app/views/course/ilias_interface/view_object.php | 2 +-
app/views/course/lti/index.php | 2 +-
app/views/course/members/accepted_list.php | 4 ++--
app/views/course/members/add_dozent.php | 2 +-
app/views/course/members/add_member.php | 2 +-
app/views/course/members/add_tutor.php | 2 +-
app/views/course/members/autor_list.php | 4 ++--
app/views/course/members/awaiting_list.php | 6 +++---
app/views/course/members/dozent_list.php | 4 ++--
app/views/course/members/import_autorlist.php | 2 +-
app/views/course/members/tutor_list.php | 4 ++--
app/views/course/members/user_list.php | 4 ++--
app/views/course/statusgroups/_group.php | 10 +++++-----
app/views/course/statusgroups/_member.php | 4 ++--
app/views/course/timesrooms/_cycleRow.php | 15 ---------------
app/views/fachabschluss/abschluesse/index.php | 2 +-
app/views/fachabschluss/faecher/index.php | 2 +-
app/views/fachabschluss/kategorien/index.php | 2 +-
app/views/lvgruppen/lvgruppen/index.php | 2 +-
app/views/module/module/details.php | 2 +-
app/views/module/module/modul.php | 2 +-
app/views/module/module/module.php | 6 +++---
app/views/module/module/modulteil.php | 2 +-
app/views/module/module/modulteil_lvg.php | 4 ++--
app/views/my_courses/_deputy_bosses.php | 4 ++--
app/views/my_ilias_accounts/index.php | 4 ++--
app/views/resources/admin/properties.php | 5 ++---
app/views/search/archive/index.php | 2 +-
app/views/studiengaenge/informationen/degree.php | 4 ++--
app/views/studiengaenge/informationen/index.php | 2 +-
.../studiengaenge/informationen/showdegree.php | 4 ++--
.../informationen/showstudycourse.php | 4 ++--
.../studiengaenge/stgteilbezeichnungen/index.php | 2 +-
.../studiengaenge/aufbaustg_table.php | 2 +-
.../studiengaenge/studiengaenge/studiengaenge.php | 2 +-
.../studiengaenge/studiengangteile.php | 4 ++--
.../studiengangteile/details_grouped.php | 2 +-
.../studiengaenge/studiengangteile/index.php | 2 +-
app/views/studiengaenge/versionen/abschnitte.php | 2 +-
.../studiengaenge/versionen/details_abschnitt.php | 4 ++--
app/views/studiengaenge/versionen/versionen.php | 6 +++---
templates/calendar/entries/icons.php | 4 ++--
templates/mail/notification_html.php | 2 +-
templates/tabs.php | 6 +++---
54 files changed, 84 insertions(+), 100 deletions(-)
diff --git a/app/controllers/course/ilias_interface.php b/app/controllers/course/ilias_interface.php
index d6ebce2083a..071fe33c5fc 100644
--- a/app/controllers/course/ilias_interface.php
+++ b/app/controllers/course/ilias_interface.php
@@ -156,7 +156,7 @@ class Course_IliasInterfaceController extends AuthenticatedController
[
'data-confirm' => sprintf(
_('Verknüpfung zum Kurs in %s entfernen? Hierdurch werden auch die Verknüpfungen zu allen Objekten innerhalb des Kurses entfernt.'),
- htmlReady($this->ilias_list[$ilias_index]->getName())
+ $this->ilias_list[$ilias_index]->getName()
)
]
);
diff --git a/app/views/admin/course_wizard_steps/index.php b/app/views/admin/course_wizard_steps/index.php
index bed64e5ad45..b8857ca35a6 100644
--- a/app/views/admin/course_wizard_steps/index.php
+++ b/app/views/admin/course_wizard_steps/index.php
@@ -58,7 +58,7 @@
'formaction' => $controller->url_for("admin/coursewizardsteps/delete/{$step->id}"),
'data-confirm' => sprintf(
_('Soll der Eintrag "%s" wirklich gelöscht werden?'),
- htmlReady($step->name)
+ $step->name
),
])
) ?>
diff --git a/app/views/admin/ilias_interface/index.php b/app/views/admin/ilias_interface/index.php
index 6c103ac3405..ddf7f23682b 100644
--- a/app/views/admin/ilias_interface/index.php
+++ b/app/views/admin/ilias_interface/index.php
@@ -72,8 +72,8 @@
'title' => _('Konfiguration löschen'),
'formaction' => $controller->url_for("admin/ilias_interface/delete/$ilias_index"),
'data-confirm' => sprintf(
- sprintf(_('Soll die ILIAS-Installation "%s" wirklich entfernt werden?'),
- htmlReady($ilias_config['name']))
+ _('Soll die ILIAS-Installation "%s" wirklich entfernt werden?'),
+ $ilias_config['name']
),
])
) ?>
diff --git a/app/views/admin/lti/index.php b/app/views/admin/lti/index.php
index 919aae895a1..0627f64993b 100644
--- a/app/views/admin/lti/index.php
+++ b/app/views/admin/lti/index.php
@@ -49,7 +49,7 @@
<?= Icon::create('trash')->asInput([
'formaction' => $controller->url_for('admin/lti/delete/' . $tool->id),
'title' => _('LTI-Tool löschen'),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich das LTI-Tool "%s" löschen?'), htmlReady($tool->name))
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich das LTI-Tool "%s" löschen?'), $tool->name)
]) ?>
</td>
</tr>
diff --git a/app/views/admin/specification/index.php b/app/views/admin/specification/index.php
index 6fbf71185c4..b661873343f 100644
--- a/app/views/admin/specification/index.php
+++ b/app/views/admin/specification/index.php
@@ -32,7 +32,7 @@
</a>
<?=Icon::create('trash', 'clickable', tooltip2(_('Regel löschen')))->asInput([
'formaction' => $controller->url_for('admin/specification/delete/' . $rule['lock_id']),
- 'data-confirm' => sprintf(_('Wollen Sie die Regel "%s" wirklich löschen?'), htmlReady($rule['name'])),
+ 'data-confirm' => sprintf(_('Wollen Sie die Regel "%s" wirklich löschen?'), $rule['name']),
'style' => 'vertical-align: bottom'
])?>
</td>
diff --git a/app/views/contact/index.php b/app/views/contact/index.php
index 503c65a2463..7443b4ef95c 100644
--- a/app/views/contact/index.php
+++ b/app/views/contact/index.php
@@ -14,7 +14,7 @@
<?= Icon::create('trash')->asInput(
['formaction' => $controller->url_for('contact/deleteGroup/' . $filter),
'title' => _('Gruppe löschen'),
- 'data-confirm' => sprintf(_('Gruppe %s wirklich löschen?'), htmlReady($title))]) ?>
+ 'data-confirm' => sprintf(_('Gruppe %s wirklich löschen?'), $title)]) ?>
<? endif; ?>
</span>
</caption>
@@ -86,7 +86,7 @@
[
'data-confirm' => sprintf(
_('Wollen Sie %s wirklich von der Liste entfernen'),
- htmlReady($contact->username)
+ $contact->username
),
'formaction' => $controller->url_for('contact/remove/' . $filter, ['user' => $contact->username])
])
diff --git a/app/views/course/forum/area/add.php b/app/views/course/forum/area/add.php
index 9524a6596b6..db681e7c40f 100644
--- a/app/views/course/forum/area/add.php
+++ b/app/views/course/forum/area/add.php
@@ -13,7 +13,7 @@
<? else : ?>
<? $num_postings = ForumVisit::getCount($entry['topic_id'], $visitdate) ?>
<?= Icon::create('forum', $num_postings > 0 ? Icon::ROLE_ATTENTION : Icon::ROLE_INFO)->asImg([
- 'title' => htmlReady(ForumHelpers::getVisitText($num_postings, $entry['topic_id'], $constraint['depth'])),
+ 'title' => ForumHelpers::getVisitText($num_postings, $entry['topic_id'], $constraint['depth']),
'style' => 'margin-bottom: 15px;',
]) ?>
<? endif ?>
diff --git a/app/views/course/forum/index/_threads.php b/app/views/course/forum/index/_threads.php
index 97f0c4c1dd6..3622ea52f6c 100644
--- a/app/views/course/forum/index/_threads.php
+++ b/app/views/course/forum/index/_threads.php
@@ -46,7 +46,7 @@
<? else : ?>
<? $num_postings = ForumVisit::getCount($entry['topic_id'], $visitdate) ?>
<?= Icon::create('forum', $num_postings > 0 ? Icon::ROLE_ATTENTION : Icon::ROLE_INFO)->asImg([
- 'title' => htmlReady(ForumHelpers::getVisitText($num_postings, $entry['topic_id'])),
+ 'title' => ForumHelpers::getVisitText($num_postings, $entry['topic_id']),
]) ?>
<? endif ?>
@@ -162,7 +162,7 @@
'formaction' => $controller->url_for("course/forum/index/delete_entry/{$entry['topic_id']}"),
'data-confirm' => sprintf(
_('Sind sie sicher dass Sie den Eintrag %s löschen möchten?'),
- htmlReady($entry['name'])
+ $entry['name']
)
]
)
diff --git a/app/views/course/ilias_interface/add_object.php b/app/views/course/ilias_interface/add_object.php
index b2b5e022a9e..356a82dcb16 100644
--- a/app/views/course/ilias_interface/add_object.php
+++ b/app/views/course/ilias_interface/add_object.php
@@ -102,7 +102,7 @@
Icon::create('info-circle'),
[
'title' => _('Info'),
- 'formaction' => $controller->url_for($module->getRoute('view_course') .'?ilias_search='.htmlReady($ilias_search)),
+ 'formaction' => $controller->url_for($module->getRoute('view_course') .'?ilias_search='.urlencode($ilias_search)),
'data-dialog' => ''
]
)->condition($edit_permission)->addButton(
diff --git a/app/views/course/ilias_interface/index.php b/app/views/course/ilias_interface/index.php
index c45d058b7e0..74370720156 100644
--- a/app/views/course/ilias_interface/index.php
+++ b/app/views/course/ilias_interface/index.php
@@ -71,7 +71,7 @@
Icon::create('learnmodule+decline', Icon::ROLE_CLICKABLE, [
'title' => _('Entfernen'),
'formaction' => $controller->url_for($module->getRoute('remove')),
- 'data-confirm' => $module->siblings_count < 2 ? sprintf(_('Dies ist die einzige Instanz des Objekts "%s". Durch das Entfernen aus dem Kurs wird das Objekt unwiderruflich gelöscht! Wollen Sie das Objekt wirklich löschen?'), htmlReady($module->getTitle())) : sprintf(_('Wollen Sie das Objekt "%s" jetzt entfernen?'), htmlReady($module->getTitle())),
+ 'data-confirm' => $module->siblings_count < 2 ? sprintf(_('Dies ist die einzige Instanz des Objekts "%s". Durch das Entfernen aus dem Kurs wird das Objekt unwiderruflich gelöscht! Wollen Sie das Objekt wirklich löschen?'), $module->getTitle()) : sprintf(_('Wollen Sie das Objekt "%s" jetzt entfernen?'), $module->getTitle()),
'target' => '_blank',
'rel' => 'noopener noreferrer'
])
diff --git a/app/views/course/ilias_interface/view_object.php b/app/views/course/ilias_interface/view_object.php
index 3ca5c705bd9..457e07e1e0a 100644
--- a/app/views/course/ilias_interface/view_object.php
+++ b/app/views/course/ilias_interface/view_object.php
@@ -11,7 +11,7 @@
<? endif ?>
<? if ($ilias->isActive() && !$mode) : ?>
<? if ($edit_permission && $module->isAllowed('delete')) : ?>
- <?= Studip\LinkButton::create(_('Entfernen'), $controller->url_for($module->getRoute('remove') . '?ilias_search=' . $ilias_search), ['data-confirm' => $module->siblings_count < 2 ? sprintf(_('Dies ist die einzige Instanz des Objekts "%s". Durch das Entfernen aus dem Kurs wird das Objekt unwiderruflich gelöscht! Wollen Sie das Objekt wirklich löschen?'), htmlReady($module->getTitle())) : sprintf(_('Wollen Sie das Objekt "%s" jetzt entfernen?'), htmlReady($module->getTitle()))]) ?>
+ <?= Studip\LinkButton::create(_('Entfernen'), $controller->url_for($module->getRoute('remove') . '?ilias_search=' . $ilias_search), ['data-confirm' => $module->siblings_count < 2 ? sprintf(_('Dies ist die einzige Instanz des Objekts "%s". Durch das Entfernen aus dem Kurs wird das Objekt unwiderruflich gelöscht! Wollen Sie das Objekt wirklich löschen?'), $module->getTitle()) : sprintf(_('Wollen Sie das Objekt "%s" jetzt entfernen?'), $module->getTitle())]) ?>
<? endif ?>
<?= $module->isAllowed('start') ? Studip\LinkButton::create(_('Starten'), $controller->url_for($module->getRoute('start')), ['target' => '_blank', 'rel' => 'noopener noreferrer']) :'' ?>
<?= $module->isAllowed('edit') ? Studip\LinkButton::create(_('Bearbeiten'), $controller->url_for($module->getRoute('edit')), ['target' => '_blank', 'rel' => 'noopener noreferrer']) :'' ?>
diff --git a/app/views/course/lti/index.php b/app/views/course/lti/index.php
index 3b2fb8072b6..e04e3bf3349 100644
--- a/app/views/course/lti/index.php
+++ b/app/views/course/lti/index.php
@@ -37,7 +37,7 @@
<?= Icon::create('trash')->asInput([
'formaction' => $controller->url_for('course/lti/delete/' . $lti_data->position),
'title' => _('Abschnitt löschen'),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich den Abschnitt "%s" löschen?'), htmlReady($lti_data->title))
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich den Abschnitt "%s" löschen?'), $lti_data->title)
]) ?>
</form>
</nav>
diff --git a/app/views/course/members/accepted_list.php b/app/views/course/members/accepted_list.php
index c484e3c4704..7b34ef37f91 100644
--- a/app/views/course/members/accepted_list.php
+++ b/app/views/course/members/accepted_list.php
@@ -125,7 +125,7 @@
'mail',
Icon::ROLE_CLICKABLE,
[
- 'title' => sprintf('Nachricht mit Weiterleitung an %s senden', htmlReady($fullname))
+ 'title' => sprintf('Nachricht mit Weiterleitung an %s senden', $fullname)
]
),
['data-dialog' => '']
@@ -139,7 +139,7 @@
'door-leave',
Icon::ROLE_CLICKABLE,
[
- 'title' => sprintf(_('%s austragen'), htmlReady($fullname))
+ 'title' => sprintf(_('%s austragen'), $fullname)
]
)
) ?>
diff --git a/app/views/course/members/add_dozent.php b/app/views/course/members/add_dozent.php
index a4a508f5ef8..4e90ab11d88 100644
--- a/app/views/course/members/add_dozent.php
+++ b/app/views/course/members/add_dozent.php
@@ -27,7 +27,7 @@
<?= Studip\Button::createAccept(
_('Eintragen'),
'add_dozent',
- ['title' => sprintf(_("als %s eintragen"), htmlReady($decoratedStatusGroups['dozent']))]
+ ['title' => sprintf(_("als %s eintragen"), $decoratedStatusGroups['dozent'])]
) ?>
<?= Studip\LinkButton::createCancel(_('Abbrechen'), $controller->url_for('course/members/index')) ?>
</td>
diff --git a/app/views/course/members/add_member.php b/app/views/course/members/add_member.php
index 8d597e4ee4f..2e258b4a8ad 100644
--- a/app/views/course/members/add_member.php
+++ b/app/views/course/members/add_member.php
@@ -85,7 +85,7 @@
_('Eintragen'),
'add_member_list',
[
- 'title' => sprintf(_("als %s eintragen"), htmlReady($decoratedStatusGroups['autor']))
+ 'title' => sprintf(_("als %s eintragen"), $decoratedStatusGroups['autor'])
]
) ?>
</td>
diff --git a/app/views/course/members/add_tutor.php b/app/views/course/members/add_tutor.php
index 95e9abc2f74..36cda6f4aab 100644
--- a/app/views/course/members/add_tutor.php
+++ b/app/views/course/members/add_tutor.php
@@ -24,7 +24,7 @@
<?= Studip\Button::createAccept(
_('Eintragen'),
'add_tutor',
- ['title' => sprintf(_('als %s eintragen'), htmlReady($decoratedStatusGroups['tutor']))]
+ ['title' => sprintf(_('als %s eintragen'), $decoratedStatusGroups['tutor'])]
) ?>
<?= Studip\LinkButton::createCancel(_('Abbrechen'), $controller->url_for('course/members/index')) ?>
</td>
diff --git a/app/views/course/members/autor_list.php b/app/views/course/members/autor_list.php
index 5b64a3b5bc0..d1a6aac68af 100644
--- a/app/views/course/members/autor_list.php
+++ b/app/views/course/members/autor_list.php
@@ -143,7 +143,7 @@
Icon::create(
'mail',
Icon::ROLE_CLICKABLE, [
- 'title' => sprintf(_('Nachricht mit Weiterleitung an %s senden'), htmlReady($fullname)),
+ 'title' => sprintf(_('Nachricht mit Weiterleitung an %s senden'), $fullname),
]),
['data-dialog' => '']
) ?>
@@ -162,7 +162,7 @@
Icon::create(
'door-leave',
Icon::ROLE_CLICKABLE,
- ['title' => sprintf(_('%s austragen'), htmlReady($fullname))])
+ ['title' => sprintf(_('%s austragen'), $fullname)])
) ?>
<? endif ?>
<?= $actionMenu->render() ?>
diff --git a/app/views/course/members/awaiting_list.php b/app/views/course/members/awaiting_list.php
index 061ddaf164c..b8c1e08c605 100644
--- a/app/views/course/members/awaiting_list.php
+++ b/app/views/course/members/awaiting_list.php
@@ -82,7 +82,7 @@
<a href="<?= $controller->url_for('profile?username=' . $waiting['username']) ?>" <? if ($waiting['mkdate'] >= $last_visitdate) echo 'class="new-member"'; ?>>
<?= Avatar::getAvatar($waiting['user_id'], $waiting['username'])->getImageTag(Avatar::SMALL, [
'style' => 'margin-right: 5px',
- 'title' => htmlReady($fullname),
+ 'title' => $fullname,
]) ?>
<?= htmlReady($fullname) ?>
</a>
@@ -112,7 +112,7 @@
[
'title' => sprintf(
_('Nachricht mit Weiterleitung an %s senden'),
- htmlReady($fullname)
+ $fullname
)
]
),
@@ -126,7 +126,7 @@
Icon::create(
'door-leave',
'clickable',
- ['title' => sprintf(_('%s austragen'), htmlReady($fullname))]
+ ['title' => sprintf(_('%s austragen'), $fullname)]
)
) ?>
<? endif ?>
diff --git a/app/views/course/members/dozent_list.php b/app/views/course/members/dozent_list.php
index ab5ae0a0ac9..9b11d075a7b 100644
--- a/app/views/course/members/dozent_list.php
+++ b/app/views/course/members/dozent_list.php
@@ -81,7 +81,7 @@
Icon::create(
'mail',
Icon::ROLE_CLICKABLE,
- ['title' => sprintf('Nachricht mit Weiterleitung an %s senden', htmlReady($fullname))]
+ ['title' => sprintf('Nachricht mit Weiterleitung an %s senden', $fullname)]
),
['data-dialog' => '1']
) ?>
@@ -100,7 +100,7 @@
Icon::create(
'door-leave',
Icon::ROLE_CLICKABLE,
- ['title' => sprintf(_('%s austragen'), htmlReady($fullname))]
+ ['title' => sprintf(_('%s austragen'), $fullname)]
)
) ?>
<? endif ?>
diff --git a/app/views/course/members/import_autorlist.php b/app/views/course/members/import_autorlist.php
index 48a73eb1ecb..450270cb260 100644
--- a/app/views/course/members/import_autorlist.php
+++ b/app/views/course/members/import_autorlist.php
@@ -40,7 +40,7 @@
<footer data-dialog-button>
<?= Button::createAccept(_('Eintragen'), 'add_member_list',
- ['title' => sprintf(_("als %s eintragen"), htmlReady(get_title_for_status('autor', 1)))]) ?>
+ ['title' => sprintf(_("als %s eintragen"), get_title_for_status('autor', 1))]) ?>
<?= LinkButton::createCancel(_('Abbrechen'), $controller->url_for('course/members/index')) ?>
</footer>
</form>
diff --git a/app/views/course/members/tutor_list.php b/app/views/course/members/tutor_list.php
index cb0405cbdcd..6af208c698f 100644
--- a/app/views/course/members/tutor_list.php
+++ b/app/views/course/members/tutor_list.php
@@ -131,7 +131,7 @@
Icon::create(
'mail',
Icon::ROLE_CLICKABLE,
- ['title' => sprintf(_('Nachricht mit Weiterleitung an %s senden'), htmlReady($fullname))]
+ ['title' => sprintf(_('Nachricht mit Weiterleitung an %s senden'), $fullname)]
),
['data-dialog' => '']
) ?>
@@ -150,7 +150,7 @@
Icon::create(
'door-leave',
'clickable',
- ['title' => sprintf(_('%s austragen'), htmlReady($fullname))]
+ ['title' => sprintf(_('%s austragen'), $fullname)]
)
) ?>
<? endif ?>
diff --git a/app/views/course/members/user_list.php b/app/views/course/members/user_list.php
index fafef93f673..9b2aceabc28 100644
--- a/app/views/course/members/user_list.php
+++ b/app/views/course/members/user_list.php
@@ -119,7 +119,7 @@
[
'title' => sprintf(
_('Nachricht mit Weiterleitung an %s senden'),
- htmlReady($fullname)
+ $fullname
)
]
),
@@ -141,7 +141,7 @@
Icon::create(
'door-leave',
Icon::ROLE_CLICKABLE,
- ['title' => sprintf(_('%s austragen'), htmlReady($fullname))]
+ ['title' => sprintf(_('%s austragen'), $fullname)]
)
) ?>
<? endif ?>
diff --git a/app/views/course/statusgroups/_group.php b/app/views/course/statusgroups/_group.php
index aa0ebb2b111..d84b456fe91 100644
--- a/app/views/course/statusgroups/_group.php
+++ b/app/views/course/statusgroups/_group.php
@@ -29,7 +29,7 @@
<?php if ($is_autor && !$is_tutor && $group->id != 'nogroup' && $group->isMember($GLOBALS['user']->id)) : ?>
<a href="<?= $controller->link_for('messages/write', [
'group_id' => $group->id,
- 'default_subject' => htmlReady($course_title . ' (' . $group->name . ')'),
+ 'default_subject' => $course_title . ' (' . $group->name . ')',
]) ?>" data-dialog="size=auto">
<?= Icon::create('mail', Icon::ROLE_CLICKABLE, ['title' => sprintf(
_('Nachricht an alle Mitglieder der Gruppe %s schicken'),
@@ -44,7 +44,7 @@
<a href="<?= $controller->url_for('course/statusgroups/join', $group->id) ?>">
<?= Icon::create('door-enter', Icon::ROLE_CLICKABLE,
['title' => sprintf(_('Mitglied von Gruppe %s werden'),
- htmlReady($group->name))]) ?></a>
+ $group->name)]) ?></a>
<?php elseif ($group->id != 'nogroup' && $group->selfassign &&
$group->selfassign_start > time()) : ?>
<?= Icon::create('door-enter', Icon::ROLE_INACTIVE,
@@ -59,7 +59,7 @@
<a href="<?= $controller->url_for('course/statusgroups/leave', $group->id) ?>" data-confirm="<?= sprintf(_('Aus Gruppe %s austragen'),htmlReady($group->name)) . '?' ?>">
<?= Icon::create('door-leave', Icon::ROLE_ATTENTION,
['title' => sprintf(_('Aus Gruppe %s austragen'),
- htmlReady($group->name))]) ?></a>
+ $group->name)]) ?></a>
<?php endif ?>
</nav>
<?php endif ?>
@@ -69,7 +69,7 @@
->addLink(
$controller->url_for('messages/write', [
'group_id' => $group->id,
- 'default_subject' => htmlReady($course_title . ' (' . $group->name . ')'),
+ 'default_subject' => $course_title . ' (' . $group->name . ')',
]),
_('Nachricht schicken'),
Icon::create('mail', Icon::ROLE_CLICKABLE, [
@@ -135,7 +135,7 @@
<a href="<?= $controller->url_for('messages/write', [
'filter' => 'not_grouped',
'course_id' => $course_id,
- 'default_subject' => htmlReady($course_title).' ('.htmlReady($group->name).')'
+ 'default_subject' => $course_title.' ('.$group->name.')'
]) ?>" data-dialog="size=auto;">
<?= Icon::create('mail')->asImg(['title' => _('Nachricht an alle nicht zugeordneten Personen schicken')]) ?>
</a>
diff --git a/app/views/course/statusgroups/_member.php b/app/views/course/statusgroups/_member.php
index ce3507b312e..2baa4adfaf8 100644
--- a/app/views/course/statusgroups/_member.php
+++ b/app/views/course/statusgroups/_member.php
@@ -74,8 +74,8 @@
]),
['data-confirm' => sprintf(
_('Soll %s wirklich aus der Gruppe %s entfernt werden?'),
- htmlReady($user_fullname),
- htmlReady($group->name)
+ $user_fullname,
+ $group->name
)]
);
}
diff --git a/app/views/course/timesrooms/_cycleRow.php b/app/views/course/timesrooms/_cycleRow.php
index d1adcdd6c8b..1d6bd2ca6f3 100644
--- a/app/views/course/timesrooms/_cycleRow.php
+++ b/app/views/course/timesrooms/_cycleRow.php
@@ -127,21 +127,6 @@ $is_exTermin = $termin instanceof CourseExDate;
['data-dialog' => '']
) ?>
- <? $params = [
- 'type' => 'image',
- 'class' => 'middle',
- 'name' => 'delete_single_date',
- 'data-confirm' => _('Wollen Sie diesen Termin wirklich löschen / ausfallen lassen?')
- . '<br>' . implode('<br>', array_map('htmlReady', $termin->getDeletionWarnings())),
- 'formaction' => $controller->url_for(
- 'course/timesrooms/deleteSingle/' . $termin->id,
- ['cycle_id' => $termin->metadate_id] + $linkAttributes
- ),
- ] ?>
- <? if (Request::isXhr()) : ?>
- <? $params['data-dialog'] = 'size=big' ?>
- <? endif ?>
-
<? $actionMenu->addLink(
$controller->url_for(
'course/timesrooms/stack',
diff --git a/app/views/fachabschluss/abschluesse/index.php b/app/views/fachabschluss/abschluesse/index.php
index e541257f7d4..99d60304d0c 100644
--- a/app/views/fachabschluss/abschluesse/index.php
+++ b/app/views/fachabschluss/abschluesse/index.php
@@ -38,7 +38,7 @@
'formaction' => $controller->url_for('/delete/' . $abschluss->id),
'data-confirm' => sprintf(
_('Wollen Sie wirklich den Abschluss "%s" löschen?'),
- htmlReady($abschluss->name)
+ $abschluss->name
),
'name' => 'delete'
]); ?>
diff --git a/app/views/fachabschluss/faecher/index.php b/app/views/fachabschluss/faecher/index.php
index 6710ec48fbc..9e2166fd5fd 100644
--- a/app/views/fachabschluss/faecher/index.php
+++ b/app/views/fachabschluss/faecher/index.php
@@ -35,7 +35,7 @@
<?= Icon::create('trash', Icon::ROLE_CLICKABLE, tooltip2(_('Fach löschen')))->asInput(
[
'formaction' => $controller->url_for('/delete/' . $fach->id),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich das Fach "%s" löschen?'), htmlReady($fach->name)),
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich das Fach "%s" löschen?'), $fach->name),
'name' => 'delete'
]); ?>
<? else : ?>
diff --git a/app/views/fachabschluss/kategorien/index.php b/app/views/fachabschluss/kategorien/index.php
index effd06695f3..51a485914a0 100644
--- a/app/views/fachabschluss/kategorien/index.php
+++ b/app/views/fachabschluss/kategorien/index.php
@@ -48,7 +48,7 @@
<?= Icon::create('trash', Icon::ROLE_CLICKABLE, tooltip2(_('Abschluss-Kategorie löschen')))->asInput(
[
'formaction' => $controller->url_for('/delete', $kategorie->id),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich die Abschluss-Kategorie "%s" löschen?'), htmlReady($kategorie->name)),
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich die Abschluss-Kategorie "%s" löschen?'), $kategorie->name),
'name' => 'delete'
]); ?>
<? else : ?>
diff --git a/app/views/lvgruppen/lvgruppen/index.php b/app/views/lvgruppen/lvgruppen/index.php
index e4800828688..25f8f1a5b68 100644
--- a/app/views/lvgruppen/lvgruppen/index.php
+++ b/app/views/lvgruppen/lvgruppen/index.php
@@ -53,7 +53,7 @@
'formaction' => $controller->url_for('/delete', $lvgruppe->id),
'data-confirm' => sprintf(
_('Wollen Sie wirklich die Lehrveranstaltungsgruppe "%s" löschen?'),
- htmlReady($lvgruppe->getDisplayName())
+ $lvgruppe->getDisplayName()
)
]
) ?>
diff --git a/app/views/module/module/details.php b/app/views/module/module/details.php
index 5ae025a034c..15dbafda073 100644
--- a/app/views/module/module/details.php
+++ b/app/views/module/module/details.php
@@ -67,7 +67,7 @@
'formaction' => $controller->url_for('/delete_modulteil/' . $modulteil->id),
'data-confirm' => sprintf(
_('Wollen Sie wirklich den Modulteil "%s" löschen?'),
- htmlReady($modulteil->getDisplayName())
+ $modulteil->getDisplayName()
)
]
) ?>
diff --git a/app/views/module/module/modul.php b/app/views/module/module/modul.php
index 6c03e424444..b078c47b44f 100644
--- a/app/views/module/module/modul.php
+++ b/app/views/module/module/modul.php
@@ -685,7 +685,7 @@ if ($GLOBALS['MVV_MODUL']['SPRACHE']['default'] != $display_language) {
<?= Button::createAccept(_('Übernehmen'), 'store', ['title' => _('Änderungen übernehmen')]) ?>
<? endif; ?>
<? if (!$def_lang && !$deskriptor->isNew() && in_array($display_language, $translations)) : ?>
- <?= Button::create(_('Löschen'), 'delete', ['title' => _('Deskriptor löschen'), 'data-confirm' => sprintf(_('Soll der Deskriptor in der Ausgabesprache %s gelöscht werden?'), htmlReady($GLOBALS['MVV_LANGUAGES']['values'][$display_language]['name'])),
+ <?= Button::create(_('Löschen'), 'delete', ['title' => _('Deskriptor löschen'), 'data-confirm' => sprintf(_('Soll der Deskriptor in der Ausgabesprache %s gelöscht werden?'), $GLOBALS['MVV_LANGUAGES']['values'][$display_language]['name']),
'formaction' => $controller->url_for('/delete_modul_deskriptor', $deskriptor->id, $display_language)]); ?>
<? endif; ?>
<? endif; ?>
diff --git a/app/views/module/module/module.php b/app/views/module/module/module.php
index a89fbd50d92..de77f03e653 100644
--- a/app/views/module/module/module.php
+++ b/app/views/module/module/module.php
@@ -72,7 +72,7 @@
Icon::create('log', Icon::ROLE_CLICKABLE,['title' => _('Modulbeschreibung ansehen')]),
[
'data-dialog' => 'size=auto',
- 'title' => htmlReady($modul->getDisplayName())
+ 'title' => $modul->getDisplayName()
]
) ?>
<? endif; ?>
@@ -106,8 +106,8 @@
[
'formaction' => $controller->url_for('/delete/' . $modul->id),
'data-confirm' => sprintf(
- _('Wollen Sie wirklich das Modul "%s" löschen?'),
- htmlReady($modul->getDisplayName())
+ _('Wollen Sie wirklich das Modul "%s" löschen?'),
+ $modul->getDisplayName()
)
]
) ?>
diff --git a/app/views/module/module/modulteil.php b/app/views/module/module/modulteil.php
index c2c4f20a484..fb1d9306420 100644
--- a/app/views/module/module/modulteil.php
+++ b/app/views/module/module/modulteil.php
@@ -467,7 +467,7 @@ if ($GLOBALS['MVV_MODULTEIL']['SPRACHE']['default'] != $display_language) {
<?= Button::create(
_('Löschen'),
'delete',
- ['title' => _('Deskriptor löschen'), 'data-confirm' => sprintf(_('Soll der Deskriptor in der Ausgabesprache %s gelöscht werden?'), htmlReady($GLOBALS['MVV_LANGUAGES']['values'][$display_language]['name'])),
+ ['title' => _('Deskriptor löschen'), 'data-confirm' => sprintf(_('Soll der Deskriptor in der Ausgabesprache %s gelöscht werden?'), $GLOBALS['MVV_LANGUAGES']['values'][$display_language]['name']),
'formaction' => $controller->url_for('/delete_modulteil_deskriptor', $deskriptor->id, $display_language)
]
); ?>s
diff --git a/app/views/module/module/modulteil_lvg.php b/app/views/module/module/modulteil_lvg.php
index 6c6d0718280..b24601684eb 100644
--- a/app/views/module/module/modulteil_lvg.php
+++ b/app/views/module/module/modulteil_lvg.php
@@ -33,8 +33,8 @@
'formaction' => $controller->url_for('/delete_lvgruppe/' . $modulteil->id . '/' . $lvgruppe->id),
'data-confirm' => sprintf(
_('Wollen Sie wirklich die Lehrveranstaltungsgruppe "%s" vom Modulteil "%s" entfernen?'),
- htmlReady($lvgruppe->getDisplayName()),
- htmlReady($modulteil->getDisplayName())
+ $lvgruppe->getDisplayName(),
+ $modulteil->getDisplayName()
)
]
) ?>
diff --git a/app/views/my_courses/_deputy_bosses.php b/app/views/my_courses/_deputy_bosses.php
index eca6c470e8a..14b88653936 100644
--- a/app/views/my_courses/_deputy_bosses.php
+++ b/app/views/my_courses/_deputy_bosses.php
@@ -32,12 +32,12 @@
<a href="<?= URLHelper::getLink(
'dispatch.php/messages/write',
['filter' => 'send_sms_to_all', 'rec_uname' => $boss->boss_username])?>" data-dialog>
- <?= Icon::create('mail')->asImg(['title' => sprintf(_('Nachricht an %s senden'), htmlReady($boss_fullname))]) ?>
+ <?= Icon::create('mail')->asImg(['title' => sprintf(_('Nachricht an %s senden'), $boss_fullname)]) ?>
</a>
<a href="<?= $controller->link_for('my_courses/delete_boss/' . $boss->range_id) ?>"
data-confirm="<?= sprintf(_('Wollen Sie sich wirklich als Standardvertretung von %s austragen?'), htmlReady($boss_fullname)) ?>">
<?= Icon::create('trash')->asImg([
- 'title' => sprintf(_('Mich als Standardvertretung von %s austragen'), htmlReady($boss_fullname))
+ 'title' => sprintf(_('Mich als Standardvertretung von %s austragen'), $boss_fullname)
]) ?>
</a>
</td>
diff --git a/app/views/my_ilias_accounts/index.php b/app/views/my_ilias_accounts/index.php
index 6b29d5d408f..8c17c796c44 100644
--- a/app/views/my_ilias_accounts/index.php
+++ b/app/views/my_ilias_accounts/index.php
@@ -116,7 +116,7 @@
'formaction' => $controller->url_for('my_ilias_accounts/new_account/'.$ilias_index),
'data-confirm' =>
sprintf(_('Möchten Sie wirklich die bestehende Zuordnung aufheben? Sie verlieren dadurch alle mit dem bestehenden Account verbundenen Inhalte und Lernfortschritte im System "%s".'),
- htmlReady($ilias->getName())
+ $ilias->getName()
),
'data-dialog' => 'size=auto;reload-on-close'
])
@@ -129,7 +129,7 @@
'formaction' => $controller->url_for('my_ilias_accounts/change_account/'.$ilias_index.'/remove'),
'data-confirm' =>
sprintf(_('Möchten Sie wirklich die bestehende Zuordnung aufheben? Sie verlieren dadurch alle mit dem bestehenden Account verbundenen Inhalte und Lernfortschritte im System "%s".'),
- htmlReady($ilias->getName())
+ $ilias->getName()
)
])
) ?>
diff --git a/app/views/resources/admin/properties.php b/app/views/resources/admin/properties.php
index ac0b97ac924..072817b5e16 100644
--- a/app/views/resources/admin/properties.php
+++ b/app/views/resources/admin/properties.php
@@ -48,9 +48,8 @@
[
'title' => _('Löschen'),
'data-confirm' => sprintf(
- _('Soll die Eigenschaft "%s" wirklich gelöscht werden?'
- ),
- htmlReady($property->name)
+ _('Soll die Eigenschaft "%s" wirklich gelöscht werden?'),
+ $property->name
)
]
) ?>
diff --git a/app/views/search/archive/index.php b/app/views/search/archive/index.php
index 4ad5c458d01..dfca45fdb76 100644
--- a/app/views/search/archive/index.php
+++ b/app/views/search/archive/index.php
@@ -117,7 +117,7 @@
),
'data-confirm' => sprintf(
_('Soll die Veranstaltung %1$s wirklich aus dem Archiv gelöscht werden?'),
- htmlReady($course->name)
+ $course->name
),
'class' => 'text-top',
]) ?>
diff --git a/app/views/studiengaenge/informationen/degree.php b/app/views/studiengaenge/informationen/degree.php
index 2fe86a826e9..5754a2e92cc 100644
--- a/app/views/studiengaenge/informationen/degree.php
+++ b/app/views/studiengaenge/informationen/degree.php
@@ -29,9 +29,9 @@
<a href="<?= $controller->messagehelper(['abschluss_id' => $deg->abschluss_id]) ?>"
data-dialog>
<?= Icon::create('mail')->asImg(
- ['title' => htmlReady(sprintf(
+ ['title' => sprintf(
_('Alle Studierenden mit dem Studienabschluss %s benachrichtigen.'),
- $deg->name))
+ $deg->name)
]) ?>
</a>
<? endif ?>
diff --git a/app/views/studiengaenge/informationen/index.php b/app/views/studiengaenge/informationen/index.php
index ada88c7c05b..9df9da7806a 100644
--- a/app/views/studiengaenge/informationen/index.php
+++ b/app/views/studiengaenge/informationen/index.php
@@ -30,7 +30,7 @@
<?= Icon::create('mail')->asImg(
['title' => sprintf(
_('Alle Studierenden des Faches %s benachrichtigen.'),
- htmlReady($studycourse->name))
+ $studycourse->name)
]) ?>
</a>
</td>
diff --git a/app/views/studiengaenge/informationen/showdegree.php b/app/views/studiengaenge/informationen/showdegree.php
index 4dfb8b0e95c..898999502cd 100644
--- a/app/views/studiengaenge/informationen/showdegree.php
+++ b/app/views/studiengaenge/informationen/showdegree.php
@@ -22,9 +22,9 @@
Icon::create(
'mail',
Icon::ROLE_CLICKABLE,
- ['title' => htmlReady(sprintf(
+ ['title' => sprintf(
_('Nachricht an alle Studierende mit dem Studiengang %s mit dem Abschluss %s'),
- $studycourse->name, $deg->name))
+ $studycourse->name, $deg->name)
]),
['data-dialog' => ''])
->render()
diff --git a/app/views/studiengaenge/informationen/showstudycourse.php b/app/views/studiengaenge/informationen/showstudycourse.php
index 94c886f14c3..e2b640bf30c 100644
--- a/app/views/studiengaenge/informationen/showstudycourse.php
+++ b/app/views/studiengaenge/informationen/showstudycourse.php
@@ -21,9 +21,9 @@
Icon::create(
'mail',
Icon::ROLE_CLICKABLE,
- ['title' => htmlReady(sprintf(
+ ['title' => sprintf(
_('Nachricht an alle Studierende mit dem Studiengang %s mit dem Abschluss %s'),
- $course->name, $degree->name))
+ $course->name, $degree->name)
]),
['data-dialog' => ''])->render()
?>
diff --git a/app/views/studiengaenge/stgteilbezeichnungen/index.php b/app/views/studiengaenge/stgteilbezeichnungen/index.php
index f1618b02593..a8019efab5a 100644
--- a/app/views/studiengaenge/stgteilbezeichnungen/index.php
+++ b/app/views/studiengaenge/stgteilbezeichnungen/index.php
@@ -43,7 +43,7 @@
<?= Icon::create('trash', Icon::ROLE_CLICKABLE , ['title' => _('Studiengangteil-Bezeichnung löschen')])
->asInput([
'formaction' => $controller->url_for('/delete/' . $stgteilbezeichnung->id),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich die Studiengangteil-Bezeichnung "%s" löschen?'), htmlReady($stgteilbezeichnung->name)),]) ?>
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich die Studiengangteil-Bezeichnung "%s" löschen?'), $stgteilbezeichnung->name),]) ?>
<? endif; ?>
</td>
</tr>
diff --git a/app/views/studiengaenge/studiengaenge/aufbaustg_table.php b/app/views/studiengaenge/studiengaenge/aufbaustg_table.php
index 09fa8eb1aed..2d615a63e73 100644
--- a/app/views/studiengaenge/studiengaenge/aufbaustg_table.php
+++ b/app/views/studiengaenge/studiengaenge/aufbaustg_table.php
@@ -63,7 +63,7 @@
<?= Icon::create('trash')->asInput([
'formaction' => $controller->url_for('studiengaenge/studiengaenge/aufbaustg_delete', $aufbau_stg->id),
'title' => _('Zuordnung als Aufbaustudiengang löschen'),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich die Zuordnung von "%s" als "%s" löschen?'), htmlReady($aufbau_stg->getDisplayName()), htmlReady($aufbau_stg->typ))
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich die Zuordnung von "%s" als "%s" löschen?'), $aufbau_stg->getDisplayName(), $aufbau_stg->typ)
]) ?>
</a>
<? endif; ?>
diff --git a/app/views/studiengaenge/studiengaenge/studiengaenge.php b/app/views/studiengaenge/studiengaenge/studiengaenge.php
index 78a1eab4a74..6f6f7c48442 100644
--- a/app/views/studiengaenge/studiengaenge/studiengaenge.php
+++ b/app/views/studiengaenge/studiengaenge/studiengaenge.php
@@ -62,7 +62,7 @@
Icon::create('trash', Icon::ROLE_CLICKABLE ,tooltip2(_('Studiengang löschen'))),
[
'formaction' => $controller->url_for('/delete/' . $studiengang->id),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich den Studiengang "%s" löschen?'), htmlReady($studiengang->name))
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich den Studiengang "%s" löschen?'), $studiengang->name)
]
) ?>
<? endif; ?>
diff --git a/app/views/studiengaenge/studiengaenge/studiengangteile.php b/app/views/studiengaenge/studiengaenge/studiengangteile.php
index 1b055cc46bc..77072452d5c 100644
--- a/app/views/studiengaenge/studiengaenge/studiengangteile.php
+++ b/app/views/studiengaenge/studiengaenge/studiengangteile.php
@@ -24,7 +24,7 @@
'trash',
Icon::ROLE_CLICKABLE,
['title' => _('Zuordnung des Studiengangteils löschen')]
- )->asInput(['data-confirm' => htmlReady($msg)]); ?>
+ )->asInput(['data-confirm' => $msg]); ?>
</form>
<? else : ?>
<? $msg = sprintf(
@@ -39,7 +39,7 @@
'trash',
Icon::ROLE_CLICKABLE ,
['title' => _('Zuordnung des Studiengangteils löschen')]
- )->asInput(['data-confirm' => htmlReady($msg)]); ?>
+ )->asInput(['data-confirm' => $msg]); ?>
</form>
<? endif; ?>
<? endif; ?>
diff --git a/app/views/studiengaenge/studiengangteile/details_grouped.php b/app/views/studiengaenge/studiengangteile/details_grouped.php
index ab4cd519b89..79949736e49 100644
--- a/app/views/studiengaenge/studiengangteile/details_grouped.php
+++ b/app/views/studiengaenge/studiengangteile/details_grouped.php
@@ -34,7 +34,7 @@
Icon::ROLE_CLICKABLE ,
['title' => _('Studiengangteil löschen'),
'formaction' => $controller->url_for('/delete/' . $stgteil->getId()),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich den Studiengangteil "%s" löschen?'), htmlReady($stgteil->getDisplayName()))]
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich den Studiengangteil "%s" löschen?'), $stgteil->getDisplayName())]
))
?>
<? endif; ?>
diff --git a/app/views/studiengaenge/studiengangteile/index.php b/app/views/studiengaenge/studiengangteile/index.php
index 5b0a1f5edf0..98e466925de 100644
--- a/app/views/studiengaenge/studiengangteile/index.php
+++ b/app/views/studiengaenge/studiengangteile/index.php
@@ -68,7 +68,7 @@
Icon::create('trash', Icon::ROLE_CLICKABLE, [
'title' => _('Studiengangteil löschen'),
'formaction' => $controller->url_for('/delete/' . $stgteil->getId()),
- 'data-confirm' => sprintf(_('Wollen Sie wirklich den Studiengangteil "%s" löschen?'), htmlReady($stgteil->getDisplayName())),
+ 'data-confirm' => sprintf(_('Wollen Sie wirklich den Studiengangteil "%s" löschen?'), $stgteil->getDisplayName()),
])
) ?>
<? endif; ?>
diff --git a/app/views/studiengaenge/versionen/abschnitte.php b/app/views/studiengaenge/versionen/abschnitte.php
index 83b90c221b3..b22079b6a8d 100644
--- a/app/views/studiengaenge/versionen/abschnitte.php
+++ b/app/views/studiengaenge/versionen/abschnitte.php
@@ -34,7 +34,7 @@
Icon::create('trash', Icon::ROLE_CLICKABLE, tooltip2(_('Studiengangteil-Abschnitt löschen'))),
[
'formaction' => $controller->url_for('/delete_abschnitt/' . $abschnitt->id),
- 'data-confirm' => sprintf(_('Wollen Sie den Studiengangteil-Abschnitt "%s" wirklich löschen?'), htmlReady($abschnitt->getDisplayName()))
+ 'data-confirm' => sprintf(_('Wollen Sie den Studiengangteil-Abschnitt "%s" wirklich löschen?'), $abschnitt->getDisplayName())
]
) ?>
<? endif; ?>
diff --git a/app/views/studiengaenge/versionen/details_abschnitt.php b/app/views/studiengaenge/versionen/details_abschnitt.php
index da2e6f31f33..81be469471f 100644
--- a/app/views/studiengaenge/versionen/details_abschnitt.php
+++ b/app/views/studiengaenge/versionen/details_abschnitt.php
@@ -35,8 +35,8 @@
'formaction' => $controller->url_for('/delete_modul', $assignment->abschnitt_id, $assignment->modul_id),
'data-confirm' => sprintf(
_('Wollen Sie die Zuordnung des Moduls "%s" zum Studiengangteil-Abschnitt "%s" wirklich löschen?'),
- htmlReady($assignment->modul->getDisplayName()),
- htmlReady($abschnitt->getDisplayName())
+ $assignment->modul->getDisplayName(),
+ $abschnitt->getDisplayName()
)
]); ?>
<? endif; ?>
diff --git a/app/views/studiengaenge/versionen/versionen.php b/app/views/studiengaenge/versionen/versionen.php
index 2f052bda541..435880e92ca 100644
--- a/app/views/studiengaenge/versionen/versionen.php
+++ b/app/views/studiengaenge/versionen/versionen.php
@@ -30,7 +30,7 @@
$controller->url_for('/approve/' . $version->id),
_('Version genehmigen'),
Icon::create('accept', Icon::ROLE_CLICKABLE, ['title' => _('Version genehmigen')]),
- ['data-dialog' => 'title=' . htmlReady($version->getDisplayName()) . ''])
+ ['data-dialog' => 'title=\'' . $version->getDisplayName() . '\''])
?>
<? endif; ?>
<? if ($perm->haveFieldPerm('abschnitte', MvvPerm::PERM_CREATE)) : ?>
@@ -59,7 +59,7 @@
'formaction' => $controller->url_for('/copy_version/' . $version->id),
'data-confirm' => sprintf(
_('Wollen Sie wirklich die Version "%s" des Studiengangteils kopieren?'),
- htmlReady($version->getDisplayName()))
+ $version->getDisplayName())
]
))
?>
@@ -73,7 +73,7 @@
'formaction' => $controller->url_for('/delete_version/' . $version->id),
'data-confirm' => sprintf(
_('Wollen Sie wirklich die Version "%s" des Studiengangteils löschen?'),
- htmlReady($version->getDisplayName()))
+ $version->getDisplayName())
]
))
?>
diff --git a/templates/calendar/entries/icons.php b/templates/calendar/entries/icons.php
index 2e0f3d21582..16ae374e977 100644
--- a/templates/calendar/entries/icons.php
+++ b/templates/calendar/entries/icons.php
@@ -2,10 +2,10 @@
<? if (is_array($entry['icons'])) foreach ($entry['icons'] as $icon) : ?>
<? if($icon['url']) : ?>
<a href="<?= $icon['url'] ?>" <?= $icon['onClick'] ? 'onClick="STUDIP.Calendar.clickEngine('. $icon['onClick'].', this, event); return false;"' : '' ?>>
- <?= Assets::img($icon['image'], ['title' => htmlReady($icon['title']), 'alt' => htmlReady($icon['title'])]) ?>
+ <?= Assets::img($icon['image'], ['title' => $icon['title'], 'alt' => $icon['title']]) ?>
</a>
<? else : ?>
- <?= Assets::img($icon['image'], ['title' => htmlReady($icon['title'])]) ?>
+ <?= Assets::img($icon['image'], ['title' => $icon['title']]) ?>
<? endif; ?>
<? endforeach ?>
</div>
diff --git a/templates/mail/notification_html.php b/templates/mail/notification_html.php
index 77a71b627b7..5060fb7452d 100644
--- a/templates/mail/notification_html.php
+++ b/templates/mail/notification_html.php
@@ -35,7 +35,7 @@
<a href="<?= URLHelper::getLink($module['url'], ['sso' => $sso]) ?>"><?= htmlReady($module['text']) ?></a>
</td>
<td>
- <a href="<?= URLHelper::getLink($module['url'], ['sso' => $sso]) ?>"><?= $module['icon']->asImg(['title' => htmlReady($module['text'])]) ?></a>
+ <a href="<?= URLHelper::getLink($module['url'], ['sso' => $sso]) ?>"><?= $module['icon']->asImg(['title' => $module['text']]) ?></a>
</td>
</tr>
<? endforeach ?>
diff --git a/templates/tabs.php b/templates/tabs.php
index 3efab21bbfd..2a5cf0bc173 100644
--- a/templates/tabs.php
+++ b/templates/tabs.php
@@ -25,9 +25,9 @@ $ebene3 = [];
?>
<a href="<?= URLHelper::getLink($nav->getURL()) ?>" <?= arrayToHtmlAttributes($attr) ?>>
<? if ($image = $nav->getImage()) : ?>
- <?= $image->asImg(['class' => "tab-icon", 'title' => $nav->getTitle() ? htmlReady($nav->getTitle()) : htmlReady($nav->getDescription())]) ?>
+ <?= $image->asImg(['class' => "tab-icon", 'title' => $nav->getTitle() ? $nav->getTitle() : $nav->getDescription()]) ?>
<? endif ?>
- <span title="<?= $nav->getDescription() ? htmlReady($nav->getDescription()) : htmlReady($nav->getTitle())?>" class="tab-title"><?= $nav->getTitle() ? htmlReady($nav->getTitle()) : ' ' ?></span>
+ <span title="<?= $nav->getDescription() ? htmlReady($nav->getDescription()) : htmlReady($nav->getTitle())?>" class="tab-title"><?= $nav->getTitle() ? htmlReady($nav->getTitle()) : ' ' ?></span>
</a>
<? if (count($nav->getSubNavigation()) > 1): ?>
@@ -50,7 +50,7 @@ $ebene3 = [];
<? else: ?>
<span class="quiet tab-title">
<? if ($image = $nav->getImage()) : ?>
- <?= $image->asImg(['class' => "tab-icon", 'title' => htmlReady($nav->getTitle())]) ?>
+ <?= $image->asImg(['class' => "tab-icon", 'title' => $nav->getTitle()]) ?>
<? endif ?>
<?= htmlReady($nav->getTitle()) ?>
</span>
--
GitLab