LP#1623955: Keep periods in subject links
authorKathy Lussier <klussier@masslnc.org>
Thu, 29 Sep 2016 17:50:40 +0000 (13:50 -0400)
committerDan Wells <dbw2@calvin.edu>
Fri, 30 Sep 2016 13:25:50 +0000 (09:25 -0400)
The subject links in the record summary were stripping periods, changing
headings that contained N.Y., for example, to NY in the subsequent search
string. Since normalization at index time replaces the period with a space,
clicking the subject links did not successfully retrieve all relevant
results.

Test plan:
Click the Subject link for a record that has New York (N.Y.) in its heading and
has no other instances of ny in the keyword index. This record will not be
retrieved when you click the link. After loading the patch, the record will be
successfully retrieved, along with other records that contain New York (N.Y.) in
their subject headings.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/src/templates/opac/parts/record/subjects.tt2

index 6b99b13..b0702a4 100644 (file)
@@ -55,7 +55,7 @@
                 single_term = raw_term | html;
 
                 # facets should be used as-is
-                IF !s.facet; raw_term = raw_term.replace('\-', ' ').replace('[#"^$\+,\.:;&|\[\]()]', ''); END;
+                IF !s.facet; raw_term = raw_term.replace('\-', ' ').replace('[#"^$\+,:;&|\[\]()]', ''); END;
 
                 all_terms.push(raw_term);