svn commit: r1652628 - /ofbiz/branches/release14.12/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl

[mr...@apache.org]

Author: mridulpathak
Date: Sat Jan 17 15:48:18 2015
New Revision: 1652628

URL: http://svn.apache.org/r1652628
Log:
[OFBIZ-3433] eCommerce main store front error returned when attempt is made to "Sign Up For Contact List" - as reported by Ruth. Not a service level bug but a UI issue. Email is a required input parameter in "signUpForContactList" service. Sign up form does have a field for email but has no label which is confusing to user, neither it is displayed as a required field. Applied patch from the issue to fix UI. Thanks Ruth for reporting the issue. Applying changes from trunk r1652627.

Modified:
    ofbiz/branches/release14.12/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl

Modified: ofbiz/branches/release14.12/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl
URL: http://svn.apache.org/viewvc/ofbiz/branches/release14.12/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl?rev=1652628&r1=1652627&r2=1652628&view=diff
==============================================================================
--- ofbiz/branches/release14.12/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl (original)
+++ ofbiz/branches/release14.12/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl Sat Jan 17 15:48:18 2015
@@ -65,7 +65,8 @@ under the License.
             <@contactList publicEmailContactLists=publicEmailContactLists/>
           </div>
           <div>
-            <select name="preferredContactMechId" class="selectBox">
+            <label for="preferredContactMechId">${uiLabelMap.CommonEmail} *</label>
+            <select id="preferredContactMechId" name="preferredContactMechId" class="selectBox">
               <#list partyAndContactMechList as partyAndContactMech>
                 <option value="${partyAndContactMech.contactMechId}"><#if partyAndContactMech.infoString?has_content>${partyAndContactMech.infoString}<#elseif partyAndContactMech.tnContactNumber?has_content>${partyAndContactMech.tnCountryCode!}-${partyAndContactMech.tnAreaCode!}-${partyAndContactMech.tnContactNumber}<#elseif partyAndContactMech.paAddress1?has_content>${partyAndContactMech.paAddress1}, ${partyAndContactMech.paAddress2!}, ${partyAndContactMech.paCity!}, ${partyAndContactMech.paStateProvinceGeoId!}, ${partyAndContactMech.paPostalCode!}, ${partyAndContactMech.paPostalCodeExt!} ${partyAndContactMech.paCountryGeoId!}</#if></option>
               </#list>
@@ -95,7 +96,8 @@ under the License.
           <@contactList publicEmailContactLists=publicEmailContactLists/>
         </div>
         <div>
-          <input name="email" class="inputBox" type="text"/>
+          <label for="email">${uiLabelMap.CommonEmail} *</label>
+          <input id="email" name="email" class="required" type="text"/>
         </div>
         <div>
           <input type="submit" value="${uiLabelMap.EcommerceSubscribe}"/>