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

Martin Renvoize <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Passed QA

--- Comment #42 from Martin Renvoize <[email protected]> ---
This works well, tests pass and QA script is happy.

Minor wonderment, should the permissions check be moved up in the the module
level rather than being in the controllers.. so we can have it unit tested
consistently?

Otherwise, this is all good and gets my PQA stamp, great work Marcel

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