From d07281006d833f2d0e4e40623ce8835863ea9fe8 Mon Sep 17 00:00:00 2001 From: Elmar Ludwig Date: Mon, 21 Nov 2022 09:28:21 +0000 Subject: 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/admin/user/show_user_courses.php | 4 ++-- 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/start/_widget.php | 2 +- app/views/studiengaenge/informationen/degree.php | 4 ++-- app/views/studiengaenge/informationen/index.php | 2 +- app/views/studiengaenge/informationen/showdegree.php | 4 ++-- app/views/studiengaenge/informationen/showstudycourse.php | 4 ++-- app/views/studiengaenge/stgteilbezeichnungen/index.php | 2 +- app/views/studiengaenge/studiengaenge/aufbaustg_table.php | 2 +- app/views/studiengaenge/studiengaenge/studiengaenge.php | 2 +- .../studiengaenge/studiengaenge/studiengangteile.php | 4 ++-- .../studiengaenge/studiengangteile/details_grouped.php | 4 ++-- app/views/studiengaenge/studiengangteile/index.php | 2 +- app/views/studiengaenge/versionen/abschnitte.php | 2 +- app/views/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/sidebar/clipboard-area.php | 2 +- templates/tabs.php | 6 +++--- 57 files changed, 89 insertions(+), 105 deletions(-) diff --git a/app/controllers/course/ilias_interface.php b/app/controllers/course/ilias_interface.php index a07e24a..c369e57 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 19314f0..5e7b869 100644 --- a/app/views/admin/course_wizard_steps/index.php +++ b/app/views/admin/course_wizard_steps/index.php @@ -65,7 +65,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 b4789a8..d786a97 100644 --- a/app/views/admin/ilias_interface/index.php +++ b/app/views/admin/ilias_interface/index.php @@ -78,8 +78,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 e40380c..ad4ac50 100644 --- a/app/views/admin/lti/index.php +++ b/app/views/admin/lti/index.php @@ -57,7 +57,7 @@ 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) ]) ?> diff --git a/app/views/admin/specification/index.php b/app/views/admin/specification/index.php index 38ec459..b2b42ea 100644 --- a/app/views/admin/specification/index.php +++ b/app/views/admin/specification/index.php @@ -38,7 +38,7 @@ 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' ])?> diff --git a/app/views/admin/user/show_user_courses.php b/app/views/admin/user/show_user_courses.php index 7136d9d..c8fbb1b 100644 --- a/app/views/admin/user/show_user_courses.php +++ b/app/views/admin/user/show_user_courses.php @@ -39,7 +39,7 @@ 'formaction' => $controller->delete_course_assignment($user, ['course_id' => $course->id]), 'data-confirm' => sprintf( _('Wollen Sie %s wirklich austragen?'), - htmlReady($user->getFullName()) + $user->getFullName() ), 'data-dialog' => 'size=auto' ]) ?> @@ -57,7 +57,7 @@ [ 'data-confirm' => sprintf( _('Wollen Sie %s wirklich austragen?'), - htmlReady($user->getFullName()) + $user->getFullName() ) ] ) ?> diff --git a/app/views/contact/index.php b/app/views/contact/index.php index 6ecef64..a988191 100644 --- a/app/views/contact/index.php +++ b/app/views/contact/index.php @@ -14,7 +14,7 @@ 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)]) ?> @@ -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 a3fdf2d..310bc9b 100644 --- a/app/views/course/forum/area/add.php +++ b/app/views/course/forum/area/add.php @@ -13,7 +13,7 @@ 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;', ]) ?> diff --git a/app/views/course/forum/index/_threads.php b/app/views/course/forum/index/_threads.php index f09cee2..5a93c8d 100644 --- a/app/views/course/forum/index/_threads.php +++ b/app/views/course/forum/index/_threads.php @@ -46,7 +46,7 @@ 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']), ]) ?> @@ -163,7 +163,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 b2b5e02..356a82d 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 8f841b3..7d33f20 100644 --- a/app/views/course/ilias_interface/index.php +++ b/app/views/course/ilias_interface/index.php @@ -71,7 +71,7 @@ Icon::create('trash', 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 3ca5c70..457e07e 100644 --- a/app/views/course/ilias_interface/view_object.php +++ b/app/views/course/ilias_interface/view_object.php @@ -11,7 +11,7 @@ isActive() && !$mode) : ?> isAllowed('delete')) : ?> - 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()))]) ?> + 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())]) ?> isAllowed('start') ? Studip\LinkButton::create(_('Starten'), $controller->url_for($module->getRoute('start')), ['target' => '_blank', 'rel' => 'noopener noreferrer']) :'' ?> 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 3b2fb80..e04e3bf 100644 --- a/app/views/course/lti/index.php +++ b/app/views/course/lti/index.php @@ -37,7 +37,7 @@ 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) ]) ?> diff --git a/app/views/course/members/accepted_list.php b/app/views/course/members/accepted_list.php index c484e3c..7b34ef3 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 a4a508f..4e90ab1 100644 --- a/app/views/course/members/add_dozent.php +++ b/app/views/course/members/add_dozent.php @@ -27,7 +27,7 @@ sprintf(_("als %s eintragen"), htmlReady($decoratedStatusGroups['dozent']))] + ['title' => sprintf(_("als %s eintragen"), $decoratedStatusGroups['dozent'])] ) ?> url_for('course/members/index')) ?> diff --git a/app/views/course/members/add_member.php b/app/views/course/members/add_member.php index 8d597e4..2e258b4 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']) ] ) ?> diff --git a/app/views/course/members/add_tutor.php b/app/views/course/members/add_tutor.php index 95e9abc..36cda6f 100644 --- a/app/views/course/members/add_tutor.php +++ b/app/views/course/members/add_tutor.php @@ -24,7 +24,7 @@ sprintf(_('als %s eintragen'), htmlReady($decoratedStatusGroups['tutor']))] + ['title' => sprintf(_('als %s eintragen'), $decoratedStatusGroups['tutor'])] ) ?> url_for('course/members/index')) ?> diff --git a/app/views/course/members/autor_list.php b/app/views/course/members/autor_list.php index 180d090..0464d47 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)]) ) ?> render() ?> diff --git a/app/views/course/members/awaiting_list.php b/app/views/course/members/awaiting_list.php index 061ddaf..b8c1e08 100644 --- a/app/views/course/members/awaiting_list.php +++ b/app/views/course/members/awaiting_list.php @@ -82,7 +82,7 @@ = $last_visitdate) echo 'class="new-member"'; ?>> getImageTag(Avatar::SMALL, [ 'style' => 'margin-right: 5px', - 'title' => htmlReady($fullname), + 'title' => $fullname, ]) ?> @@ -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)] ) ) ?> diff --git a/app/views/course/members/dozent_list.php b/app/views/course/members/dozent_list.php index ab5ae0a..9b11d07 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)] ) ) ?> diff --git a/app/views/course/members/import_autorlist.php b/app/views/course/members/import_autorlist.php index 48a73eb..450270c 100644 --- a/app/views/course/members/import_autorlist.php +++ b/app/views/course/members/import_autorlist.php @@ -40,7 +40,7 @@ diff --git a/app/views/course/members/tutor_list.php b/app/views/course/members/tutor_list.php index cb0405c..6af208c 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)] ) ) ?> diff --git a/app/views/course/members/user_list.php b/app/views/course/members/user_list.php index fafef93..9b2acea 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)] ) ) ?> diff --git a/app/views/course/statusgroups/_group.php b/app/views/course/statusgroups/_group.php index aa0ebb2..d84b456 100644 --- a/app/views/course/statusgroups/_group.php +++ b/app/views/course/statusgroups/_group.php @@ -29,7 +29,7 @@ id != 'nogroup' && $group->isMember($GLOBALS['user']->id)) : ?> sprintf( _('Nachricht an alle Mitglieder der Gruppe %s schicken'), @@ -44,7 +44,7 @@ sprintf(_('Mitglied von Gruppe %s werden'), - htmlReady($group->name))]) ?> + $group->name)]) ?> id != 'nogroup' && $group->selfassign && $group->selfassign_start > time()) : ?> url_for('course/statusgroups/leave', $group->id) ?>" data-confirm="name)) . '?' ?>"> sprintf(_('Aus Gruppe %s austragen'), - htmlReady($group->name))]) ?> + $group->name)]) ?> @@ -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 @@ asImg(['title' => _('Nachricht an alle nicht zugeordneten Personen schicken')]) ?> diff --git a/app/views/course/statusgroups/_member.php b/app/views/course/statusgroups/_member.php index 7c39a77..cbe78dd 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 8f05d78..d5a26b3 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' => ''] ) ?> - 'image', - 'class' => 'middle', - 'name' => 'delete_single_date', - 'data-confirm' => _('Wollen Sie diesen Termin wirklich löschen / ausfallen lassen?') - . '
' . implode('
', array_map('htmlReady', $termin->getDeletionWarnings())), - 'formaction' => $controller->url_for( - 'course/timesrooms/deleteSingle/' . $termin->id, - ['cycle_id' => $termin->metadate_id] + $linkAttributes - ), - ] ?> - - - - addLink( $controller->url_for( 'course/timesrooms/stack', diff --git a/app/views/fachabschluss/abschluesse/index.php b/app/views/fachabschluss/abschluesse/index.php index 99422ab..6750a76 100644 --- a/app/views/fachabschluss/abschluesse/index.php +++ b/app/views/fachabschluss/abschluesse/index.php @@ -38,7 +38,7 @@ 'formaction' => $controller->action_url('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 80cdfe3..352dcb7 100644 --- a/app/views/fachabschluss/faecher/index.php +++ b/app/views/fachabschluss/faecher/index.php @@ -35,7 +35,7 @@ asInput( [ 'formaction' => $controller->action_url('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' ]); ?> diff --git a/app/views/fachabschluss/kategorien/index.php b/app/views/fachabschluss/kategorien/index.php index 7cfa6a5..44c9bad 100644 --- a/app/views/fachabschluss/kategorien/index.php +++ b/app/views/fachabschluss/kategorien/index.php @@ -48,7 +48,7 @@ asInput( [ 'formaction' => $controller->action_url('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' ]); ?> diff --git a/app/views/lvgruppen/lvgruppen/index.php b/app/views/lvgruppen/lvgruppen/index.php index 984cac8..d5ab6a8 100644 --- a/app/views/lvgruppen/lvgruppen/index.php +++ b/app/views/lvgruppen/lvgruppen/index.php @@ -53,7 +53,7 @@ 'formaction' => $controller->action_url('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 fed9e53..694d8af 100644 --- a/app/views/module/module/details.php +++ b/app/views/module/module/details.php @@ -67,7 +67,7 @@ 'formaction' => $controller->action_url('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 2fa0cdd..a6d534f 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) { _('Änderungen übernehmen')]) ?> isNew() && in_array($display_language, $translations)) : ?> - _('Deskriptor löschen'), 'data-confirm' => sprintf(_('Soll der Deskriptor in der Ausgabesprache %s gelöscht werden?'), htmlReady($GLOBALS['MVV_LANGUAGES']['values'][$display_language]['name'])), + _('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->action_url('delete_modul_deskriptor', $deskriptor->id, $display_language)]); ?> diff --git a/app/views/module/module/module.php b/app/views/module/module/module.php index 7700a3f..09cd3f3 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() ] ) ?> @@ -106,8 +106,8 @@ [ 'formaction' => $controller->action_url('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 cf6452d..d7e75f4 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) { _('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->action_url('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 3bb5014..945fcff 100644 --- a/app/views/module/module/modulteil_lvg.php +++ b/app/views/module/module/modulteil_lvg.php @@ -33,8 +33,8 @@ 'formaction' => $controller->action_url('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 eca6c47..14b8865 100644 --- a/app/views/my_courses/_deputy_bosses.php +++ b/app/views/my_courses/_deputy_bosses.php @@ -32,12 +32,12 @@ - asImg(['title' => sprintf(_('Nachricht an %s senden'), htmlReady($boss_fullname))]) ?> + asImg(['title' => sprintf(_('Nachricht an %s senden'), $boss_fullname)]) ?> asImg([ - 'title' => sprintf(_('Mich als Standardvertretung von %s austragen'), htmlReady($boss_fullname)) + 'title' => sprintf(_('Mich als Standardvertretung von %s austragen'), $boss_fullname) ]) ?> diff --git a/app/views/my_ilias_accounts/index.php b/app/views/my_ilias_accounts/index.php index f263baa..d4042bb 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 ac0b97a..072817b 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 4ad5c45..dfca45f 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/start/_widget.php b/app/views/start/_widget.php index f343a89..65747eb 100644 --- a/app/views/start/_widget.php +++ b/app/views/start/_widget.php @@ -28,7 +28,7 @@ - htmlReady($admin_title)]) ?> + $admin_title]) ?> diff --git a/app/views/studiengaenge/informationen/degree.php b/app/views/studiengaenge/informationen/degree.php index 60d913f..be9a69f 100644 --- a/app/views/studiengaenge/informationen/degree.php +++ b/app/views/studiengaenge/informationen/degree.php @@ -35,9 +35,9 @@ asImg( - ['title' => htmlReady(sprintf( + ['title' => sprintf( _('Alle Studierenden mit dem Studienabschluss %s benachrichtigen.'), - $deg->name)) + $deg->name) ]) ?> diff --git a/app/views/studiengaenge/informationen/index.php b/app/views/studiengaenge/informationen/index.php index 2b3ab5d..fcdbc3b 100644 --- a/app/views/studiengaenge/informationen/index.php +++ b/app/views/studiengaenge/informationen/index.php @@ -36,7 +36,7 @@ asImg( ['title' => sprintf( _('Alle Studierenden des Faches %s benachrichtigen.'), - htmlReady($studycourse->name)) + $studycourse->name) ]) ?> diff --git a/app/views/studiengaenge/informationen/showdegree.php b/app/views/studiengaenge/informationen/showdegree.php index 1a01603..82aa007 100644 --- a/app/views/studiengaenge/informationen/showdegree.php +++ b/app/views/studiengaenge/informationen/showdegree.php @@ -28,9 +28,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 3a3f733..7cd7ced 100644 --- a/app/views/studiengaenge/informationen/showstudycourse.php +++ b/app/views/studiengaenge/informationen/showstudycourse.php @@ -28,9 +28,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 31da64b..96bf7d0 100644 --- a/app/views/studiengaenge/stgteilbezeichnungen/index.php +++ b/app/views/studiengaenge/stgteilbezeichnungen/index.php @@ -51,7 +51,7 @@ _('Studiengangteil-Bezeichnung löschen')]) ->asInput([ 'formaction' => $controller->action_url('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)]) ?> diff --git a/app/views/studiengaenge/studiengaenge/aufbaustg_table.php b/app/views/studiengaenge/studiengaenge/aufbaustg_table.php index d84265c..039d464 100644 --- a/app/views/studiengaenge/studiengaenge/aufbaustg_table.php +++ b/app/views/studiengaenge/studiengaenge/aufbaustg_table.php @@ -69,7 +69,7 @@ $perm = MvvPerm::get($grund_stg); 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) ]) ?> diff --git a/app/views/studiengaenge/studiengaenge/studiengaenge.php b/app/views/studiengaenge/studiengaenge/studiengaenge.php index ad677ed..6d2ba90 100644 --- a/app/views/studiengaenge/studiengaenge/studiengaenge.php +++ b/app/views/studiengaenge/studiengaenge/studiengaenge.php @@ -76,7 +76,7 @@ Icon::create('trash', Icon::ROLE_CLICKABLE ,tooltip2(_('Studiengang löschen'))), [ 'formaction' => $controller->action_url('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) ] ) ?> diff --git a/app/views/studiengaenge/studiengaenge/studiengangteile.php b/app/views/studiengaenge/studiengaenge/studiengangteile.php index 5815b2f..7c3cf92 100644 --- a/app/views/studiengaenge/studiengaenge/studiengangteile.php +++ b/app/views/studiengaenge/studiengaenge/studiengangteile.php @@ -35,7 +35,7 @@ 'trash', Icon::ROLE_CLICKABLE, ['title' => _('Zuordnung des Studiengangteils löschen')] - )->asInput(['data-confirm' => htmlReady($msg)]); ?> + )->asInput(['data-confirm' => $msg]); ?> _('Zuordnung des Studiengangteils löschen')] - )->asInput(['data-confirm' => htmlReady($msg)]); ?> + )->asInput(['data-confirm' => $msg]); ?> diff --git a/app/views/studiengaenge/studiengangteile/details_grouped.php b/app/views/studiengaenge/studiengangteile/details_grouped.php index 28345c1..6f10bd0 100644 --- a/app/views/studiengaenge/studiengangteile/details_grouped.php +++ b/app/views/studiengaenge/studiengangteile/details_grouped.php @@ -13,7 +13,7 @@ - asImg(['title' => htmlReady($ampelstatus), 'style' => 'vertical-align: text-top;']) ?> + asImg(['title' => $ampelstatus, 'style' => 'vertical-align: text-top;']) ?> getDisplayName()) ?> @@ -42,7 +42,7 @@ Icon::ROLE_CLICKABLE , ['title' => _('Studiengangteil löschen'), 'formaction' => $controller->action_url('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())] )) ?> diff --git a/app/views/studiengaenge/studiengangteile/index.php b/app/views/studiengaenge/studiengangteile/index.php index 8c50c4c..353b2c6 100644 --- a/app/views/studiengaenge/studiengangteile/index.php +++ b/app/views/studiengaenge/studiengangteile/index.php @@ -77,7 +77,7 @@ Icon::create('trash', Icon::ROLE_CLICKABLE, [ 'title' => _('Studiengangteil löschen'), 'formaction' => $controller->action_url('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()), ]) ) ?> diff --git a/app/views/studiengaenge/versionen/abschnitte.php b/app/views/studiengaenge/versionen/abschnitte.php index bad6673..1620450 100644 --- a/app/views/studiengaenge/versionen/abschnitte.php +++ b/app/views/studiengaenge/versionen/abschnitte.php @@ -44,7 +44,7 @@ Icon::create('trash', Icon::ROLE_CLICKABLE, tooltip2(_('Studiengangteil-Abschnitt löschen'))), [ 'formaction' => $controller->action_url('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()) ] ) ?> diff --git a/app/views/studiengaenge/versionen/details_abschnitt.php b/app/views/studiengaenge/versionen/details_abschnitt.php index 843316c..3b1c0e7 100644 --- a/app/views/studiengaenge/versionen/details_abschnitt.php +++ b/app/views/studiengaenge/versionen/details_abschnitt.php @@ -46,8 +46,8 @@ 'formaction' => $controller->action_url('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() ) ]); ?> diff --git a/app/views/studiengaenge/versionen/versionen.php b/app/views/studiengaenge/versionen/versionen.php index f4ad9af..aff5d5f 100644 --- a/app/views/studiengaenge/versionen/versionen.php +++ b/app/views/studiengaenge/versionen/versionen.php @@ -30,7 +30,7 @@ $controller->action_url('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() . '\'']) ?> haveFieldPerm('abschnitte', MvvPerm::PERM_CREATE)) : ?> @@ -59,7 +59,7 @@ 'formaction' => $controller->action_url('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->action_url('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 c9fd74b..9829d41 100644 --- a/templates/calendar/entries/icons.php +++ b/templates/calendar/entries/icons.php @@ -2,10 +2,10 @@ > - htmlReady($icon['title']), 'alt' => htmlReady($icon['title'])]) ?> + $icon['title'], 'alt' => $icon['title']]) ?> - htmlReady($icon['title'])]) ?> + $icon['title']]) ?> diff --git a/templates/mail/notification_html.php b/templates/mail/notification_html.php index 77a71b6..5060fb7 100644 --- a/templates/mail/notification_html.php +++ b/templates/mail/notification_html.php @@ -35,7 +35,7 @@ - asImg(['title' => htmlReady($module['text'])]) ?> + asImg(['title' => $module['text']]) ?> diff --git a/templates/sidebar/clipboard-area.php b/templates/sidebar/clipboard-area.php index 4a4adf5..7905ea2 100644 --- a/templates/sidebar/clipboard-area.php +++ b/templates/sidebar/clipboard-area.php @@ -83,7 +83,7 @@ asInput( [ - 'title' => sprintf(_('%s löschen.'), htmlReady($item['name'])), + 'title' => sprintf(_('%s löschen.'), $item['name']), 'data-confirm-message' => _('Sind Sie sicher?'), 'class' => 'text-bottom clipboard-item-remove-button' ] diff --git a/templates/tabs.php b/templates/tabs.php index 3efab21..2a5cf0b 100644 --- a/templates/tabs.php +++ b/templates/tabs.php @@ -25,9 +25,9 @@ $ebene3 = []; ?> > getImage()) : ?> - asImg(['class' => "tab-icon", 'title' => $nav->getTitle() ? htmlReady($nav->getTitle()) : htmlReady($nav->getDescription())]) ?> + asImg(['class' => "tab-icon", 'title' => $nav->getTitle() ? $nav->getTitle() : $nav->getDescription()]) ?> - getTitle() ? htmlReady($nav->getTitle()) : ' ' ?> + getTitle() ? htmlReady($nav->getTitle()) : ' ' ?> getSubNavigation()) > 1): ?> @@ -50,7 +50,7 @@ $ebene3 = []; getImage()) : ?> - asImg(['class' => "tab-icon", 'title' => htmlReady($nav->getTitle())]) ?> + asImg(['class' => "tab-icon", 'title' => $nav->getTitle()]) ?> getTitle()) ?> -- cgit v1.0