[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

Christopher Brannon <cbrannon at cdalibrary.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |In Discussion

--- Comment #19 from Christopher Brannon <cbrannon at cdalibrary.org> ---
Test passes, and everything works according to the test.  Let's discuss the
following issues before I sign off, since we are funding this project:

* When you update priority in the item (yes/no), there is nothing that
indicates that it is done or set.  It would be helpful if there was something
that appeared to indicate that it took.  Maybe (SET/UNSET) next to the dropbox?

* The patron exclusion seems to be in an uncommon place.  Technically, it makes
sense to have this in the patron category, but usually these kinds of
conditions are setup in the preferences.pl.  I state this, because if we do
some forward thinking here, if we want to do other exclusions in the future, it
might be better to set this as a textbox pref, where you would set this
exclusion with a patroncategory: PT, and then in the future we could exclude by
itemtype, collectioncode, or other ways.  It just makes sense to keep the
exclusions all in one place rather than spread out everywhere.  And if you do
this, could biblio or barcode/itemnumber be set here as well, rather than in
the item record itself?

* A couple of the criteria we had asked for in the exclusions was item types
and formats.  Those were not addressed in these patches.  I would be willing to
forget about formats at this point, but would like to see item types.  I am
kicking myself for not including biblio in possible exclusions.  As is, we have
to exclude individual items.  If that can be added easily, great.  If not, at
least the item types needs to be, since it was in the original request.

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