Hi I notice another issue when is updated account balance non admin (uid = 1).
In my case testing user is marinex :-) user create order to deposit 10$ and payment method is "bank transfer" admin must confirm manually this payment. resp. that order is paid. So when admin confirm that marinex order is paid it should update marinex balance but it updates admin balance.
1) There is missing marinex transaction, this is marinex balance:
2) Marinex order bank transfer is confirm by admin, this is admin view:
3)All transaction on site, there is updated admin balance instead of marinex balance:
| Comment | File | Size | Author |
|---|---|---|---|
| All_transactions_Inzertline.png | 27.13 KB | marinex | |
| Orders_Inzertline.png | 33.58 KB | marinex | |
| testting_user_deposit.png | 84.61 KB | marinex |
Comments
Comment #2
marinex commentedComment #3
marinex commentedI thinks that problem is in the function _commerce_funds_rules_update_account_balance() because:
global $user is admin when admin manually confirm payment for marinex order.
I suggest changing to a local variable: $user = user_load($order->uid);
Comment #4
marinex commentedComment #6
aporieI have updated the rules so it updates the account of the user which actually creates the order.
For the specific scenario where the deposit is paid with a second-time payment method (such as cash or bank transfer) I think you can play with rules to make something like that :
- Modify the actual rule for deposit putting a restriction on these payment methods.
- Create a new rule which triggers "Update user account balance" on order update (like, paid) when payment method = these payment methods.
Comment #7
aporie