Bug 18654 - Translatability: Get rid of tt directives starting with [%% in translatio...
authorMarc Véron <veron@veron.ch>
Thu, 29 Jun 2017 13:22:54 +0000 (15:22 +0200)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 30 Aug 2017 19:43:35 +0000 (16:43 -0300)
This patch removes entries like the following in translations of itemsearch.tt:
"[%% INCLUDE form_field_select name=\"homebranch\" options = branches "
"empty_option = \"All libraries\" %%] [%% INCLUDE form_field_select name="
"\"holdingbranch\" options = branches empty_option = \"All libraries\" %%] %s "
"[%% INCLUDE form_field_select name=\"location\" options = locations "
"empty_option = \"All locations\" %%] %s "

New patch on top of Bug 18633 that resolves parts of initial comment.

To test:
- Verify that in itemsearch.tt no tt directives are splitted by new lines
  (search for [% INCLUDE )
- Verify that itemsearch.tt works as before

Followed test plan and verified that tt directives are not split by new
lines, the changes to the fieldset tags in comment 3 have been removed
and itemtype.tt still works correctly as before

Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt

index f4b2e23..bd47edf 100644 (file)
               </fieldset>
           </div>
           <fieldset>
-            [% INCLUDE form_field_select
-              name="homebranch"
-              options = branches
-              empty_option = "All libraries"
-            %]
-            [% INCLUDE form_field_select
-              name="holdingbranch"
-              options = branches
-              empty_option = "All libraries"
-            %]
+            [% INCLUDE form_field_select name="homebranch" options = branches empty_option = "All libraries" %]
+            [% INCLUDE form_field_select name="holdingbranch" options = branche empty_option = "All libraries" %]
             [% IF locations.size %]
-                [% INCLUDE form_field_select
-                  name="location"
-                  options = locations
-                  empty_option = "All locations"
-                %]
+                [% INCLUDE form_field_select name="location" options = locations empty_option = "All locations" %]
             [% END %]
           </fieldset>
           <fieldset>
-            [% INCLUDE form_field_select
-              name="itype"
-              options = itemtypes
-              empty_option = "All item types"
-            %]
-            [% INCLUDE form_field_select
-              name="ccode"
-              options = ccodes
-              empty_option = "All collection codes"
-            %]
+            [% INCLUDE form_field_select name="itype" options = itemtypes empty_option = "All item types" %]
+            [% INCLUDE form_field_select name="ccode" options = ccodes empty_option = "All collection codes" %]
             [% IF notforloans.size %]
-                [% INCLUDE form_field_select
-                  name="notforloan"
-                  options = notforloans
-                  empty_option = "All statuses"
-                %]
+                [% INCLUDE form_field_select name="notforloan" options = notforloans empty_option = "All statuses" %]
             [% END %]
           </fieldset>
           <fieldset>