Sales: Call 877.832.5289 (N America)ProTx Form - Subtracting Stock On Successful Payment
When using Magento with stock management turned on and the ‘Decrease Stock When Order is Placed’ set to Yes, stock will be deducted when the order is created, this takes place before the user is redirected to the site which could easily lead to stock issues if the customer does not complete the payment or uses multiple attempts to order.
I have created a work-around for the Protx Form module which credits the stock back to the system before the customer is redirected and then re-deducts the stock when the customer successfully returns.
This requires adding two changes to:
/app/code/core/Mage/Protx/controllers/StandardController.php
Line 63
- public function redirectAction()
- {
- $session = Mage::getSingleton('checkout/session');
- $session->setProtxStandardQuoteId($session->getQuoteId());
- $order = Mage::getModel('sales/order');
- $order->loadByIncrementId($session->getLastRealOrderId());
- $order->addStatusToHistory(
- $order->getStatus(),
- Mage::helper('protx')->__('Customer was redirected to Protx')
- );
- $order->save();
Change to:
- public function redirectAction()
- {
- $session = Mage::getSingleton('checkout/session');
- $session->setProtxStandardQuoteId($session->getQuoteId());
- $order = Mage::getModel('sales/order');
- $order->loadByIncrementId($session->getLastRealOrderId());
- $order->addStatusToHistory(
- $order->getStatus(),
- Mage::helper('protx')->__('Customer was redirected to Protx')
- );
- $order->save();
- /* Put the stock back on, we don't want it taken off yet */
- if(!isset($_SESSION['updated']) || $_SESSION['updated'] != $session->getLastRealOrderId()) {
- /* Put the stock back on, we don't want it taken off yet */
- $items = $order->getAllItems(); // Get all items from the order
- if ($items) {
- foreach($items as $item) {
- if ($item->getParentItem()) { // If product is a parent of a configured product (protected function getFormattedCart ())
- $quantity = $item->getQtyOrdered(); // get Qty ordered
- $product_id = $item->getProductId(); // get it's ID
- $stock = Mage::getModel('cataloginventory/stock_item')->loadByProduct($product_id); // Load the stock for this product
- $stock->setQty($stock->getQty()+$quantity); // Set to new Qty
- $stock->save(); // Save
- }
- continue;
- }
- }
- $_SESSION['updated'] = $session->getLastRealOrderId();
- /* end stock addition */
- }
Line 206 (With above change made)
- protected function saveInvoice (Mage_Sales_Model_Order $order)
- {
- if ($order->canInvoice()) {
- $invoice = $order->prepareInvoice();
- $invoice->register()->capture();
- Mage::getModel('core/resource_transaction')
- ->addObject($invoice)
- ->addObject($invoice->getOrder())
- ->save();
Change to
- protected function saveInvoice (Mage_Sales_Model_Order $order)
- {
- if ($order->canInvoice()) {
- $invoice = $order->prepareInvoice();
- $invoice->register()->capture();
- Mage::getModel('core/resource_transaction')
- ->addObject($invoice)
- ->addObject($invoice->getOrder())
- ->save();
- /* order was paid, deduct stock here */
- $items = $order->getAllItems(); // Get all items from the order
- if ($items) {
- foreach($items as $item) {
- if ($item->getParentItem()) { // If product is a parent of a configured product (protected function getFormattedCart ())
- $quantity = $item->getQtyOrdered(); // get Qty ordered
- $product_id = $item->getProductId(); // get it's ID
- $stock = Mage::getModel('cataloginventory/stock_item')->loadByProduct($product_id); // Load the stock for this product
- $stock->setQty($stock->getQty()-$quantity); // Set to new Qty
- $stock->save(); // Save
- }
- continue;
- }
- }
- /* end stock deduction */
This will subtract the stock only when the invoice is made after a successfull order.
I also changed the Module to add the Protx payment information to the Order Status/Comments to make it easier to keep track of the response from ProTx.
Line 137 (With above changes made)
- if (!$order->getId()) {
- /*
- * need to have logic when there is no order with the order id from protx
- */
- return false;
- }
- $order->addStatusToHistory(
- $order->getStatus(),
- Mage::helper('protx')->__('Customer successfully returned from Protx
- <BR>Status:'.$this->responseArr['Status'].'
- <BR>StatusDetail:'.$this->responseArr['StatusDetail'].'
- <BR>VendorTxCode:'.$this->responseArr['VendorTxCode'].'
- <BR>Amount:'.$this->responseArr['Amount'].'
- <BR>VPSTxId:'.$this->responseArr['VPSTxId'].'
- <BR>TxAuthNo:'.$this->responseArr['TxAuthNo'].'
- <BR>AVSCV2:'.$this->responseArr['AVSCV2'].'
- <BR>AddressResult:'.$this->responseArr['AddressResult'].'
- <BR>PostCodeResult:'.$this->responseArr['PostCodeResult'].'
- <BR>CV2Result:'.$this->responseArr['CV2Result'].'
- <BR>GiftAid:'.$this->responseArr['GiftAid'].'
- <BR>3DSecureStatus:'.$this->responseArr['3DSecureStatus'])
- );
Hopefully this helps someone, I understand it is not an ideal solution to the problem but it does allow you to continue to use Magento’s stock functions without leaving yourself open to stock problems from incomplete orders.
Hopefully in the future Magento will address this and give better control over when stock is deducted in the order proccess.
