Attachment 145070 Details for Bug 32507 – Bug 32507: (follow-up) Change parameter name to avoid collisions

[patch] Bug 32507: (follow-up) Change parameter name to avoid collisions

Bug-32507-follow-up-Change-parameter-name-to-avoid.patch (text/plain), 2.30 KB, created by Owen Leonard on 2023-01-05 18:51:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2023-01-05 18:51:56 UTC

Size: 2.30 KB

patch

obsolete

>From b0163256af42e3da24bd3ccb27e3f44803f22e98 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 5 Jan 2023 18:44:47 +0000
>Subject: [PATCH] Bug 32507: (follow-up) Change parameter name to avoid
> collisions
>
>The variable name "active" is very likely to already be set for other
>reasons, so changing the parameter name to "bc_active" to avoid
>problems.
>---
> koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc | 2 +-
> koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt   | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
>index 1d974c8bfa..aa17fd7b90 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
>@@ -357,7 +357,7 @@
> 
> [%# Used to build individual breadcrumb items in the breadcrumb nav %]
> [% BLOCK breadcrumb_item %]
>-    [% IF ( active ) %]
>+    [% IF ( bc_active ) %]
>         <li>
>             <a href="#" aria-current="page">
>                 [% content | $raw %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt
>index b71bcd2f72..98235c8215 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt
>@@ -33,7 +33,7 @@
>             [% WRAPPER breadcrumb_item %]
>                 <a href="/cgi-bin/koha/admin/cities.pl">Cities</a>
>             [% END %]
>-            [% WRAPPER breadcrumb_item active = 1 %]
>+            [% WRAPPER breadcrumb_item bc_active = 1 %]
>                 [% IF city.cityid %]
>                     <span>Modify city</span>
>                 [% ELSE %]
>@@ -45,12 +45,12 @@
>             [% WRAPPER breadcrumb_item %]
>                 <a href="/cgi-bin/koha/admin/cities.pl">Cities</a>
>             [% END %]
>-            [% WRAPPER breadcrumb_item active = 1 %]
>+            [% WRAPPER breadcrumb_item bc_active = 1 %]
>                 Confirm deletion of city
>             [% END %]
> 
>         [% ELSE %]
>-            [% WRAPPER breadcrumb_item active = 1 %]
>+            [% WRAPPER breadcrumb_item bc_active = 1 %]
>                 Cities
>             [% END %]
>         [% END #/IF op = add_form %]
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 32507: 144742 | 144754 | 145070 | 145147 | 145148 | 146400 | 146401