[Bug 14385] New: Extend OpacHiddenItems to allow specifying exempt borrower categories
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385 Bug ID: 14385 Summary: Extend OpacHiddenItems to allow specifying exempt borrower categories Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: aleishaamohia@hotmail.com QA Contact: testopia@bugs.koha-community.org Want to be able to allow certain borrower categories to be able to see items, even if they are marked hidden by OpacHiddenItems Todo: • C4::Items->GetHiddenItemnumbers: ** It must take an optional borrowercategory ** It must check the borrowercategory against the exception list, and if there's a match, then it doesn't mark that item as hidden. ** Update the test cases to check this. • C4::Search->searchResults: ** $search_context should be modified to include a reference to the borrower details ** the places using $search_context will need to be updated to handle the new structure ** the borrower category will be passed on to GetHiddenItemnumbers ** update the test cases to account for this • opac/opac-search.pl, opac/opac-detail.pl: ** Call Members::GetMember to get info on the current user, pass the category into searchResults as the search context. • opac-MARCdetail.pl, opac/opac-detail.pl, opac/opac-ISBDdetail.pl: ** Call Members::GetMember to get info on the current user, pass the category into GetHiddenItemnumbers *** Account for no one being logged in (i.e. don't pass the category.) ** Note: opac-MARCdetail.pl will need a bit of rearrangement to where it figures out the current user. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385 Aleisha Amohia <aleishaamohia@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385 --- Comment #1 from Aleisha Amohia <aleishaamohia@hotmail.com> --- Created attachment 40158 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40158&action=edit Bug 14385: Extend OpacHiddenItems to allow specifying exempt borrower categories Changes the API for calling GetHiddenItems and all the places in the code that call it. This is to allow borrower categories to be passed in. Adds an OpacHiddenItemsExceptions syspref to allow certain borrower categories to be able to see items, even if they are marked hidden by OpacHiddenItems To test: 1) Make two borrowers, one in a category that should see everything (ie Adult), and another in a category that should only see certain things (ie Adult - exceptions) 2) Add the borrower that can see everything (the Adult) to OpacHiddenItemsExceptions 3) To the OpacHiddenItems syspref, add an item type (ensure that you have some records that fall under this type in your library). 4) Log in as the borrower that should only see certain things (Adult - exception) 5) Do a search, filtered to show records which are the item type that you specified in the OpacHiddenItems syspref. No records should show for this borrower as this item type is hidden to them. 6) Log in as the borrower that should see everything (Adult) 7) Do the same search. There should be results from this search, as this borrower category has been specified as an exception to the hidden items -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385 Zeno Tajoli <z.tajoli@cineca.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Medium patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz --- Comment #2 from Chris Cormack <chris@bigballofwax.co.nz> --- Another one I can't sign off on, could someone please test this -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org