CMS MADE SIMPLE FORGE

Orders

 

[#7772] Order not going to the payment form

avatar
Created By: Mathieu Muths (airelibre) (airelibre)
Date Submitted: Mon Apr 02 05:49:15 -0400 2012

Assigned To: Robert Campbell (calguy1000)
Version: 1.13.1
CMSMS Version: 1.10.3
Severity: Critical
Resolution: Fixed
State: Closed
Summary:
Order not going to the payment form
Detailed Description:
Hi Robert,

There is a bug in the new version of Orders.

File : action.default.php
Line 206 :
$parms = array('order_id'=>$order->get_id);

get_id() is a function, so it must be :
$parms = array('order_id'=>$order->get_id());

This bug makes the module not going to the payment form

Thank you


History

Comments
avatar
Date: 2012-04-18 09:43
Posted By: Deleted User (deleteduser_11200)

Confirmed. I should have looked here first :)

=========================================

* The short story:

In: action.default.php

Change:

205       session_write_close();
206       $parms = array('order_id'=>$order->get_id);

Into:

205       session_write_close();
206       $parms = array('order_id'=>$order->get_id());

* The long story

After setting up  CGEcommerce with Products, FEU, Cart, Orders and the Paypal
gateway I was able to add product items to my cart.
On the Orders confirmation form (FEU data populated from the user logged in) I
was not able to reach the "Confirm Your Order" page.
Everytime I submitted the first step of the billing form {Orders} I was
redirected back to the "Billing Form template" (first step of the orderpage).

By printing some data via $this->Audit() I found out that the order_id was
pulled from the session but not inserted to the params for the Redirect method.
So the oder_id is simply missing, for the fix see above.

CGExtensions: [b]1.28.1[/b]
CGEcommerceBase: [b]1.3.9[/b]
CGSimpleSmarty: [b]1.5[/b]
Products: [b]2.17[/b]
Cart: [b]1.8.2[/b]
CGPaymentGatewayBase: [b]1.0.11[/b]
FrontEndUsers: [b]1.17.1[/b]
Orders: [b]1.13.1[/b]
PaypalGateway: [b]2.3.5[/b]



      
avatar
Date: 2012-04-24 12:01
Posted By: Robert Campbell (calguy1000)

fixed in svn... thanks
      
Updates

Updated: 2013-01-23 12:53
state: Open => Closed

Updated: 2012-04-24 12:01
resolution_id: => 7