From screen ../cgi-bin/koha/catalogue/detail.pl?biblionumber= Different lists(tabs) for my holdings(items with branch connexion) and other items(item with branch different of branch connexion). Add filter at the top right of the item list. Filter that points to the enumchron 4 / Add a column "Selection" check at the table of results 5/At the bottom add two buttons: "Remove items Selected" and "Modify items Selected" Buttons that copy / paste barcodes on the screens: batchMod.pl or batchMod.pl? Del = 1 6/Buttons activating a pop up screen avoiding leaving detail.pl From items page, reach in 1 click the "batch del item" (usefull when you trash a year of a daily serial) Notes : new permissions associated with these functions see ticket 7673
Notes : the two tabs My Holdings et others Holdings must also be activated in the OPAC : ..//cgi-bin/koha/opac-detail.pl
For reference, this bug is linked to biblibre's tickets 4656 and 4655
Created attachment 12416 [details] [review] Bug 7674: Separate items in two tabs (opac/opac-detail.pl) Two tabs: "My holdings" / "Other holdings" "My holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user, or not logged in), the display is unchanged (1 tab "Holdings")
Created attachment 12417 [details] [review] Bug 7674: Separate items in two tabs (catalogue/detail.pl) Two tabs: "My holdings" / "Other holdings" "My holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user), the display is unchanged (1 tab "Holdings")
Created attachment 12418 [details] [review] Bug 7674: Add menu entry to delete items in batch In catalogue/detail.pl you can now select "Delete items in batch" in "Edit" menu. You are redirected to batchMod.pl for chosing which items have to be deleted. You need to have 'tools.items_batchdel' permission to see this menu entry.
This sounds like it should be two bug reports: One for the separate holdings tabs, one for the batch operations. The one for the separate holdings tabs needs to be controlled by a system preference, one for the OPAC and one for the staff client. I would suggest that instead of "My holdings" the tab use the library name: "Athens holdings."
New bug for the batch operation: Bug 8801
Created attachment 12433 [details] [review] Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Two tabs: "<branch> holdings" / "Other holdings" "<branch> holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user or not logged in (opac)), the display is unchanged (1 tab "Holdings") -- + Added two sysprefs SeparateHoldings and OpacSeparateHoldings (YesNo) + changed "My holdings" to "<branchname> holdings" Previous patches squashed into this one.
Thanks for the improvements. I see another issue: Clicking a "browse shelf" link in the "other holdings" tab opens the shelf browser in the "<branch> holdings" tab. It would be easy enough to do something like this: [% PROCESS items_table items=otheritemloop tab="otherholdings" %] ...and modify the link to the shelf browser: <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ITEM_RESULT.biblionumber %]&shelfbrowse_itemnumber=[% ITEM_RESULT.itemnumber %]#[% tab %]">Browse shelf</a> ...but I'm not sure how you get the shelf browser to appear under the correct tab.
i have this message when i want to test in a sandbox omething went wrong ! Applying: Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Using index info to reconstruct a base tree... Falling back to patching base and 3-way merge... Auto-merging installer/data/mysql/sysprefs.sql CONFLICT (content): Merge conflict in installer/data/mysql/sysprefs.sql Auto-merging installer/data/mysql/updatedatabase.pl Failed to merge in the changes. Patch failed at 0001 Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort". Bug 7674 - Separate tabs for my holdings and others holdings Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Apply? [yn] Patch left in /tmp/Bug-7674-Separate-items-in-two-tabs-on-biblio-deta-iPrfAr.patch
Marking as Failed QA for comment 9. Owen, how can I get this "browse shelf" link appearing ? When I made this patch I didn't notice it.
Hi Julian, there are system preferences to control the shelf browser and your items need to have callnumbers. I hope there is no UNIMARC/MARC21 problem involved here. Hope that helps!
Created attachment 12749 [details] [review] Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) rebased on master
Created attachment 12750 [details] [review] Bug 7674: Open shelf browser in appropriate tab If you click on "Browse shelf" under "<mylibrary> holdings", shelf browser should open under this tab. If you click on "Browse shelf" under "other holdings" tab, it should open under "other holdings" tab.
Thanks Katrin, in fact the link didn't show up because my items have not a callnumber. Patch should fix the problem reported by Owen.
i have this message when i want to test in a sandbox(biblibre) CONFLICT (content): Merge conflict in catalogue/detail.pl Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/cataloguing.pref Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
Created attachment 13002 [details] [review] Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Two tabs: "<branch> holdings" / "Other holdings" "<branch> holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user or not logged in (opac)), the display is unchanged (1 tab "Holdings")
Created attachment 13003 [details] [review] Bug 7674: Open shelf browser in appropriate tab If you click on "Browse shelf" under "<mylibrary> holdings", shelf browser should open under this tab. If you click on "Browse shelf" under "other holdings" tab, it should open under "other holdings" tab.
Question, it looks like this splits the items based on the homebranch. Shouldn't it be split by the holdingbranch? This seems like the sensible way to do it, if this is a 'show items in my library' feature. It especially makes more sense considering the tabs are labeled 'holdings' and not 'owned by'.
Yes you are probably right. I reset status to Assigned and will send a follow up soon.
Hi Kyle and Jonathan, I think it's debatable. I have been thinking about is since I saw this bug a whlie ago and I am not sure which is the right field to display and if every library wants it to be the same. When I am working in a library where the books are always sent back after being sent to another library for being checked out there and I want to see how many items my library owns - home branch is right. If I want to see what is currently on shelf for a borrower standing at the circulation desk and asking about the book, I would want the holdingbranch. These are 2 examples I came up with, but I think we had this discussion before about the branch to show with the item in OPAC and staff. I am not sure which bug that was, but before changing the patch it's maybe worth trying to figure out what exactly libraries want and if we need to make this an option maybe to be flexible enough.
Making this choice a system pref is the easy solution. There is a lot of conflict over where to show holding vs home branch. For example, in search results I believe we see an items home branch, but in the record details, we see the holding branch. Kyle (In reply to comment #21) > Hi Kyle and Jonathan, > > I think it's debatable. I have been thinking about is since I saw this bug a > whlie ago and I am not sure which is the right field to display and if every > library wants it to be the same. > > When I am working in a library where the books are always sent back after > being sent to another library for being checked out there and I want to see > how many items my library owns - home branch is right. > > If I want to see what is currently on shelf for a borrower standing at the > circulation desk and asking about the book, I would want the holdingbranch. > > These are 2 examples I came up with, but I think we had this discussion > before about the branch to show with the item in OPAC and staff. I am not > sure which bug that was, but before changing the patch it's maybe worth > trying to figure out what exactly libraries want and if we need to make this > an option maybe to be flexible enough.
Created attachment 13189 [details] [review] Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Two tabs: "<branch> holdings" / "Other holdings" "<branch> holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user or not logged in (opac)), the display is unchanged (1 tab "Holdings")
Created attachment 13190 [details] [review] Bug 7674: Open shelf browser in appropriate tab If you click on "Browse shelf" under "<mylibrary> holdings", shelf browser should open under this tab. If you click on "Browse shelf" under "other holdings" tab, it should open under "other holdings" tab.
Created attachment 13191 [details] [review] Bug 7674: Add 2 sysprefs to indicate the branch we want to separate on The 2 sysprefs are SeparateHoldingsBranch and OpacSeparateHoldingsBranch and permit to chose between home branch and holding branch.
2 previous patches rebased on master + additional patch to choose whether we want to split items by home branch or by holding branch
Created attachment 13204 [details] [review] [SIGNED-OFF] Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Two tabs: "<branch> holdings" / "Other holdings" "<branch> holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user or not logged in (opac)), the display is unchanged (1 tab "Holdings") Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 13205 [details] [review] [SIGNED-OFF] Bug 7674: Open shelf browser in appropriate tab If you click on "Browse shelf" under "<mylibrary> holdings", shelf browser should open under this tab. If you click on "Browse shelf" under "other holdings" tab, it should open under "other holdings" tab. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 13206 [details] [review] [SIGNED-OFF] Bug 7674: Add 2 sysprefs to indicate the branch we want to separate on The 2 sysprefs are SeparateHoldingsBranch and OpacSeparateHoldingsBranch and permit to chose between home branch and holding branch. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 13207 [details] [review] Bug 7674 - Followup - Add missing right-hand curly bracket
Tested in both OPAC and staff with both settings. Everything appears to be working great! I did get a syntax error testing the staff side, but it was only a missing curly bracket. I've attached a followup. It is so small that I don't think it justifies holding back a signed-off status. Great work Paul & Julian!
Created attachment 13215 [details] [review] [SIGNED-0FF] Bug 7674 - Followup - Add missing right-hand curly bracket Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Thanks Kyle for the follow-up (next time I will test *after* doing the rebase...)
Hi Julian and others, can someone please rebase those patches? I fixed a conflict in sysprefs.sql, but was not sure how to resolve the next in opac-detail. Please set back to 'signed off' if the conflicts are not severe.
Created attachment 13832 [details] [review] [SIGNED-OFF] Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Two tabs: "<branch> holdings" / "Other holdings" "<branch> holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user or not logged in (opac)), the display is unchanged (1 tab "Holdings") Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 13833 [details] [review] [SIGNED-OFF] Bug 7674: Open shelf browser in appropriate tab If you click on "Browse shelf" under "<mylibrary> holdings", shelf browser should open under this tab. If you click on "Browse shelf" under "other holdings" tab, it should open under "other holdings" tab. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 13834 [details] [review] [SIGNED-OFF] Bug 7674: Add 2 sysprefs to indicate the branch we want to separate on The 2 sysprefs are SeparateHoldingsBranch and OpacSeparateHoldingsBranch and permit to chose between home branch and holding branch. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 13835 [details] [review] [SIGNED-OFF] Bug 7674 - Followup - Add missing right-hand curly bracket Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
All patches rebased on master.
Working on this now.
Created attachment 13990 [details] [review] [SIGNED-OFF] Bug 7674: Separate items in two tabs on biblio detail page (staff+opac) Two tabs: "<branch> holdings" / "Other holdings" "<branch> holdings" tab contains items whose homebranch is the current branch. "Other holdings" tab contains all other items. If current branch is not set (connected with mysql user or not logged in (opac)), the display is unchanged (1 tab "Holdings") Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Created attachment 13991 [details] [review] [SIGNED-OFF] Bug 7674: Open shelf browser in appropriate tab If you click on "Browse shelf" under "<mylibrary> holdings", shelf browser should open under this tab. If you click on "Browse shelf" under "other holdings" tab, it should open under "other holdings" tab. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Created attachment 13992 [details] [review] [SIGNED-OFF] Bug 7674: Add 2 sysprefs to indicate the branch we want to separate on The 2 sysprefs are SeparateHoldingsBranch and OpacSeparateHoldingsBranch and permit to chose between home branch and holding branch. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Created attachment 13993 [details] [review] [SIGNED-OFF] Bug 7674 - Followup - Add missing right-hand curly bracket Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> All tests and perlcritic pass. Tests: Database update - OK, system preferences were added correctly - Version number of updatedatabase was only XXX, changed to 3.11.00.XX OPAC 1) Detail page - OK, no visible change when OpacSeparateHoldings is turned off - OK, switching OpacSeparateHoldingsBranch works correctly - OK, shelf browser opens in correct tab - OK, only one tab shows when logged out INTRANET 2) Detail page - OK, no visible change when SeparateHoldings is turned off - Number of items is missing on top of the tabs Notes: - database update doesn't include bug number - other holdings tab does show when there are 0 items at other branches, it could be hidden in those cases - if there are over 50 items in the holdings or Other holdings tab, the warning and link 'This record has many physical items. Click here to view them all.' is only shown for the first tab. - closing the shelf browser on the second tab goes back to first tab - number ofitems is missing on top of the tabs - there are lots of indentation changes in the second patch, this makes it very hard to see the changes, please provide separate patches in the future
Created attachment 14082 [details] Picture of the problem An extra (and ugly) "%]" shows up on the OPAC in the holdings tab if I do not change any sysprefs at all after applying this patch.
I'm unable to reproduce this problem. I think I've tried any combination of the 2 sysprefs involved here (OpacSeparateHoldings, OpacSeparateHoldingsBranch), logged in status, shelf browser displayed or not, and branch of items. I recreated the git branch like this git fetch origin git checkout -b bug/7674 origin/master git bz apply 7674 # applied all patches I found some debug text still present in the browser shelf so I will submit a patch for that. Jared if you are able to reproduce, could you investigate to find the line of code that produce this extra '%]' ?
Created attachment 14111 [details] [review] Bug 7674: Remove some debug code
It turns out the merge problem was caused trailing whitespace. I am not sure why. However, disabling the whitespace error squelching fixed it. I am going to push this patchset, with the following complaints: 1) If you choose to separate based on homebranch, the display makes no sense at all... the tab says "Centerville holdings ( 1 )" and the items table says "Fairfield" in the location column. However, the display makes no sense anyway since apparently at some point we removed the possibility of displaying the home branch at all. 2) The two holdings tabs display whether or not there is anything to go in them.
Created attachment 14528 [details] [review] Bug 7674: Hide empty items tab
I obsoleted all previous patches that are in master and reset the status to Needs signoff. This patch is for the 2nd Jared's complaint. I don't think there is something to do for the first (in this bug).
Created attachment 16493 [details] [review] Bug 7674: Hide empty items tab Signed-off-by: Brendan <brendan@bywatersolutions.com>
Worked as expected and you have meet all of the other thoughts and options presented. Great. I like this feature.
Patch tested with a sandbox, by Corinne Bulac <corinne.hayet@bulac.fr>
Created attachment 16502 [details] [review] Bug 7674: Hide empty items tab Signed-off-by: Corinne Bulac <corinne.hayet@bulac.fr>
Perhaps this latest patch could be moved to a separate bug?
Moved to 9875