Re: svn commit: r612450 - /ofbiz/trunk/applications/order/webapp/ordermgr/entry/checkoutoptions.ftl

[Bilgin Ibryam]

Jacques,

I suppose that you saw the bug in ecommece application I did not realized that checkoutoptions.ftl is used also in ecommerce screens.
I wanted to fix the broken link in order manager quick checkout screen.

Something like this should fix the bug in both screens(it is ugly but works)

Index: applications/order/webapp/ordermgr/entry/checkoutoptions.ftl
===================================================================
--- applications/order/webapp/ordermgr/entry/checkoutoptions.ftl (revision 612454)
+++ applications/order/webapp/ordermgr/entry/checkoutoptions.ftl (working copy)
@@ -276,7 +276,8 @@
                       </#list>
                       </b>
                       </div>
-                      <div class="tabletext">${uiLabelMap.OrderUpdateEmailAddress} <a href="<@ofbizUrl>viewprofile?DONE_PAGE=quickcheckout</@ofbizUrl>" class="buttontext">${uiLabelMap.PartyProfile}</a>.</div>
+                      <div class="tabletext">${uiLabelMap.OrderUpdateEmailAddress} <a href="<#if customerDetailLink?exists>${customerDetailLink}${shoppingCart.getPartyId()}" target="partymgr"
+                        <#else><@ofbizUrl>viewprofile?DONE_PAGE=quickcheckout</@ofbizUrl>"</#if> class="buttontext">${uiLabelMap.PartyProfile}</a>.</div>
                       <br/>
                       <div class="tabletext">${uiLabelMap.OrderCommaSeperatedEmailAddresses}:</div>
                       <input type="text" class="inputBox" size="30" name="order_additional_emails" value="${shoppingCart.getOrderAdditionalEmails()?if_exists}"/>

What do you think?
Bilgin

[Jacques Le Roux]

Bilgin,

Yes I understood, but had no time to test it more. If you test it and it works (it should from your previous tests I suppose) I
think it's good for now

Thanks

Jacques

From: "Bilgin Ibryam" <[hidden email]>

> Jacques,
>
> I suppose that you saw the bug in ecommece application I did not realized that checkoutoptions.ftl is used also in ecommerce
> screens.
> I wanted to fix the broken link in order manager quick checkout screen.
>
> Something like this should fix the bug in both screens(it is ugly but works)
>
> Index: applications/order/webapp/ordermgr/entry/checkoutoptions.ftl
> ===================================================================
> --- applications/order/webapp/ordermgr/entry/checkoutoptions.ftl (revision 612454)
> +++ applications/order/webapp/ordermgr/entry/checkoutoptions.ftl (working copy)
> @@ -276,7 +276,8 @@
>                       </#list>
>                       </b>
>                       </div>
> -                      <div class="tabletext">${uiLabelMap.OrderUpdateEmailAddress} <a
> href="<@ofbizUrl>viewprofile?DONE_PAGE=quickcheckout</@ofbizUrl>" class="buttontext">${uiLabelMap.PartyProfile}</a>.</div>
> +                      <div class="tabletext">${uiLabelMap.OrderUpdateEmailAddress} <a href="<#if
> customerDetailLink?exists>${customerDetailLink}${shoppingCart.getPartyId()}" target="partymgr"
> +                        <#else><@ofbizUrl>viewprofile?DONE_PAGE=quickcheckout</@ofbizUrl>"</#if>
> class="buttontext">${uiLabelMap.PartyProfile}</a>.</div>
>                       <br/>
>                       <div class="tabletext">${uiLabelMap.OrderCommaSeperatedEmailAddresses}:</div>
>                       <input type="text" class="inputBox" size="30" name="order_additional_emails"
> value="${shoppingCart.getOrderAdditionalEmails()?if_exists}"/>
>
> What do you think?
> Bilgin
>
>