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

David Cook <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #2 from David Cook <[email protected]> ---
This could actually be a good opportunity to do some refactoring...

If you go to
https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=26791&attachment=113660
and click "Auth.pm", you'll see a function called "get_authz_from_flags".

That could be further optimized by fetching the permissions from userflags (as
you note) to dynamically make sure that the superlibrarian has all the right
permissions.

By the way, if we refactor it along the lines of what I've done on Bug 26791,
we could add unit tests too. 

Originally I wanted to refactor C4::Auth::get_template_and_user, but I figured
there was no hope doing it alone. But if you're interested in proving it, I'm
happy to work with you.

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