---- Reported by jwagner@ptfs.com 2010-03-22 17:50:22 ---- Created an attachment screen shot of OPAC shelving location search Sponsored by Arcadia University. The OPAC can search by item type or collection code, but not by shelving location. This enhancement adds a shelving location search, under More Options. Since some sites do not use shelving locations, I debated controlling it by a syspref, but decided not to. If a site wants to hide the search option, inserting this into opacuserjs will do so: $(document).ready(function(){ $("#advsearch #advsearch-shelvingloc").remove(); }); ---- Additional Comments From jwagner@ptfs.com 2010-04-01 12:51:32 ---- Created an attachment screen shot of OPAC shelving location search, revised Revised screen shot to show stylesheet formatting (matching item type search section) ---- Additional Comments From jwagner@ptfs.com 2010-04-01 13:07:13 ---- Created an attachment patch file against current head patch file sent April 1, 2010. ---- Additional Comments From jwagner@ptfs.com 2010-05-12 14:02:32 ---- The fix for this bug (in ptfs-harley) can be pulled from the public git repository at http://github.com/Koha-PTFS, branch http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4329 --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:25 UTC --- This bug was previously known as _bug_ 4329 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4329 Imported an attachment (id=2014) Imported an attachment (id=2015) Imported an attachment (id=2016) Unknown operating system Windows 2000. Setting to default OS "All". Actual time not defined. Setting to 0.0 The original reporter of this bug does not have an account here. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. Previous reporter was jwagner@ptfs.com. The original assignee of this bug does not have an account here. Reassigning to the default assignee for the component, chris@bigballofwax.co.nz. Previous assignee was jwagner@ptfs.com. Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one. Bug reassigned, setting status to "NEW". Previous status was "ASSIGNED". The original submitter of attachment 2014 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 2015 [details] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. The original submitter of attachment 2016 [details] [review] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
A basic search by shelving location seems to work fine. However, when I combine a shelving location limit with an itemtype limit the results are not as I expect. I would think it would do a search for items with my selected itemtype AND my shelving location. Instead it does OR: "mc-itype:DVD or mc-loc:DISPLAY" In my opinion this is incorrect.
I've been looking at the code, but I can't see where it decides between AND and OR. Somehow it knows to AND e.g. keyword and item types. How do I tell it to also AND shelving location search?
Created attachment 2950 [details] [review] signed off patch
Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all y Applying: Bug 4329: OPAC search by shelving location error: short SHA1 ec77493 is ambiguous. fatal: sha1 information is lacking or useless (koha-tmpl/opac-tmpl/prog/en/modules/opac-advsearch.tmpl). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 Bug 4329: OPAC search by shelving location
Hmmm, applied for me, will try again.
Chris, did you use the signed off patch? I just did it again against head and got: Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all y Applying Bug 4329: OPAC search by shelving location nengard@debian:~/kohaclone$ It applied
Yes i did the signed off one from the mailing list will try again after the meeting
This bug is mentioned in: Bug 4329: OPAC search by shelving location http://lists.koha-community.org/pipermail/koha-patches/2011-January/013518.html
Chris, do I need to try this again?
This needs a new patch because the old one edited the tmpl files.
I have submitted a more robust enhancement request (7031) that obsoletes this particular feature. Marking as a duplicate. *** This bug has been marked as a duplicate of bug 7031 ***