https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40982

--- Comment #4 from Jonathan Druart <[email protected]> ---
(In reply to Slava Shishkin from comment #3)
> (In reply to Jonathan Druart from comment #2)
> > Hum I thought we fixed this problem with bug 40753.
> > 
> > I will have a look at your patch later.
> 
> Bug 40753 didn’t cover this case. It didn’t touch acqui/basket.tt nor the
> last two TT-conditional columns of the Orders table. 

Have a look at the patch, it is exactly the code it touches ;)

https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186239

> DataTables caches .visible in localStorage (DataTables_acqui_basket_orders)
> and reuses it across baskets; when switching open ↔ closed, the cached flags
> override current TT conditions, so Modify/Cancel order end up missing or
> shown incorrectly.
> 
> This patch synchronizes only columns [21] and [22] in the cached state on
> load to match TT logic (active && !closedate and !closedate ||
> CancelOrdersInClosedBaskets). 

Yes, and IIRC it's exactly what bug 40753 tried to fix, or at least something
similar.

I have spotted a couple of problemd with the code there, I will attach
something here.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://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