Bug 23386: Add language of original in advanced search
authorFridolin Somers <fridolin.somers@biblibre.com>
Fri, 26 Jul 2019 14:50:23 +0000 (16:50 +0200)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 5 Aug 2019 14:03:21 +0000 (15:03 +0100)
In advanced search there is a language limit (UNIMARC 101$a), add language of original (UNIMARC 101$c).

This patch adds a TT BLOCK to avoid copy the code.

Test plan:
1) Create some records with language and language of original
2) Go to advanced search page
3) Check limit on language is OK
4) Check limit on language of the original is OK
5) Check using both is OK

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/advsearch.tt

index b41c45f..e1da783 100644 (file)
@@ -7,6 +7,23 @@
 [% INCLUDE 'doc-head-close.inc' %]
 </head>
 
+[%- BLOCK language_limit_select -%]
+<select name="limit" id="[% ln_id | html %]">
+    <option value="">No limit</option>
+    [% FOREACH ln_loo IN ln_loop %]
+        [% IF ( ln_loo.selected ) %]
+            <option value="[% ln_index | html %],rtrn:[% ln_loo.iso639_2_code | html %]" selected="selected">
+                [% ln_loo.language_description | html %]
+            </option>
+        [% ELSE %]
+            <option value="[% ln_index | html %],rtrn:[% ln_loo.iso639_2_code | html %]">
+                [% ln_loo.language_description | html %]
+            </option>
+        [% END %]
+    [% END %]
+</select>
+[%- END -%]
+
 <body id="catalog_advsearch" class="catalog">
 [% INCLUDE 'header.inc' %]
 [% INCLUDE 'adv-search.inc' %]
 </fieldset>
 <fieldset id="language">
 <!-- LANGUAGE LIMIT -->
-               <p><label for="language-limit">Language: </label>                                        
-               <select name="limit" id="language-limit">
-        <option value="">No limit</option>
-               [% FOREACH search_languages_loo IN search_languages_loop %]
-               [% IF ( search_languages_loo.selected ) %]
-               <option value="ln,rtrn:[% search_languages_loo.iso639_2_code | html %]" selected="selected">[% search_languages_loo.language_description | html %]</option>
-               [% ELSE %]
-               <option value="ln,rtrn:[% search_languages_loo.iso639_2_code | html %]">[% search_languages_loo.language_description | html %]</option>
-               [% END %]
-               
-               [% END %]
-               </select></p><!-- <a href="">Show all languages</a>-->
+    <p>
+        <label for="language-limit">Language: </label>
+        [% PROCESS language_limit_select ln_loop=search_languages_loop ln_id='language-limit' ln_index='ln' %]
+        <label for="language-original-limit">Language of original: </label>
+        [% PROCESS language_limit_select ln_loop=search_languages_loop ln_id='language-original-limit' ln_index='language-original' %]
+    </p>
 <!-- /LANGUAGE LIMIT -->
 </fieldset>
 </fieldset>