From 23b52df1d7b906a2434770e62199f78d9fc013ed Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 30 Apr 2010 12:48:53 -0400
Subject: [PATCH] Markup corrections to fix for Bug 4248

- Missing "=" in attribute
- TMPL_IF nested in HTML tag, which breaks translation script

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
---
 .../prog/en/modules/members/memberentrygen.tmpl          | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl
index 811b6223b9..b21d35a260 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl
@@ -404,10 +404,13 @@
         
         <input type="text" id="city" name="city" size="20" value="<!-- TMPL_VAR NAME="city" -->" />
         <!-- TMPL_IF NAME="city_cgipopup" -->or <strong>choose</strong>
-        <select id="select_city" name "select_city">
+        <select id="select_city" name="select_city">
         <!-- TMPL_LOOP NAME="city_loop" -->
-            <option value="<!-- TMPL_VAR NAME='city_zipcode'-->|<!-- TMPL_VAR NAME='city_name' -->"
-                <!-- TMPL_IF NAME="selected" --> selected="1" <!-- /TMPL_IF --> >
+            <!-- TMPL_IF NAME="selected" -->
+            <option value="<!-- TMPL_VAR NAME="city_zipcode"-->|<!-- TMPL_VAR NAME="city_name" -->" selected="selected">
+            <!-- TMPL_ELSE -->
+            <option value="<!-- TMPL_VAR NAME="city_zipcode"-->|<!-- TMPL_VAR NAME="city_name" -->">
+            <!-- /TMPL_IF -->
                 <!-- TMPL_VAR NAME="city_name" --> <!-- TMPL_VAR NAME="city_zipcode" -->
             </option>
         <!-- /TMPL_LOOP -->
-- 
2.39.2