---- Reported by nengard@gmail.com 2008-02-25 16:03:14 ---- When in an academic setting for example the librarians may want to know that a professor is requesting a book for a class. Or they may want to know that someone found a damaged book and a new one should be ordered. Since there is a list why librarians approve/reject requests, it would also be nice if patrons had the ability to explain why they were requesting a book. ---- Additional Comments From nengard@gmail.com 2008-02-25 16:03:46 ---- Created an attachment originally i thought that SUGGEST was for the user - and I came up with this list ---- Additional Comments From nengard@gmail.com 2008-03-06 05:00:14 ---- Created an attachment 1 - approved suggestion ---- Additional Comments From nengard@gmail.com 2008-03-06 05:00:50 ---- Created an attachment 2 - order from vendor ---- Additional Comments From nengard@gmail.com 2008-03-06 05:01:13 ---- Created an attachment 3 - order from suggestion --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 00:35 UTC --- This bug was previously known as _bug_ 1883 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=1883 Imported an attachment (id=400) Imported an attachment (id=401) Imported an attachment (id=402) Imported an attachment (id=403) Actual time not defined. Setting to 0.0 CC member bchurch@ptfs.com does not have an account here The original submitter of attachment 400 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 401 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 402 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 403 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
Created attachment 2809 [details] [review] Proposed fix Fix for Bug 1883, authorised list of reasons why patron requesting book - Adding form controls to opac submission form and staf client edit form. - Adding display of patron reason for suggestion on opac list - Adding function for pulling an authorized value description using the category and value TODO: Add some default authorised values
Note for testers: my proposed fix uses an authorized value with the code "OPAC_SUG."
Created attachment 2828 [details] [review] Proposed fix ( revised) Part of proposed fix is already in master from a previously applied patch (changes in C4/Koha.pm) Removed the duplication so patch applies cleanly on current master
Pushed, please test
This works, but the auth val category 'OPAC_SUG' doesn't get created on upgrade and there is no hint anywhere that this authorized value is available. I'll add it to the manual, but we should also add it on upgrade or include some sort of sample data for it.
This bug is mentioned in: Fix for Bug 1883, authorised list of reasons why patron requesting book http://lists.koha-community.org/pipermail/koha-patches/2010-December/013177.html Fix for Bug 1883, authorised list of reasons why patron requesting book http://lists.koha-community.org/pipermail/koha-patches/2010-December/013227.html
I can submit a patch which would add a couple of default suggestion reasons via updatedatabase.pl: INSERT INTO authorised_values (category,authorised_value,lib,lib_opac) VALUES ('OPAC_SUG','damaged','The copy on the shelf is damaged','The copy on the shelf is damaged INSERT INTO authorised_values (category,authorised_value,lib,lib_opac) VALUES ('OPAC_SUG','bestseller','Upcoming title by popular author','Upcoming title by popular author Anyone have objections or suggestions?
That would be great!
Created attachment 3088 [details] [review] Follow-up fix Adds two default authorized values in the category OPAC_SUG: damaged, "The copy on the shelf is damaged" bestseller, "Upcoming title by popular author"
(In reply to comment #9) > Created attachment 3088 [details] [review] > Follow-up fix > > Adds two default authorized values in the category OPAC_SUG: > > damaged, "The copy on the shelf is damaged" > bestseller, "Upcoming title by popular author" This patch added these types only on upgrade. We need them to be added to new installs to.
Created attachment 3195 [details] [review] solve comment 10
Patch pushed, please test
This bug is mentioned in: Bug 1883 follow-up: Adding default OPAC_SUG values http://lists.koha-community.org/pipermail/koha-patches/2011-February/013777.html