Bug 22886: Missing space between fields from Keyword to MARC mapping in cataloguing...
authorKatrin Fischer <katrin.fischer.83@web.de>
Sat, 11 May 2019 22:34:03 +0000 (00:34 +0200)
committerLiz Rea <liz@bywatersolutions.com>
Tue, 18 Jun 2019 18:19:37 +0000 (18:19 +0000)
If you have multiple subfields mapped to the 'subtitle' keyword,
those fields will show up without separating space.

To test:
- Add 3 subtitle entries in Keyword to MARC mapping for a framework
  - 245 b
  - 245 c
  - 245 h
- Edit a record, make sure the framework matches, add:
  - 245 b, c, h
- Search for it in cataloguing search (not catalog!)
- Verify the subtitle fields show, but are all concatenated
- Apply patch
- Search again and verify the display is improved with a separating
  space

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit f00d909220ebde5162335e255e4ba938a9563bff)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

 Conflicts:
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbooks.tt

Signed-off-by: Liz Rea <liz@bywatersolutions.com>

koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbooks.tt

index 8bace55..3425304 100644 (file)
@@ -75,8 +75,8 @@
             <td><input type="checkbox" class="selection" id="bib[% resultsloo.biblionumber %]" name="biblionumber" value="[% resultsloo.biblionumber %]" /></td>
             <td>
                 <p>
-                    <a href="/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=[% resultsloo.biblionumber %]">[% resultsloo.title |html %]</a>
-                [% FOREACH subtitl IN resultsloo.subtitle %][% subtitl.subfield %][% END %]</p>
+                    <a href="/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=[% resultsloo.biblionumber | uri %]">[% resultsloo.title | html %]</a>
+                [% FOREACH subtitl IN resultsloo.subtitle %][% subtitl.subfield | html %] [% END %]</p>
                 [% IF ( resultsloo.summary ) %]
                     <p>[% resultsloo.summary %]</p>
                 [% ELSE %]