Currently, if a shop is using "Authorization only" as their Default credit transaction type there is no way to change the order status using conditional actions. When a shop is using "Auth & Capture Immediately" there is a conditional action trigger (payment entered) and a condition (check order balance) which can move a successful order into a new order status like "payment received."

What is needed is for the "check order balance" to also have the ability to include authorizations, which I've included as a patch. Probably also need another trigger, "authorization entered", but not sure where that would go.

Comments

jrust’s picture

Status: Active » Needs review
tr’s picture

Status: Needs review » Needs work
tr’s picture

Status: Needs work » Needs review

Restarting testing on this thread.

Status: Needs review » Needs work

The last submitted patch, payment-authorize-ca.patch, failed testing.

jrust’s picture

Status: Needs work » Needs review
StatusFileSize
new1.24 KB

Not sure why it didn't pass, but this latest patch is against 2.7 using git.

jrust’s picture

StatusFileSize
new1.21 KB

Fixing syntax error.

Status: Needs review » Needs work

The last submitted patch, 1066264-payment-authorize-ca-2.patch, failed testing.

tr’s picture

The remaining exceptions are PHP notices. See http://drupal.org/node/34341 for how to turn on E_ALL in your local environment so you can see and debug these notices.

jrust’s picture

StatusFileSize
new1.22 KB

Digging the test system! Attached without the warning.

jrust’s picture

Status: Needs work » Needs review
longwave’s picture

Indentation appears to be one space out in the second part:

   );
 
+   $form['include_authorizations'] = array(
+     '#type' => 'checkbox',
+     '#title' => t('Include authorizations?'),
+     '#description' => t('Should "authorization only" credit card transactions be used in calculating the order balance?'),
+     '#default_value' => isset($settings['include_authorizations']) ? $settings['include_authorizations'] : FALSE,
+   );
+
   return $form;
jrust’s picture

StatusFileSize
new1.2 KB

fixing whitespace.

Status: Needs review » Needs work

The last submitted patch, 1066264-payment-authorize-ca-3.patch, failed testing.

jrust’s picture

StatusFileSize
new1.21 KB

Try again.

longwave’s picture

Status: Needs work » Needs review
longwave’s picture

Version: 6.x-2.x-dev » 7.x-3.x-dev
Status: Needs review » Patch (to be ported)

Thanks for your work on this! Committed to 6.x, needs porting to 7.x.

Island Usurper’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new1.24 KB

Had to cross my eyes at that to make sure the logic was right, but I think I've got it now.

Status: Needs review » Needs work

The last submitted patch, 1066264_balance_authorizations.patch, failed testing.

Island Usurper’s picture

Status: Needs work » Needs review
StatusFileSize
new1.3 KB

Looks like setting the default value for that setting is important.

longwave’s picture

Status: Needs review » Fixed

Committed #19.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

cookiecookl’s picture

Issue summary: View changes

Hi,

I have the problem of my customer making a payment, it goes through and is completed as and order. But, the balance doesn't automatically amend itself to release the downloadable file.

If I manually change the balance everything works fine.

So my only question is how to make it automatically change the balance after a payment is received?