[Koha-bugs] [Bug 7791] Deleting basket should not delete all existing orders without warning

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

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

--- Comment #11 from mathieu saby <mathieu.saby at univ-rennes2.fr> ---
(In reply to M. de Rooy from comment #10)
> QA Comment:
> This looks quite good to me. No complaints from qa tools.
> I just have one question: I think it would be even more friendly to the user
> if you could report the errors DelBiblio returns on refused deletions
> (because of items not connected to orders).
> At this time your code does not look at a possible return value of DelBiblio.
> Would it be possible for you to add this in a followup? (But I do not think
> this should block your patch..)

Hi
I will look if it is possible and add a followup.



> 
> And I got struck by another problem when testing your patch, but it does not
> depend on your changes but is connected to booksellers.pl.
> If I delete a basket, the return page is booksellers.pl. Every time I get
> the Firefox message: A script on this page may be busy, or it may have
> stopped responding. You can stop the script now, or you can continue to see
> if the script will complete. 
> Script: http://[servername]/intranet-tmpl/lib/jquery/jquery.js:4
> Just mentioning it here for completeness.

To be clear : does it occur in master without my patch?

Mathieu

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