• Resolved Mwmatter

    (@mwmatter)


    Hello,

    We are implementing ACH payments through our website and have many existing subscriptions that are being paid for by Credit Card. We are hoping there is a way we can allow our customers to change their payment method to ACH on an existing subscription. We have the ACH portion of the plugin setup and when going to an existing subscription and selecting the option to Change Payment we get the checkout form. We can select ACH and get the Plaid box to connect to a bank (currently doing this all in Sandbox mode). But after going through the Plaid authentication process we get a fatal error. Checking the logs we see this:

    2020-04-08T20:15:34+00:00 CRITICAL Uncaught Error: Call to a member function get_payment_method_title() on null in /var/www/html/testsalonxstore.greatclips.com/wp-content/plugins/woo-stripe-payment/includes/abstract/abstract-wc-payment-gateway-stripe.php:1010
    Stack trace:
    #0 /var/www/html/testsalonxstore.greatclips.com/wp-includes/class-wp-hook.php(290): WC_Payment_Gateway_Stripe->subscription_payment_method_updated(Object(WC_Subscription))
    #1 /var/www/html/testsalonxstore.greatclips.com/wp-includes/class-wp-hook.php(312): WP_Hook->apply_filters(”, Array)
    #2 /var/www/html/testsalonxstore.greatclips.com/wp-includes/plugin.php(478): WP_Hook->do_action(Array)
    #3 /var/www/html/testsalonxstore.greatclips.com/wp-content/plugins/woocommerce-subscriptions/includes/class-wc-subscriptions-change-payment-gateway.php(575): do_action(‘woocommerce_sub…’, Object(WC_Subscription), ‘stripe_ach’)
    #4 /var/www/html/testsalonxstore.greatclips.com/wp-content/plugins/woocommerce-subscriptions/includes/class-wc-subscriptions-change-payment-gateway.php(385): WC_Sub in /var/www/html/testsalonxstore.greatclips.com/wp-content/plugins/woo-stripe-payment/includes/abstract/abstract-wc-payment-gateway-stripe.php on line 1010

    Is this capability supported?

    Thanks!

Viewing 10 replies - 1 through 10 (of 10 total)
  • Plugin Author Payment Plugins

    (@mrclayton)

    Hi @mwmatter,

    Thanks for contacting us. I will try and re-create the error and report back later today. The ability to switch to ACH for a subscription is supported, but there may be a bug in our code.

    We are planning on releasing an update to the plugin in a few days for some added translation strings and we can include any fixed to ACH in that release.

    Kind Regards,

    Plugin Author Payment Plugins

    (@mrclayton)

    Hi @mwmatter,

    I was able to recreate the error you described. It’s a bug in the code specific to adding an ach payment method on the change subscription payment page.

    We’re wrapping up some testing of other features and then we’ll release the update. Most likely tomorrow.

    Kind regards,

    Thread Starter Mwmatter

    (@mwmatter)

    @mrclayton Thanks for the update. Let me know when the update has been released so I can test.

    Plugin Author Payment Plugins

    (@mrclayton)

    @mwmatter,

    Hit us up at our support email and we’ll send you a copy early if you want to help us test.

    Kind Regards,

    Plugin Author Payment Plugins

    (@mrclayton)

    @mwmatter,

    New version released.

    Thread Starter Mwmatter

    (@mwmatter)

    @mrclayton thank you! I will test it out shortly and let you know if I have any issues.

    Thread Starter Mwmatter

    (@mwmatter)

    @mrclayton I was able to successfully change an existing subscription from credit card to ACH. I did however encounter an error when checking the box to “Update the payment method used for all of my current subscriptions (optional)” that appears during the change payment option. Here is the log that was generated:

    2020-04-14T20:03:23+00:00 CRITICAL Uncaught Error: Call to a member function payment_method_supports() on bool in /var/www/html/testsalonxstore.greatclips.com/wp-content/plugins/woocommerce-subscriptions/includes/class-wc-subscriptions-change-payment-gateway.php:427
    Stack trace:
    #0 /var/www/html/testsalonxstore.greatclips.com/wp-content/plugins/woocommerce-subscriptions/includes/class-wc-subscriptions-change-payment-gateway.php(377): WC_Subscriptions_Change_Payment_Gateway::update_all_payment_methods_from_subscription(Object(WC_Subscription), ‘stripe_ach’)
    #1 /var/www/html/testsalonxstore.greatclips.com/wp-includes/class-wp-hook.php(288): WC_Subscriptions_Change_Payment_Gateway::change_payment_method_via_pay_shortcode(”)
    #2 /var/www/html/testsalonxstore.greatclips.com/wp-includes/class-wp-hook.php(312): WP_Hook->apply_filters(NULL, Array)
    #3 /var/www/html/testsalonxstore.greatclips.com/wp-includes/plugin.php(478): WP_Hook->do_action(Array)
    #4 /var/www/html/testsalonxstore.greatclips.com/wp-settings.php(545): do_action(‘wp_loaded’)
    #5 /var/www in /var/www/html/testsalonxstore.greatclips.com/wp-content/plugins/woocommerce-subscriptions/includes/class-wc-subscriptions-change-payment-gateway.php on line 427

    Plugin Author Payment Plugins

    (@mrclayton)

    Hi @mwmatter,

    That error message is coming directly from WCS code. Are you using a new version of the WCS plugin?

    Thanks,

    Thread Starter Mwmatter

    (@mwmatter)

    @mrclayton running the latest version of WC Subscriptions, 3.0.3. Looking at things further, the change all option does work. All of my subs on my test account got updated to ACH as the payment method despite the critical error happening. Wondering if this may be an issue with running this on a test server. Our WCS is in staging mode so maybe there is some compatibility issue there.

    I’ll try testing this on our live site. Thanks!

    Mike

    Plugin Author Payment Plugins

    (@mrclayton)

    @mwmatter,

    Yep it could be due to your data on the test server. I reviewed the WCS code and it doesn’t call any gateways so it couldn’t be caused by the Stripe plugin.

    That being said, feel free to reach out to our support email if you have specific questions.

    Kind Regards,

Viewing 10 replies - 1 through 10 (of 10 total)
  • The topic ‘Change Subscription Payment Method to ACH’ is closed to new replies.