Jump to content

idea: validateOrder is bad idea


icreator

Recommended Posts

set payment method for order and validateOrder is bad idea 

HOW I CAN make payment and set transaction_id if I can not known the ORDER_ID ???

 
public function validateOrder($id_cart, $id_order_state, $amount_paid, $payment_method = 'Unknown',
$message = null, $extra_vars = array(), $currency_special = null, $dont_touch_amount = false,
$secure_key = false, Shop $shop = null)
.....
if (isset($extra_vars['transaction_id']))
$transaction_id = $extra_vars['transaction_id'];
.............
tis is bad idea - use so algoritm

 

good idea:

1. make order from cart and return:

$order_id = $order->add();
 

2. open order details an than select a payment method !

 

than send email for that order via seletted payment method

 

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

плохая идея изначально создавать заказ дл заданного способа оплаты - а если я хочу оплатить другим способом оплаты??

 

надо поменять алгоритм создания заказа и его оплаты на следующий:

1. создаем заказ из корзины, корзина очищается

2. по order_id дальше все делаем - даем пользователю выбрать любой сопосб оплаты и пошлем потом письмо уже с выбранным способом оплаты

3. в истории заказов - в деталях заказа - позволить выбрать любой способ и оплатить им

так чтобы покупатель мог заказ оплатить любым способом а не только одним который был выбран перед созданием заказа

 

 

именно так я сделал в openCart с помощью модуля Multi_Pay

https://opencartforum.com/files/file/2061-%D1%83%D0%BF%D1%80%D0%B0%D0%B2%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5-%D0%B4%D0%B5%D0%BF%D0%BE%D0%B7%D0%B8%D1%82%D0%BE%D0%BC-%D0%B8-%D1%81%D0%BF%D0%BE%D1%81%D0%BE%D0%B1%D0%B0%D0%BC%D0%B8-%D0%BE%D0%BF%D0%BB%D0%B0%D1%82%D1%8B-%D0%B7%D0%B0%D0%BA%D0%B0%D0%B7/

Edited by icreator (see edit history)
Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...