Browse Source

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.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
23.05.x
Owen Leonard 11 months ago
committed by Tomas Cohen Arazi
parent
commit
bf05e0898b
Signed by: tomascohen GPG Key ID: 0A272EA1B2F3C15F
  1. 8
      koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
  2. 6
      koha-tmpl/intranet-tmpl/prog/en/modules/admin/cities.tt

8
koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc

@ -412,7 +412,7 @@
[ WRAPPER breadcrumb_item ]
<a href="/cgi-bin/koha/module/page.pl"> Module page </a>
[ END ]
[ WRAPPER breadcrumb_item active = 1 ]
[ WRAPPER breadcrumb_item bc_active = 1 ]
Add form
[ END ]
@ -420,12 +420,12 @@
[ WRAPPER breadcrumb_item ]
<a href="/cgi-bin/koha/module/page.pl"> Module page </a>
[ END ]
[ WRAPPER breadcrumb_item active = 1 ]
[ WRAPPER breadcrumb_item bc_active = 1 ]
Confirm deletion
[ END ]
[ ELSE ]
[ WRAPPER breadcrumb_item active = 1 ]
[ WRAPPER breadcrumb_item bc_active = 1 ]
Module page
[ END ]
[ END #/IF op = add_form ]
@ -448,7 +448,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 %]

6
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 %]

Loading…
Cancel
Save