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

Kyle M Hall <k...@bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |k...@bywatersolutions.com

--- Comment #4 from Kyle M Hall <k...@bywatersolutions.com> ---
(In reply to Jonathan Druart from comment #3)
> The 'cancel hold' checkbox is not taken into account (but it is already the
> case for the "No, don't check out" button).
> The redirect to returns.pl is a little bit unexpected, in my opinion.
> 
> I need other qa points of view for this one.

I agree, the behavior of this patch is very odd. I don't think the redirect is
a very good idea. It feels very much like a hack especially since the user is
not told this is what will happen. I would expect it to remain on the
circulation screen and open a hold slip popup the same way the "print and
confirm" button does on returns.pl. I see no reason why it could not be
implemented that way.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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