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

Martin Renvoize <martin.renvo...@ptfs-europe.com> changed:

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

--- Comment #42 from Martin Renvoize <martin.renvo...@ptfs-europe.com> ---
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
Koha-bugs@lists.koha-community.org
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