svn commit: r887173 - /ofbiz/trunk/applications/order/webapp/ordermgr/return/returnItems.ftl

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

svn commit: r887173 - /ofbiz/trunk/applications/order/webapp/ordermgr/return/returnItems.ftl

ashish-18
Author: ashish
Date: Fri Dec  4 12:39:39 2009
New Revision: 887173

URL: http://svn.apache.org/viewvc?rev=887173&view=rev
Log:
Applied patch from jira issue OFBIZ-3304 - Remove links for adjustments in create return screen is not working.
Remove links for adjustment on return screen is not working.

Steps to regenerate issue:

1) Create an sales order and complete order by 'Quick Ship Entire Order' from order detail page.

2) Clicks on Create return link on order detail page

3) Clicks on Select all checkbox and then click on 'Return Selected items'

4) On Return Items screen 'Remove' links for adjustment gives secure url error.

Thanks Arun for the contribution.

Modified:
    ofbiz/trunk/applications/order/webapp/ordermgr/return/returnItems.ftl

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/return/returnItems.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/return/returnItems.ftl?rev=887173&r1=887172&r2=887173&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/return/returnItems.ftl (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/return/returnItems.ftl Fri Dec  4 12:39:39 2009
@@ -56,12 +56,13 @@
           </div>
        </td>
        <#if (adjEditable)>
-       <td align='right'><a href="<@ofbizUrl>removeReturnAdjustment?returnAdjustmentId=${returnAdjustment.returnAdjustmentId}&returnId=${returnAdjustment.returnId}</@ofbizUrl>" class="buttontext">${uiLabelMap.CommonRemove}</a></td>
+         <td align='right'><a href='javascript:document.removeReturnAdjustment_${rowCountForAdjRemove}.submit()' class='buttontext'>${uiLabelMap.CommonRemove}</a></td>
        <#else>
        <td>&nbsp;</td>
        </#if>
         <#if (adjEditable)>
           <#assign rowCount = rowCount + 1>
+          <#assign rowCountForAdjRemove = rowCountForAdjRemove + 1>
        </#if>
        <#assign returnTotal = returnTotal + returnAdjustment.amount?default(0)>
     </tr>
@@ -139,6 +140,7 @@
           </tr>
           <#assign returnTotal = 0.0>
           <#assign rowCount = 0>
+          <#assign rowCountForAdjRemove = 0>
           <form method="post" action="<@ofbizUrl>updateReturnItems</@ofbizUrl>">
           <input type="hidden" name="_useRowSubmit" value="Y">
           <#if returnItems?has_content>
@@ -322,6 +324,14 @@
             </form>
           </#list>
         </#if>
+        <#if returnAdjustments?has_content>
+          <#list returnAdjustments as returnAdjustment>
+            <form name="removeReturnAdjustment_${returnAdjustment_index}" method="post" action="<@ofbizUrl>removeReturnAdjustment</@ofbizUrl>">
+              <input type="hidden" name="returnId" value="${returnAdjustment.returnId}"/>
+              <input type="hidden" name="returnAdjustmentId" value="${returnAdjustment.returnAdjustmentId}"/>
+            </form>
+          </#list>
+        </#if>
         <#if (returnHeader.statusId == "RETURN_REQUESTED" || returnHeader.statusId == "SUP_RETURN_REQUESTED") && (rowCount > 0)>
         <br/>
         <form name="acceptReturn" method="post" action="<@ofbizUrl>/updateReturn</@ofbizUrl>">