TPac shows too many MARC field values

Bug #968556 reported by Thomas Berezansky
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

The branch below seeks to fix this by adding an "OPAC Visible" flag to Coded Value Maps, allowing hiding of those you don't want to show up in your search boxes. For example, I am assuming that most libraries don't actually feel that patrons need to be able to search for every language defined by MARC. Or perhaps you don't have any maps cataloged, so you don't need to show that bib type at all.

Also, because patrons don't always understand the language catalogers do it adds a "Search Label" that, if set, is used instead of the value. For example, you might want to change "Projected Medium" to "Video Recordings" or change "Language Material" to "Books" for patron benefit.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/tpac_value_maps

Tags: pullrequest
Revision history for this message
Thomas Berezansky (tsbere) wrote :

Pushed a rebased branch that includes an extra commit to add a "Simple" flag to value maps. This pushes things to the top of TPac lists. JSPac currently ignores those entries.

The idea is to use comma delimited code lists to "group" things. If you are keeping all the specific entries for advanced users you flag the grouped entries as "Simple" and they show up at the top of lists in TPac. If you are grouping things to reduce the length of lists you just hide the single entries with the opac visible option.

This basically ends up reproducing the "Simple/Advanced" views from JSPac with simple above a divider and advanced below it.

It also fixes an excessive escaping issue that was causing things to not match from TPac.

Revision history for this message
Ben Shum (bshum) wrote :

Tested the rebased branch including the "Simple" flag and everything seems to be working so far.

Signed off on all three top commits and rebased against master here: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/tpac_value_maps

Revision history for this message
Thomas Berezansky (tsbere) wrote :

Pushed the signed off branch to master :D

Changed in evergreen:
status: New → Fix Committed
Ben Shum (bshum)
Changed in evergreen:
milestone: none → 2.3.0-alpha
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.