[Koha-bugs] [Bug 14480] Warns when modifying patron

[bugzilla-daemon at bugs.koha-community.org]

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

M. Tompsett <mtompset at hotmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mtompset at hotmail.com

--- Comment #12 from M. Tompsett <mtompset at hotmail.com> ---
(In reply to Jonathan Druart from comment #11)
> Anyway, what I have proposed previously was the wrong test indeed, it should
> be
>   [% UNLESS debarments %]

Or clearer, swap the IF/ELSE.
You know you are generating the table [% IF (debarments) %], [% ELSE %] you are
putting out that other message.

The advantage of Jonathan's suggestion is smaller patch size. The disadvantage
is that the novice programmer does not usually think in negation logic.

-- 
You are receiving this mail because:
You are watching all bug changes.