Bug 25871

Summary: Add "only library" to OpacItemLocation options
Product: Koha Reporter: Owen Leonard <oleonard>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: amitddng135, lucas
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00
Bug Depends on:    
Bug Blocks: 12561    
Attachments: Bug 25871: Add "only library" to OpacItemLocation options
0001-Bug-25871-Add-only-library-to-OpacItemLocation-optio.patch
Bug 25871: Add "only library" to OpacItemLocation options
Bug 25871: (QA follow-up) Update syspref

Description Owen Leonard 2020-06-24 19:56:20 UTC
The OpacItemLocation preference currently has three choices:

- location: location
- ccode: "collection code"
- callnum: "call number only"

I propose that we clarify the labels for these options and add a new one:

- location: "library, location, and call number"
- ccode: "library, collection code, and call number"
- callnum: "library and call number"
- library: "only library"

The "only library" option would provide an option which was previously only available in the non-XSLT view via the "don't show" option under the "OPACItemsResultsDisplay" : 

"Availability: Items available for loan: Athens (2), Nelsonville (1)."

If this change is acceptable we can WONTFIX Bug 5229 and create a bug to remove the OpacItemsResultsDisplay preference.
Comment 1 Owen Leonard 2020-06-24 20:25:49 UTC
Created attachment 106262 [details] [review]
Bug 25871: Add "only library" to OpacItemLocation options

This patch adds a new option to the OpacItemLocation system preference,
"only library." It also rewords the descriptions of the existing options
in order to make their use more clear.

To test, apply the patch and make sure your OPACXSLTResultsDisplay
system preference is set to "default."

Test the OPAC search results using each of the OpacItemLocation
preference options:

- location: "library, location, and call number," e.g.

  "Availability: Items available for loan: Athens (1).
   Location(s): Athens new book shelf Call number: JNF 952.025 Bl."

- ccode: "library, collection code, and call number," e.g.

  "Availability: Items available for loan: Athens (1).
   Collection(s): Juvenile non-fiction Call number: JNF 952.025 Bl."

- callnum: "library and call number," e.g.

  "Availability: Items available for loan: AthensCall number: JNF
   952.025 Bl (1)."

- library: "only library," e.g."

  "Availability: Items available for loan: Athens (1)."
Comment 2 Amit Gupta 2020-07-27 16:46:40 UTC
Created attachment 107443 [details] [review]
0001-Bug-25871-Add-only-library-to-OpacItemLocation-optio.patch

Work OK.

This patch adds a new option to the OpacItemLocation system preference,
"only library." It also rewords the descriptions of the existing options
in order to make their use more clear.

To test, apply the patch and make sure your OPACXSLTResultsDisplay
system preference is set to "default."

Test the OPAC search results using each of the OpacItemLocation
preference options:

- location: "library, location, and call number," e.g.

  "Availability: Items available for loan: Athens (1).
   Location(s): Athens new book shelf Call number: JNF 952.025 Bl."

- ccode: "library, collection code, and call number," e.g.

  "Availability: Items available for loan: Athens (1).
   Collection(s): Juvenile non-fiction Call number: JNF 952.025 Bl."

- callnum: "library and call number," e.g.

  "Availability: Items available for loan: AthensCall number: JNF
   952.025 Bl (1)."

- library: "only library," e.g."

  "Availability: Items available for loan: Athens (1)."
Comment 3 Nick Clemens 2020-08-04 00:50:20 UTC
Created attachment 107723 [details] [review]
Bug 25871: Add "only library" to OpacItemLocation options

This patch adds a new option to the OpacItemLocation system preference,
"only library." It also rewords the descriptions of the existing options
in order to make their use more clear.

To test, apply the patch and make sure your OPACXSLTResultsDisplay
system preference is set to "default."

Test the OPAC search results using each of the OpacItemLocation
preference options:

- location: "library, location, and call number," e.g.

  "Availability: Items available for loan: Athens (1).
   Location(s): Athens new book shelf Call number: JNF 952.025 Bl."

- ccode: "library, collection code, and call number," e.g.

  "Availability: Items available for loan: Athens (1).
   Collection(s): Juvenile non-fiction Call number: JNF 952.025 Bl."

- callnum: "library and call number," e.g.

  "Availability: Items available for loan: AthensCall number: JNF
   952.025 Bl (1)."

- library: "only library," e.g."

  "Availability: Items available for loan: Athens (1)."

Signed-off-by: Amit Gupta <amit.gupta@informaticsglobal.com>
Comment 4 Nick Clemens 2020-08-04 00:50:23 UTC
Created attachment 107724 [details] [review]
Bug 25871: (QA follow-up) Update syspref
Comment 5 Jonathan Druart 2020-08-06 09:38:07 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 6 Lucas Gass 2020-08-13 18:33:09 UTC
not backporting enhancement to 20.05.x series