Jonathan Druart <> changed:

           What    |Removed                     |Added
             Status|Passed QA                   |Failed QA
                 CC|                            |jonathan.dru...@bugs.koha-c
                   |                            |

--- Comment #39 from Jonathan Druart <> 
1. In tests, prefer
instead of the DELETE then INSERT. It makes the changes more readable in my

2. What about "NULL" string values? :)

3. Should not we have the same code for 
OpacHiddenItems and ItemsDeniedRenewal (and UpdateNotForLoanStatusOnCheckin?)
There is also UpdateItemLocationOnCheckin (bug 14576)

The minimum would be a new get_rules_from_prefs sub

Marking as FQA for discussion

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

Reply via email to