Author: mridulpathak
Date: Sat Jan 17 15:58:58 2015
New Revision: 1652633
URL: http://svn.apache.org/r1652633
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/release13.07/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl
Modified:
ofbiz/branches/release13.07/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl?rev=1652633&r1=1652632&r2=1652633&view=diff
==============================================================================
---
ofbiz/branches/release13.07/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl
(original)
+++
ofbiz/branches/release13.07/specialpurpose/ecommerce/webapp/ecommerce/customer/miniSignUpForContactList.ftl
Sat Jan 17 15:58:58 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?if_exists}-${partyAndContactMech.tnAreaCode?if_exists}-${partyAndContactMech.tnContactNumber}<#elseif
partyAndContactMech.paAddress1?has_content>${partyAndContactMech.paAddress1},
${partyAndContactMech.paAddress2?if_exists},
${partyAndContactMech.paCity?if_exists},
${partyAndContactMech.paStateProvinceGeoId?if_exists},
${partyAndContactMech.paPostalCode?if_exists},
${partyAndContactMech.paPostalCodeExt?if_exists}
${partyAndContactMech.paCountryGeoId?if_exists}</#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}"/>
