Katrin Fischer <> changed:

           What    |Removed                     |Added
             Status|In Discussion               |Passed QA

--- Comment #9 from Katrin Fischer <> ---
(In reply to Jonathan Druart from comment #8)
> I need a confirmation here.
> The test plan says "Set CheckPrevCheckout to Do, or Do, with confirmation",
> but it's "unless overriden, do". That means we should check the patron
> category and patron's settings as well.
> Finally it sounds like this flag is set when the error/warning must be
> displayed, and maybe the following code would be enough:
> } elsif ($confirmation eq 'PREVISSUE') {
>     $self->screen_msg("This item was previously checked out by you");
>     last;
> }
> Could someone confirm or tell me what I am missing?

There is no setting on patron category here - so this is alright. The pref has
4 values that seem a little confusing.

I feel the patch is correct.

You are receiving this mail because:
You are watching all bug changes.
Koha-bugs mailing list
website :
git :
bugs :

Reply via email to