[ 
https://issues.apache.org/jira/browse/OFBIZ-583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12467496
 ] 

Si Chen commented on OFBIZ-583:
-------------------------------

I have noticed this problem too and will try to help you with it.

To everybody on this thread:

If you are creating a fix please update a patch file so that it can be applied. 
 It is very time consuming to walk through descriptive text like Iain Fogg's 
above.  Also we need to have the contribution be made with the "Grant 
license..." button checked on as well.

> Offline payment selection now prevents completion of sales order
> ----------------------------------------------------------------
>
>                 Key: OFBIZ-583
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-583
>             Project: OFBiz (The Open for Business Project)
>          Issue Type: Bug
>          Components: order
>            Reporter: Iain Fogg
>         Assigned To: Si Chen
>            Priority: Critical
>
> I upgraded to the latest SVN on the weekend and now I find that when I raise 
> a Sales Order in the Order Manager, and select OFFLINE as the payment option, 
> the sale cannot proceed because it complains about no payment method being 
> selected.
> CheckOutEvent.java has undergone some significant changes regarding checkout 
> processing, and I suspect the following snippet:
>         methodType = request.getParameter("paymentMethodType");
>         if ("offline".equals(methodType)) {
>             Debug.log("Changing mode from->to: " + mode + "->payment", 
> module);
>             mode = "payment";
>         }
> If I've selected offline payment, shouldn't the mode be set to "addparty", 
> not "payment"?
> Not sure who contributed the most recent changes, but would appreciate a 
> speedy resolution to this one as I can't do sales orders without it!!!
> Cheers, Iain

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to