You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2012/04/20 08:12:26 UTC

svn commit: r1328205 - in /ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce: WEB-INF/controller.xml customer/changepassword.ftl customer/editcreditcard.ftl customer/editeftaccount.ftl customer/editgiftcard.ftl customer/editperson.ftl

Author: jleroux
Date: Fri Apr 20 06:12:25 2012
New Revision: 1328205

URL: http://svn.apache.org/viewvc?rev=1328205&view=rev
Log:
A patch from Tom Burns for "eCommerce Customer Code Cleanup" https://issues.apache.org/jira/browse/OFBIZ-4830

1. In OFBIZ-4822 it was decided that request to authview would be removed from the eCommerce customer ftl files because their only purpose was to set URL to https. The set field "donePage" in CustomerScreens.xml editperson effects the same result.

2. There are brackets in the password.ftl display for the href:
<a href="<@o...@ofbizUrl>" class="button">[${uiLabelMap.CommonGoBack}]</a>

The brackets are inconsistent with the display for similar elements in the applications

Modified:
    ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml
    ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/changepassword.ftl
    ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editcreditcard.ftl
    ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editeftaccount.ftl
    ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editgiftcard.ftl
    ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editperson.ftl

Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml?rev=1328205&r1=1328204&r2=1328205&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml Fri Apr 20 06:12:25 2012
@@ -116,10 +116,10 @@ under the License.
         <security https="false" auth="false"/>
         <response name="success" type="view" value="main"/>
     </request-map>
-    <request-map uri="authview">
-        <security https="true" auth="true"/>
-        <response name="success" type="view" value="main"/>
-    </request-map>
+<!--     <request-map uri="authview"> -->
+<!--         <security https="true" auth="true"/> -->
+<!--         <response name="success" type="view" value="main"/> -->
+<!--     </request-map> -->
 
     <request-map uri="main"><security https="false" auth="false"/><response name="success" type="view" value="main"/></request-map>
     <request-map uri="policies"><security https="false" auth="false"/><response name="success" type="view" value="policies"/></request-map>

Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/changepassword.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/changepassword.ftl?rev=1328205&r1=1328204&r2=1328205&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/changepassword.ftl (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/changepassword.ftl Fri Apr 20 06:12:25 2012
@@ -18,8 +18,8 @@ under the License.
 -->
 <div class="screenlet">
   <h2>${uiLabelMap.PartyChangePassword}</h2>  
-  &nbsp;<a href="<@o...@ofbizUrl>" class="button">${uiLabelMap.CommonGoBack}</a>
-  &nbsp;<a href="javascript:document.getElementById('changepasswordform').submit()" class="button">${uiLabelMap.CommonSave}</a>
+  &nbsp;<a id="CommonGoBack1" href="<@o...@ofbizUrl>" class="button">${uiLabelMap.CommonGoBack}</a>
+  &nbsp;<a id="CommonSave1" href="javascript:document.getElementById('changepasswordform').submit()" class="button">${uiLabelMap.CommonSave}</a>
   <p/>   
   <div class="screenlet-body">
     <form id="changepasswordform" method="post" action="<@o...@ofbizUrl>">
@@ -43,7 +43,7 @@ under the License.
         <label>${uiLabelMap.CommonFieldsMarkedAreRequired}</label>
       </fieldset>
     </form>
-    <a href="<@o...@ofbizUrl>" class="button">[${uiLabelMap.CommonGoBack}]</a>
+    <a href="<@o...@ofbizUrl>" class="button">[${uiLabelMap.CommonGoBack}]</a>
     <a href="javascript:document.getElementById('changepasswordform').submit()" class="button">[${uiLabelMap.CommonSave}]</a>
   </div>
 </div>
\ No newline at end of file

Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editcreditcard.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editcreditcard.ftl?rev=1328205&r1=1328204&r2=1328205&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editcreditcard.ftl (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editcreditcard.ftl Fri Apr 20 06:12:25 2012
@@ -19,7 +19,7 @@ under the License.
 
 <#if canNotView>
   <h3>${uiLabelMap.AccountingCardInfoNotBelongToYou}.</h3>
-<a href="<@o...@ofbizUrl>" class="button">${uiLabelMap.CommonGoBack}</a>
+<a href="<@o...@ofbizUrl>" class="button">${uiLabelMap.CommonGoBack}</a>
 <#else>
   <#if !creditCard?exists>
       <h2>${uiLabelMap.AccountingAddNewCreditCard}</h2>

Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editeftaccount.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editeftaccount.ftl?rev=1328205&r1=1328204&r2=1328205&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editeftaccount.ftl (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editeftaccount.ftl Fri Apr 20 06:12:25 2012
@@ -19,7 +19,7 @@ under the License.
 
 <#if canNotView>
   <p><h3>${uiLabelMap.AccountingEFTNotBelongToYou}.</h3></p>
-&nbsp;<a href="<@o...@ofbizUrl>" class="buttontext">${uiLabelMap.CommonGoBack}</a>
+&nbsp;<a href="<@o...@ofbizUrl>" class="buttontext">${uiLabelMap.CommonGoBack}</a>
 <#else>
     <#if !eftAccount?exists>
       <h1>${uiLabelMap.AccountingAddNewEftAccount}</h1>

Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editgiftcard.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editgiftcard.ftl?rev=1328205&r1=1328204&r2=1328205&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editgiftcard.ftl (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editgiftcard.ftl Fri Apr 20 06:12:25 2012
@@ -19,7 +19,7 @@ under the License.
 
 <#if canNotView>
   <p><h3>${uiLabelMap.AccountingCardInfoNotBelongToYou}.</h3></p>
-&nbsp;<a href="<@o...@ofbizUrl>" class="buttontext">[${uiLabelMap.CommonGoBack}]</a>
+&nbsp;<a href="<@o...@ofbizUrl>" class="buttontext">[${uiLabelMap.CommonGoBack}]</a>
 <#else>
     <#if !giftCard?exists>
       <h1>${uiLabelMap.AccountingAddNewGiftCard}</h1>

Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editperson.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editperson.ftl?rev=1328205&r1=1328204&r2=1328205&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editperson.ftl (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/customer/editperson.ftl Fri Apr 20 06:12:25 2012
@@ -24,7 +24,7 @@ under the License.
     &nbsp;<form id="editpersonform2" method="post" action="<@o...@ofbizUrl>" name="editpersonform">
 </#if>
 <div>
-  &nbsp;<a href='<@o...@ofbizUrl>' class="button">${uiLabelMap.CommonGoBack}</a>
+  &nbsp;<a href='<@o...@ofbizUrl>' class="button">${uiLabelMap.CommonGoBack}</a>
   &nbsp;<a href="javascript:document.editpersonform.submit()" class="button">${uiLabelMap.CommonSave}</a>
   <p/>    
   <input type="hidden" name="partyId" value="${person.partyId?if_exists}" />
@@ -174,5 +174,5 @@ under the License.
 </table>
 </div>
 </form>
-&nbsp;<a href='<@o...@ofbizUrl>' class="button">${uiLabelMap.CommonGoBack}</a>
+&nbsp;<a href='<@o...@ofbizUrl>' class="button">${uiLabelMap.CommonGoBack}</a>
 &nbsp;<a id="editpersonform3" href="javascript:document.editpersonform.submit()" class="button">${uiLabelMap.CommonSave}</a>
\ No newline at end of file