[Koha-bugs] [Bug 19889] LocalHoldsPriority needs exclusions

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

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

Kyle M Hall <kyle at bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kyle at bywatersolutions.com

--- Comment #24 from Kyle M Hall <kyle at bywatersolutions.com> ---
(In reply to Christopher Brannon from comment #22)
> I'd prefer to keep these settings together, but if you think it makes more
> sense to put exclusions in the patron categories, then that would be
> acceptable.  It works.  My biggest concern is trying to debug behavior. 
> It's one thing to have the items with their own exclusions in the record. 
> But trying to figure out why it excluding items in some cases and not others
> because of a patron category exclusion might be more challenging.  If there
> were some indicator when the exclusion is skipped somewhere, then I would
> have no problem with it.  If it is logged, then it would be easy to trace.
> 
> I'm just trying to make sure that our enhancement doesn't become a headache
> for those using it to try and understand system behavior.  Circulation rules
> alone are hard to follow when dealing with multiple branches.

I think the feedback idea is both a great one, and also out of scope for this
specific bug and development. What I'm imagining is inside the hold trapping
code, every time we skip over a hold of higher priority, we add it to a list of
skipped holds. This list then needs to be passed all the way back to the
template where we can have some kind of 'reason' icon that when clicked will
show the higher priority holds that were skipped, and the reason they were
skipped. This would definitely be useful even excluding this specific
development, and is only tangential to this bug report. I have filed bug 26027
for this new functionality.

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