Attachment 168005 Details for Bug 37162 – Bug 37162: Remove the [% IF ( delete_confirmed ) %] stuff in auth_tag_structure.tt

[patch] Bug 37162: Remove the [% IF ( delete_confirmed ) %] stuff in auth_tag_structure.tt

Bug-37162-Remove-the--IF--deleteconfirmed---stuff-.patch (text/plain), 3.36 KB, created by Phil Ringnalda on 2024-06-22 23:04:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Phil Ringnalda

Created: 2024-06-22 23:04:04 UTC

Size: 3.36 KB

patch

obsolete

>From e6b0cabb095dcc17d17b389ee889c2d07e0a7edb Mon Sep 17 00:00:00 2001
>From: Phil Ringnalda <phil@chetcolibrary.org>
>Date: Sat, 22 Jun 2024 15:57:01 -0700
>Subject: [PATCH] Bug 37162: Remove the [% IF ( delete_confirmed ) %] stuff in
> auth_tag_structure.tt
>
>Since auth_tag_structure.pl redirects after deleting when the op is
>delete_confirmed (which is actually cud-delete_confirmed) rather than loading
>the template, there's no reason to have dead code for IF ( delete_confirmed )
>which is both never set, and is never called when anything like it is set.
>
>Test plan:
>1. With the patch applied, Administration - Authority types - Chronological
> Term row - Actions menu - MARC Structure
>2. You've already verified that auth_tag_structure.tt isn't malformed, but for
> extra fun choose a tag, Actions menu - Delete - Yes, delete and verify that
> it was deleted
>---
> .../en/modules/admin/auth_tag_structure.tt | 19 +++++--------------
> 1 file changed, 5 insertions(+), 14 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
>index 1d1b1e83ae..41539f80ec 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
>@@ -16,9 +16,6 @@
> [% IF ( delete_confirm ) %]
> [% tx("Confirm deletion of tag '{searchfield}'", { searchfield = searchfield }) | html %] &rsaquo;
> [% END %]
>- [% IF ( delete_confirmed ) %]
>- [% t("Data deleted") | html %] &rsaquo;
>- [% END %]
> [% IF ( authtypecode ) %]
> [% tx("'{authtypecode}' framework", { authtypecode = authtypecode }) | html %]
> [% ELSE %]
>@@ -46,7 +43,7 @@
> <a href="/cgi-bin/koha/admin/authtypes.pl">Authority types</a>
> [% END %]
> 
>- [% IF ( add_form || delete_confirm || delete_confirmed ) %]
>+ [% IF ( add_form || delete_confirm ) %]
> [% IF ( authtypecode ) %]
> [% WRAPPER breadcrumb_item %]
> <a href="/cgi-bin/koha/admin/auth_tag_structure.pl?authtypecode=[% authtypecode | uri %]">[% tx("'{authtypecode}' framework", { authtypecode = authtypecode }) | html %]</a>
>@@ -84,19 +81,13 @@
> [% tx("Confirm deletion of tag '{searchfield}'", { searchfield = searchfield }) | html %]
> [% END %]
> [% ELSE %]
>- [% IF ( delete_confirmed ) %]
>+ [% IF ( authtypecode ) %]
> [% WRAPPER breadcrumb_item bc_active= 1 %]
>- Data deleted
>+ [% tx("{authtypecode} framework", { authtypecode = authtypecode }) | html %]
> [% END %]
> [% ELSE %]
>- [% IF ( authtypecode ) %]
>- [% WRAPPER breadcrumb_item bc_active= 1 %]
>- [% tx("{authtypecode} framework", { authtypecode = authtypecode }) | html %]
>- [% END %]
>- [% ELSE %]
>- [% WRAPPER breadcrumb_item bc_active= 1 %]
>- Default framework
>- [% END %]
>+ [% WRAPPER breadcrumb_item bc_active= 1 %]
>+ Default framework
> [% END %]
> [% END %]
> 
>-- 
>2.44.0

Actions: View | Diff | Splinter Review

Attachments on bug 37162: 168005 | 168034 | 168109