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

--- Comment #88 from Jonathan Druart <[email protected]> 
---
I am quite ok with this patch, just a couple of things:

1. Who is the original author of this patch? Bernardo? Regarding the history I
do not think so.

2. Not blocker but, when a code is not nice, better refactore and improve it
than duplicate it ;)
I am referring to the JS code to test if a mandatory field is filled, it would
have been easier to make the function accepts an argument to check for
mandatories or importants.

-- 
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