[Bug 17367] New: Showing all items must keep show holdings tab in OPAC details
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Bug ID: 17367 Summary: Showing all items must keep show holdings tab in OPAC details Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: fridolin.somers@biblibre.com QA Contact: testopia@bugs.koha-community.org In OPAC details page, when record as too many items (depending on syspref OpacMaxItemsToDisplay), they are not displayed in holdings tab. You can click on link "Click here to view them all" to show them : page reloads with a new arg viewallitems=1. Also you can choose which tab is shown by default using syspref opacSerialDefaultTab. The bug is that when default tab is not holdings, clicking on link to show all items will show another tab. So one must reclick on holings tab to see all items. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|oleonard@myacpl.org |fridolin.somers@biblibre.co | |m -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 --- Comment #1 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Created attachment 55886 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55886&action=edit Bug 17367 - Showing all items must keep show holdings tab in OPAC details In OPAC details page, when record as too many items (depending on syspref OpacMaxItemsToDisplay), they are not displayed in holdings tab. You can click on link "Click here to view them all" to show them : page reloads with a new arg viewallitems=1. Also you can choose which tab is shown by default using syspref opacSerialDefaultTab. The bug is that when default tab is not holdings, clicking on link to show all items will show another tab. So one must reclick on holings tab to see all items. This patch corrects by forcing holdings tab when arg viewallitems is defined. Test plan : - Create a serial record with more items than syspref OpacMaxItemsToDisplay (or decrease this syspref) - Select "Subscriptions tab" for syspref opacSerialDefaultTab - Go to opac details on this record : /cgi-bin/koha/opac-detail.pl?biblionumber=xxx => You see Subscriptions tab selected - Click on Holdings tab => You see no items but a link "Click here to view them all" - Click on this link => Without patch you see Subscriptions tab selected => Wih patch you see Holdings tab selected and the items -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 delaye <stephane.delaye@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |stephane.delaye@biblibre.co | |m --- Comment #2 from delaye <stephane.delaye@biblibre.com> --- i test the patch with de biblibre sandbox N°7 but the system preference opacSerialDefaultTab does not work and the enabling this system preference hide the link "Click here to view them all" in the tab "Holdings" -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sandboxes@biblibre.com Status|Needs Signoff |Signed Off --- Comment #3 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Patch tested with a sandbox, by Juliette <juliette.levast@iepg.fr> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 sandboxes@biblibre.com <sandboxes@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #55886|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 --- Comment #4 from sandboxes@biblibre.com <sandboxes@biblibre.com> --- Created attachment 56193 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56193&action=edit Bug 17367 - Showing all items must keep show holdings tab in OPAC details In OPAC details page, when record as too many items (depending on syspref OpacMaxItemsToDisplay), they are not displayed in holdings tab. You can click on link "Click here to view them all" to show them : page reloads with a new arg viewallitems=1. Also you can choose which tab is shown by default using syspref opacSerialDefaultTab. The bug is that when default tab is not holdings, clicking on link to show all items will show another tab. So one must reclick on holings tab to see all items. This patch corrects by forcing holdings tab when arg viewallitems is defined. Test plan : - Create a serial record with more items than syspref OpacMaxItemsToDisplay (or decrease this syspref) - Select "Subscriptions tab" for syspref opacSerialDefaultTab - Go to opac details on this record : /cgi-bin/koha/opac-detail.pl?biblionumber=xxx => You see Subscriptions tab selected - Click on Holdings tab => You see no items but a link "Click here to view them all" - Click on this link => Without patch you see Subscriptions tab selected => Wih patch you see Holdings tab selected and the items Signed-off-by: Juliette <juliette.levast@iepg.fr> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #56193|0 |1 is obsolete| | --- Comment #5 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 56541 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56541&action=edit Bug 17367 - Showing all items must keep show holdings tab in OPAC details In OPAC details page, when record as too many items (depending on syspref OpacMaxItemsToDisplay), they are not displayed in holdings tab. You can click on link "Click here to view them all" to show them : page reloads with a new arg viewallitems=1. Also you can choose which tab is shown by default using syspref opacSerialDefaultTab. The bug is that when default tab is not holdings, clicking on link to show all items will show another tab. So one must reclick on holings tab to see all items. This patch corrects by forcing holdings tab when arg viewallitems is defined. Test plan : - Create a serial record with more items than syspref OpacMaxItemsToDisplay (or decrease this syspref) - Select "Subscriptions tab" for syspref opacSerialDefaultTab - Go to opac details on this record : /cgi-bin/koha/opac-detail.pl?biblionumber=xxx => You see Subscriptions tab selected - Click on Holdings tab => You see no items but a link "Click here to view them all" - Click on this link => Without patch you see Subscriptions tab selected => Wih patch you see Holdings tab selected and the items Signed-off-by: Juliette <juliette.levast@iepg.fr> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com --- Comment #6 from Nick Clemens <nick@bywatersolutions.com> --- I don't really like the viewalitems as a URL parameter, page refreshes don't revert to the original view, but that is more a comment saying this page would be a good candidate for a js render/Rest API call, but that's a different bug. Nice work Frido! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Brendan Gallagher <brendan@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |brendan@bywatersolutions.co | |m Status|Passed QA |Pushed to Master --- Comment #7 from Brendan Gallagher <brendan@bywatersolutions.com> --- Pushed to Master - Should be in the November 16.11 Release. Thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Frédéric Demians <frederic@tamil.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |frederic@tamil.fr Status|Pushed to Master |Pushed to Stable --- Comment #8 from Frédéric Demians <frederic@tamil.fr> --- Pushed in 16.05. Will be in 16.05.05. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice@biblibre.com --- Comment #9 from Julian Maurice <julian.maurice@biblibre.com> --- Pushed in 3.22.x, will be in 3.22.12 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17367 --- Comment #10 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- (In reply to Nick Clemens from comment #6)
I don't really like the viewalitems as a URL parameter, page refreshes don't revert to the original view, but that is more a comment saying this page would be a good candidate for a js render/Rest API call, but that's a different bug.
Indeed, REST API would be pretty cool. Maybe with a paginated JS DataTable. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org