[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Jonathan Druart changed: What|Removed |Added Blocks||33567 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33567 [Bug 33567] Remove fallback for Reference_NFL_statuses in C4/XSLT module -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 David Nind changed: What|Removed |Added CC||da...@davidnind.com Keywords||Manual-updated -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Jonathan Druart changed: What|Removed |Added Blocks||28483 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28483 [Bug 28483] Warnings from Search.t must be removed -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #43 from Jonathan Druart --- Follow-up pushed to master. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Jonathan Druart changed: What|Removed |Added Keywords|release-notes-needed| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Marcel de Rooy changed: What|Removed |Added Text to go in the||This report adds preference release notes||Reference_NFL_Statuses that ||allows you to define the ||not for loan statuses that ||you want to be reported as ||Available for reference on ||the OPAC results display ||(XSLT based). ||Basis for ||further adjustments on bug ||26302. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #42 from Marcel de Rooy --- Created attachment 119569 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119569&action=edit Bug 21260: (follow-up) Consolidate preference The bug added a local preference. This should now become a real one. Renamed in the process to better reflect its use. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #41 from Jonathan Druart --- (In reply to Marcel de Rooy from comment #40) > (In reply to Jonathan Druart from comment #39) > > I overlooked this one, I didn't notice the use of the "local pref". > > > > Why didn't we create a "real" system preference here? > > Comment12 > Do you need a follow-up here? Yes please. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #40 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #39) > I overlooked this one, I didn't notice the use of the "local pref". > > Why didn't we create a "real" system preference here? Comment12 Do you need a follow-up here? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org Keywords||release-notes-needed --- Comment #39 from Jonathan Druart --- I overlooked this one, I didn't notice the use of the "local pref". Why didn't we create a "real" system preference here? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Lucas Gass changed: What|Removed |Added Blocks||28114 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28114 [Bug 28114] OPAC Results availability line does not show homebranch/holding branch correctly -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #38 from Marcel de Rooy --- (In reply to Fridolin Somers from comment #37) > This visual change may be too disturbing for stable versions right ? > I'd say we do not backport. Agree -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #37 from Fridolin Somers --- This visual change may be too disturbing for stable versions right ? I'd say we do not backport. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #36 from Jonathan Druart --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Jonathan Druart changed: What|Removed |Added Version(s)||21.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #35 from Marcel de Rooy --- (In reply to Martin Renvoize from comment #34) > Passing QA Thx -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Passed QA --- Comment #34 from Martin Renvoize --- Passing QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added Attachment #115271|0 |1 is obsolete|| --- Comment #33 from Martin Renvoize --- Created attachment 118806 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118806&action=edit Bug 21260: (follow-up) Fix typo in holdingbranch segment Copy-and-paste error. We need holdingbranch here. Signed-off-by: Marcel de Rooy Signed-off-by: ava li Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added Attachment #115270|0 |1 is obsolete|| --- Comment #32 from Martin Renvoize --- Created attachment 118805 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118805&action=edit Bug 21260: Move the count before the callnumbers The current display is a bit confusing [Call number: A](2) seems to indicate that we have two call numbers A. But what it means here, is: we have two items and we list only one, being A. So, we'd better do something like: Items available for loan: Centerville (2) [Call number: A, ..] Which says we have two items at Centerville, listing only one (A), but indicating with the two dots that there are more. Test plan: Pick a biblio with multiple available items at a branch. Include it in a search and check results display. Test the same but with only one branch in Koha. Signed-off-by: Marcel de Rooy Signed-off-by: ava li Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added Attachment #115269|0 |1 is obsolete|| --- Comment #31 from Martin Renvoize --- Created attachment 118804 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118804&action=edit Bug 21260: Introduce local pref to affect status grouping Introducing a local preference Available_NFL to control which not for loan statuses are considered to be 'available for reference'. Standard value is '1|2' which comes down to the former >0 when using the initial Koha defaults. Test plan: [1] Pick a biblio with an available item A, an item B with notforloan 1, an item C with notforloan 2 and two damaged items. (Former patch.) Include it in a OPAC search. You should see: Available: A. Reference: B, C. Not-available: Damaged(2). [2] Add local pref Available_NFL with value '1'. Repeat the search. You should see now: Available: A. Reference: B. Not-available: S.C.(1), Damaged(2). [where S.C. stands for Staff Collection] [3] Run test t/db_dependent/XSLT.t Signed-off-by: Marcel de Rooy Signed-off-by: ava li Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260
Martin Renvoize changed:
What|Removed |Added
Attachment #115268|0 |1
is obsolete||
--- Comment #30 from Martin Renvoize ---
Created attachment 118803
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118803&action=edit
Bug 21260: Remove item details from reallynotforloan block
No need to display itemcallnumbers and branch info for items that
are ('really') not available.
We only show a status and a count for the reallynotforloan ('ordered')
categories (not per branch). This simplifies the code too.
We use the preceding-sibling axis to loop over the unique substatuses.
Test plan:
[1] Pick a biblio with an available item A, an item B with notforloan 1,
an item C with notforloan 2 and two damaged items.
Include it in a OPAC search. You should see:
Available: A. Reference: B, C. Not-available: Damaged(2).
[2] Edit item C (notforloan 2). Change to notforloan -1.
Search again. You should see:
Available: A. Reference: B. Not-available: On order(1), Damaged(2).
Signed-off-by: Marcel de Rooy
Signed-off-by: ava li
Signed-off-by: Martin Renvoize
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added Attachment #115267|0 |1 is obsolete|| --- Comment #29 from Martin Renvoize --- Created attachment 118802 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118802&action=edit Bug 21260: Create three main XSLT Availability segments XSLT changes: [1] Add an item count at the start. Also add variables for counting status available and reference. Use these vars in the corresponding 'blocks'. [2] Refine the No items-test with the new itemcount. [3] Combine the reallynotforloan block with the other statuses by refining (extending) its initial test. All if's are moved up into the former block but are unchanged. Result of these changes makes that the Availability line consists of three segments: 1 Available items, 2 Reference items, 3 Other statuses. Test plan: [1] Check a biblio without any items in the OAPC results. You should see the No items only here. [2] Check a biblio with one item checked out. You should see only the 'third' segment with Checked out(1). [3] Check a biblio with one available and a notforloan==1. You should see segment 1 and 2. Both listing item call numbers. [4] Check a biblio with one available and a notforloan==-1. You should see segment 1 and 3. Both listing item call numbers. (See also the following patch.) Signed-off-by: Marcel de Rooy Signed-off-by: ava li Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added Attachment #115266|0 |1 is obsolete|| --- Comment #28 from Martin Renvoize --- Created attachment 118801 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118801&action=edit Bug 21260: Simple adjustments in Availability blocks [1] Add reallynotforloan=0 test to 'Block 1'. Add a period after No items available. [2] Remove the if with colon block. Remove the and add a span for reallynotforloan (items on order). [3] Add a 'Not available' label for the reallynotforloan block. [4] Remove the On order status, since it is not used anymore since 7611. Test plan: See subsequent patches. Signed-off-by: Marcel de Rooy Signed-off-by: ava li Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added Attachment #115265|0 |1 is obsolete|| --- Comment #27 from Martin Renvoize --- Created attachment 118800 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118800&action=edit Bug 21260: Indentation at higher levels for Availability/Location Only whitespace changes and few comments. Test plan: Nothing to test. Count the spaces ;) Signed-off-by: Marcel de Rooy Signed-off-by: ava li Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #26 from Marcel de Rooy --- (In reply to Martin Renvoize from comment #25) > Could you summarise what this patchset changes, perhaps in the 'text for > release notes'. > > I'm struggling to wrap my head around it. I am willing to condense the commit messages and copy them into the release notes. These messages should probably be enough for now? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com --- Comment #25 from Martin Renvoize --- Could you summarise what this patchset changes, perhaps in the 'text for release notes'. I'm struggling to wrap my head around it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 l...@wgc.school.nz changed: What|Removed |Added Attachment #109182|0 |1 is obsolete|| --- Comment #24 from l...@wgc.school.nz --- Created attachment 115271 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115271&action=edit Bug 21260: (follow-up) Fix typo in holdingbranch segment Copy-and-paste error. We need holdingbranch here. Signed-off-by: Marcel de Rooy Signed-off-by: ava li -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 l...@wgc.school.nz changed: What|Removed |Added Attachment #109112|0 |1 is obsolete|| --- Comment #23 from l...@wgc.school.nz --- Created attachment 115270 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115270&action=edit Bug 21260: Move the count before the callnumbers The current display is a bit confusing [Call number: A](2) seems to indicate that we have two call numbers A. But what it means here, is: we have two items and we list only one, being A. So, we'd better do something like: Items available for loan: Centerville (2) [Call number: A, ..] Which says we have two items at Centerville, listing only one (A), but indicating with the two dots that there are more. Test plan: Pick a biblio with multiple available items at a branch. Include it in a search and check results display. Test the same but with only one branch in Koha. Signed-off-by: Marcel de Rooy Signed-off-by: ava li -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 l...@wgc.school.nz changed: What|Removed |Added Attachment #109111|0 |1 is obsolete|| --- Comment #22 from l...@wgc.school.nz --- Created attachment 115269 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115269&action=edit Bug 21260: Introduce local pref to affect status grouping Introducing a local preference Available_NFL to control which not for loan statuses are considered to be 'available for reference'. Standard value is '1|2' which comes down to the former >0 when using the initial Koha defaults. Test plan: [1] Pick a biblio with an available item A, an item B with notforloan 1, an item C with notforloan 2 and two damaged items. (Former patch.) Include it in a OPAC search. You should see: Available: A. Reference: B, C. Not-available: Damaged(2). [2] Add local pref Available_NFL with value '1'. Repeat the search. You should see now: Available: A. Reference: B. Not-available: S.C.(1), Damaged(2). [where S.C. stands for Staff Collection] [3] Run test t/db_dependent/XSLT.t Signed-off-by: Marcel de Rooy Signed-off-by: ava li -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260
l...@wgc.school.nz changed:
What|Removed |Added
Attachment #109110|0 |1
is obsolete||
--- Comment #21 from l...@wgc.school.nz ---
Created attachment 115268
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115268&action=edit
Bug 21260: Remove item details from reallynotforloan block
No need to display itemcallnumbers and branch info for items that
are ('really') not available.
We only show a status and a count for the reallynotforloan ('ordered')
categories (not per branch). This simplifies the code too.
We use the preceding-sibling axis to loop over the unique substatuses.
Test plan:
[1] Pick a biblio with an available item A, an item B with notforloan 1,
an item C with notforloan 2 and two damaged items.
Include it in a OPAC search. You should see:
Available: A. Reference: B, C. Not-available: Damaged(2).
[2] Edit item C (notforloan 2). Change to notforloan -1.
Search again. You should see:
Available: A. Reference: B. Not-available: On order(1), Damaged(2).
Signed-off-by: Marcel de Rooy
Signed-off-by: ava li
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 l...@wgc.school.nz changed: What|Removed |Added Attachment #109109|0 |1 is obsolete|| --- Comment #20 from l...@wgc.school.nz --- Created attachment 115267 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115267&action=edit Bug 21260: Create three main XSLT Availability segments XSLT changes: [1] Add an item count at the start. Also add variables for counting status available and reference. Use these vars in the corresponding 'blocks'. [2] Refine the No items-test with the new itemcount. [3] Combine the reallynotforloan block with the other statuses by refining (extending) its initial test. All if's are moved up into the former block but are unchanged. Result of these changes makes that the Availability line consists of three segments: 1 Available items, 2 Reference items, 3 Other statuses. Test plan: [1] Check a biblio without any items in the OAPC results. You should see the No items only here. [2] Check a biblio with one item checked out. You should see only the 'third' segment with Checked out(1). [3] Check a biblio with one available and a notforloan==1. You should see segment 1 and 2. Both listing item call numbers. [4] Check a biblio with one available and a notforloan==-1. You should see segment 1 and 3. Both listing item call numbers. (See also the following patch.) Signed-off-by: Marcel de Rooy Signed-off-by: ava li -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 l...@wgc.school.nz changed: What|Removed |Added Attachment #109108|0 |1 is obsolete|| --- Comment #19 from l...@wgc.school.nz --- Created attachment 115266 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115266&action=edit Bug 21260: Simple adjustments in Availability blocks [1] Add reallynotforloan=0 test to 'Block 1'. Add a period after No items available. [2] Remove the if with colon block. Remove the and add a span for reallynotforloan (items on order). [3] Add a 'Not available' label for the reallynotforloan block. [4] Remove the On order status, since it is not used anymore since 7611. Test plan: See subsequent patches. Signed-off-by: Marcel de Rooy Signed-off-by: ava li -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 l...@wgc.school.nz changed: What|Removed |Added Attachment #109107|0 |1 is obsolete|| --- Comment #18 from l...@wgc.school.nz --- Created attachment 115265 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115265&action=edit Bug 21260: Indentation at higher levels for Availability/Location Only whitespace changes and few comments. Test plan: Nothing to test. Count the spaces ;) Signed-off-by: Marcel de Rooy Signed-off-by: ava li -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 l...@wgc.school.nz changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #17 from Victor Grousset/tuxayo --- Indeed, that makes sense ^^ Is there a step to do before testing except restarting all? I'm getting > Availability: Items available for loan: 1 , 1 , 2 . Instead of > Availability: Items available for loan: Midway (1), Centerville (1), Fairview > (2). https://wtf.roflcopter.fr/pics/zlFdbnni/w3vBdmWN.png -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #16 from Marcel de Rooy --- (In reply to Victor Grousset/tuxayo from comment #15) > Is the status "Needs Signoff" correct? All the patches have been signed off. By the author only. So he tested :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Victor Grousset/tuxayo changed: What|Removed |Added CC||vic...@tuxayo.net --- Comment #15 from Victor Grousset/tuxayo --- Is the status "Needs Signoff" correct? All the patches have been signed off. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #14 from Marcel de Rooy --- Created attachment 109182 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109182&action=edit Bug 21260: (follow-up) Fix typo in holdingbranch segment Copy-and-paste error. We need holdingbranch here. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #13 from Marcel de Rooy --- FIXME Call number: , .. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Marcel de Rooy changed: What|Removed |Added Blocks||26302 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26302 [Bug 26302] OPAC XSLT Results: List variable number of itemcallnumbers -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #12 from Marcel de Rooy --- NOTE: Obviously we can promote the local pref to a real pref if this development meets sufficient interest. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Marcel de Rooy changed: What|Removed |Added Status|ASSIGNED|Needs Signoff CC||n...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #11 from Marcel de Rooy --- Created attachment 109112 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109112&action=edit Bug 21260: Move the count before the callnumbers The current display is a bit confusing [Call number: A](2) seems to indicate that we have two call numbers A. But what it means here, is: we have two items and we list only one, being A. So, we'd better do something like: Items available for loan: Centerville (2) [Call number: A, ..] Which says we have two items at Centerville, listing only one (A), but indicating with the two dots that there are more. Test plan: Pick a biblio with multiple available items at a branch. Include it in a search and check results display. Test the same but with only one branch in Koha. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #10 from Marcel de Rooy --- Created attachment 109111 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109111&action=edit Bug 21260: Introduce local pref to affect status grouping Introducing a local preference Available_NFL to control which not for loan statuses are considered to be 'available for reference'. Standard value is '1|2' which comes down to the former >0 when using the initial Koha defaults. Test plan: [1] Pick a biblio with an available item A, an item B with notforloan 1, an item C with notforloan 2 and two damaged items. (Former patch.) Include it in a OPAC search. You should see: Available: A. Reference: B, C. Not-available: Damaged(2). [2] Add local pref Available_NFL with value '1'. Repeat the search. You should see now: Available: A. Reference: B. Not-available: S.C.(1), Damaged(2). [where S.C. stands for Staff Collection] [3] Run test t/db_dependent/XSLT.t Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #6 from Marcel de Rooy --- Created attachment 109107 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109107&action=edit Bug 21260: Indentation at higher levels for Availability/Location Only whitespace changes and few comments. Test plan: Nothing to test. Count the spaces ;) Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #7 from Marcel de Rooy --- Created attachment 109108 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109108&action=edit Bug 21260: Simple adjustments in Availability blocks [1] Add reallynotforloan=0 test to 'Block 1'. Add a period after No items available. [2] Remove the if with colon block. Remove the and add a span for reallynotforloan (items on order). [3] Add a 'Not available' label for the reallynotforloan block. [4] Remove the On order status, since it is not used anymore since 7611. Test plan: See subsequent patches. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260
--- Comment #9 from Marcel de Rooy ---
Created attachment 109110
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109110&action=edit
Bug 21260: Remove item details from reallynotforloan block
No need to display itemcallnumbers and branch info for items that
are ('really') not available.
We only show a status and a count for the reallynotforloan ('ordered')
categories (not per branch). This simplifies the code too.
We use the preceding-sibling axis to loop over the unique substatuses.
Test plan:
[1] Pick a biblio with an available item A, an item B with notforloan 1,
an item C with notforloan 2 and two damaged items.
Include it in a OPAC search. You should see:
Available: A. Reference: B, C. Not-available: Damaged(2).
[2] Edit item C (notforloan 2). Change to notforloan -1.
Search again. You should see:
Available: A. Reference: B. Not-available: On order(1), Damaged(2).
Signed-off-by: Marcel de Rooy
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 --- Comment #8 from Marcel de Rooy --- Created attachment 109109 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109109&action=edit Bug 21260: Create three main XSLT Availability segments XSLT changes: [1] Add an item count at the start. Also add variables for counting status available and reference. Use these vars in the corresponding 'blocks'. [2] Refine the No items-test with the new itemcount. [3] Combine the reallynotforloan block with the other statuses by refining (extending) its initial test. All if's are moved up into the former block but are unchanged. Result of these changes makes that the Availability line consists of three segments: 1 Available items, 2 Reference items, 3 Other statuses. Test plan: [1] Check a biblio without any items in the OAPC results. You should see the No items only here. [2] Check a biblio with one item checked out. You should see only the 'third' segment with Checked out(1). [3] Check a biblio with one available and a notforloan==1. You should see segment 1 and 2. Both listing item call numbers. [4] Check a biblio with one available and a notforloan==-1. You should see segment 1 and 3. Both listing item call numbers. (See also the following patch.) Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21260] Improve the Availability line of OPAC XSLT search results
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21260 Marcel de Rooy changed: What|Removed |Added Summary|Show a few more items in|Improve the Availability |Availability line of XSLT |line of OPAC XSLT search |search results |results -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
