26 Feb
2022
26 Feb
'22
6:40 p.m.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30178 --- Comment #10 from Janusz Kaczmarek <januszop@gmail.com> ---
We just arrived at the same spot :) I first confirmed it by adding the line to the templates:
[% item.homebranch %] | [% Branches.GetLoggedInBranchname() %]
So the fix is correct, but I feel we might want to revisit this confusing variable at some point. Changing code to name is an easy template task, so passing it like this appears not necessary.
The more that the use of GetLoggedInBranchname creates a possible security issue in a malignant case of two different branches with the same name. -- You are receiving this mail because: You are watching all bug changes.