svn commit: r1040426 - /ofbiz/trunk/framework/common/webcommon/includes/countries.ftl

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1040426 - /ofbiz/trunk/framework/common/webcommon/includes/countries.ftl

jleroux@apache.org
Author: jleroux
Date: Tue Nov 30 07:45:53 2010
New Revision: 1040426

URL: http://svn.apache.org/viewvc?rev=1040426&view=rev
Log:
revert r1037521 "Preselect default country into drop down" (https://issues.apache.org/jira/browse/OFBIZ-4013) - OFBIZ-4013

This is not how it should be done. For instance when you update a postal address from a party profile you always get selected the default country. OFBiz records correctly the change underneath but it's confusing as you don't see the right information on screen

Modified:
    ofbiz/trunk/framework/common/webcommon/includes/countries.ftl

Modified: ofbiz/trunk/framework/common/webcommon/includes/countries.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/includes/countries.ftl?rev=1040426&r1=1040425&r2=1040426&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/webcommon/includes/countries.ftl (original)
+++ ofbiz/trunk/framework/common/webcommon/includes/countries.ftl Tue Nov 30 07:45:53 2010
@@ -16,11 +16,8 @@ KIND, either express or implied.  See th
 specific language governing permissions and limitations
 under the License.
 -->
-<#assign
-    countries = Static["org.ofbiz.common.CommonWorkers"].getCountryList(delegator)
-    defaultCountryId = Static["org.ofbiz.base.util.UtilProperties"].getPropertyValue("general","country.geo.id.default")
->
+<#assign countries = Static["org.ofbiz.common.CommonWorkers"].getCountryList(delegator)>
 <#list countries as country>
-    <option value='${country.geoId}'<#if country.geoId == defaultCountryId> selected="selected"</#if>>${country.get("geoName",locale)?default(country.geoId)}</option>
+    <option value='${country.geoId}'>${country.get("geoName",locale)?default(country.geoId)}</option>
 </#list>