[Koha-bugs] [Bug 10904] would like to limit patron update request management by branch

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10904

Christopher Brannon <cbrannon at cdalibrary.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |cbrannon at cdalibrary.org

--- Comment #5 from Christopher Brannon <cbrannon at cdalibrary.org> ---
(In reply to Jonathan Druart from comment #4)
> The behavior introduced by this patch goes against the IndependentBranches
> pref value. I'm not sure it is relevant.
> 
> Or maybe should we add the 2 values on the mainpage (total changes/lib
> changes) and introduce a filter by branch on the members update page.
> 
> Don't hesitate to argue if you think I am wrong.

Jonathan,
I see what you are getting at.  While this patch adds information and
functionality that the current feature lacks, it does seem to limit outside of
the scope of IndependentBranches pref.  While our consortium would not choose
to make such a broad sweep of limits as the IndependenBrances does, it seems as
though this feature should, at some point, be controlled by the
IndependentBranches pref.

As it is now, this patch appears to conform to the same limits that have been
imposed on the suggestions counter, and in my opinion, doesn't function much
different from that other feature.  However, it would be preferable if an
enhancement were made down the road where both of these limits (and any others)
had individual prefs, and the IndependentBranches set to Enable overrode those
individual settings.

I think it is very important to maintain individual preferences, where
consortiums like ours might want some limits, but not all.  But there should
also be the ability to have a master switch, if you want it all.  Kind of like
how staff permissions are set in a tree fashion right now.

I hope that makes sense.  I still think this patch is acceptable to how Koha
currently behaves, and am keeping this signed off, based on other functions,
but I suggest another enhancement be considered and submitted to allow the
IndependentBranches pref to be a master over this feature and other similar
features with the same imposed limits.

-- 
You are receiving this mail because:
You are watching all bug changes.