Thank you,

More than happy to at my usual email address!

Leon

Great! They have indeed!

There is one issue in sagepay.php with the

do_action( ‘ims_sagepay_notice_error’, $ImStore->error, $cartid );

hook.

Note that one on the parameters is $cartid which is not set until after the if statement. Moving the line

$cartid = trim( $crypt_array[‘VendorTxCode’] );

to before the if statement

if( $crypt_array[‘Status’] != ‘OK’ ){

fixes the issue. Apart from that the hooks work great 🙂

Thanks again!

Leon

Thanks,

Could you also make the following change to sagepay.php

in the function add_hidden_fields( $output ),

could you add a conditional statement for the lines

$crypt .= ‘BillingState=’ . $data[‘billing_state’] . ‘&’;

and

$crypt .= ‘DeliveryState=’ . $data[‘address_state’] . ‘&’;

as these are only required for the US, at the moment I must comment these out after every release.

So, for example in psuedocode

if ( currency = US dollars ) $crypt .= ‘BillingState=’ . $data[‘billing_state’] . ‘&’;

and

if ( currency = US dollars ) $crypt .= ‘DeliveryState=’ . $data[‘address_state’] . ‘&’;

Thanks!