http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8606

--- Comment #7 from Kyle M Hall <[email protected]> ---
(In reply to comment #4)
> QA Comments:
> No side effect possible. I don't tested the patch but it seems good.
> 
> No blocker:
> * C4/Letters.pm FAIL
>   forbidden patterns FAIL
>     The patch introduces a forbidden pattern: withespace character  (156)
> 
> But I don't understand why do you use the | operator. I think you wanted to
> use the || operator and wrote
>   $branchcode ||= '';

You are correct. The smallest followup ever written has been attached ; )

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to