[Koha-patches] [PATCH] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / branches

[Katrin Fischer]

From: Owen Leonard <oleonard at myacpl.org>

Correction for branch admin template stops tag attribute markup from
being passed to the template directly in favor of using template
logic.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
---
 admin/branches.pl                                  |    6 +++---
 .../prog/en/modules/admin/branches.tt              |    7 ++++++-
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/admin/branches.pl b/admin/branches.pl
index d6388f5..12c49e2 100755
--- a/admin/branches.pl
+++ b/admin/branches.pl
@@ -254,15 +254,15 @@ sub editbranchform {
     #
     # We export a "categoryloop" array to the template, each element of which
     # contains separate 'categoryname', 'categorycode', 'codedescription', and
-    # 'checked' fields. The $checked field is either '' or 'checked="checked"'
+    # 'checked' fields. The $checked field is either empty or 1'
 
     my $catinfo = GetBranchCategory();
     my @categoryloop = ();
     foreach my $cat (@$catinfo) {
-        my $checked = "";
+        my $checked;
         my $tmp     = quotemeta( $cat->{'categorycode'} );
         if ( grep { /^$tmp$/ } @{ $data->{'categories'} } ) {
-            $checked = "checked=\"checked\"";
+            $checked = 1;
         }
         push @categoryloop, {
             categoryname    => $cat->{'categoryname'},
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
index 42ddb52..1107624 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/branches.tt
@@ -99,7 +99,12 @@
         <ol>
 		[% FOREACH categoryloo IN categoryloop %]
             <li><label for="[% categoryloo.categorycode %]">[% categoryloo.categoryname %]</label>
-                <input type="checkbox" id="[% categoryloo.categorycode %]" name="[% categoryloo.categorycode %]" [% categoryloo.checked %] /> <span class="hint">[% categoryloo.codedescription %]</span>
+                [% IF ( categoryloo.checked ) %]
+                    <input type="checkbox" id="[% categoryloo.categorycode %]" name="[% categoryloo.categorycode %]" checked="checked" />
+                [% ELSE %]
+                    <input type="checkbox" id="[% categoryloo.categorycode %]" name="[% categoryloo.categorycode %]" />
+                [% END %]
+                <span class="hint">[% categoryloo.codedescription %]</span>
             </li>
         [% END %]
 		</ol>
-- 
1.7.4.1