Re: svn commit: r767136 - in /ofbiz/branches/release09.04: ./ applications/product/webapp/catalog/category/EditCategoryParties.ftl

[Ashish Vijaywargiya]

These are the way of handling the same thing.
To maintain consistency I did this change.

Let's see what other has to say about this.
I am fine with either option.

--
Ashish


Hans Bakker wrote:

can you tell me what the advantage is to do this?
in my opinion it makes the code  unnecessary complicated.

regards,
Hans

On Wed, 2009-04-22 at 10:57 +0530, Ashish Nagar wrote:

jler...@apache.org wrote:

-                <a 
href="<@ofbizUrl>removePartyFromCategory?productCategoryId=${(productCategoryRole.productCategoryId)?if_exists}&partyId=${(productCategoryRole.partyId)?if_exists}&roleTypeId=${(productCategoryRole.roleTypeId)?if_exists}&fromDate=${productCategoryRole.getString("fromDate")}</@ofbizUrl>"
 class="buttontext">${uiLabelMap.CommonDelete}</a>
+                <FORM method="post" action="<@ofbizUrl>removePartyFromCategory</@ofbizUrl>" 
name="lineForm_delete${line}">
+                    <#assign hasExpired = false>
+                    <input type="hidden" name="productCategoryId" 
value="${(productCategoryRole.productCategoryId)?if_exists}">
+                    <input type="hidden" name="partyId" 
value="${(productCategoryRole.partyId)?if_exists}">
+                    <input type="hidden" name="roleTypeId" 
value="${(productCategoryRole.roleTypeId)?if_exists}">
+                    <input type="hidden" name="fromDate" 
value="${(productCategoryRole.getTimestamp("fromDate"))?if_exists}">
+                    <INPUT type="submit" value="${uiLabelMap.CommonDelete}" 
style="font-size: x-small;">
+                </FORM>

Just wondering, could we use anchor <a href= "javascript: document.lineForm_delete${line}.submit()" class= "buttontext">${uiLabelMap.CommonDelete}</a> instead of a submit button so as to maintain consistency?

--
Ashish Nagar

smime.p7s
Description: S/MIME Cryptographic Signature