Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WC Subscription plugin support #1

Closed
grumpyoldman-io opened this issue Mar 20, 2014 · 4 comments
Closed

WC Subscription plugin support #1

grumpyoldman-io opened this issue Mar 20, 2014 · 4 comments

Comments

@grumpyoldman-io
Copy link

We have the woocommerce subscription plugin installed and when activating the Mollie plugin this stops working correctly. This is an error on the subscription setting page:

Fatal error: Call to undefined method MPM_Settings::supports() in ***/plugins/woocommerce-subscriptions/classes/class-wc-subscriptions-admin.php on line 1036

here is a complete step-by-step tut for including the plugin support:
http://docs.woothemes.com/document/subscriptions/develop/payment-gateway-integration/
(woothemes.com is down at the moment)

@jesse-mollie
Copy link
Contributor

Thank you for reporting this issue! We'll be looking into it.

@grumpyoldman-io
Copy link
Author

Hey Jesse, thanks, is there an rough estimate in time you take to address issues (be it with a solution or an update)? We have a deadline coming up and I need to take action based on this issue.

@jesse-mollie
Copy link
Contributor

We'll be doing an upgrade for the module soon. We'll be adding some new features and address several issues, including this one. The new version will probably be released somewhere next month.

Seeing how you're on a deadline, here's a quick fix that should solve your immediate problem. Add the following to mpm_settings.php:

public function supports()
{
return array();
}

Note: This code should be within the class, before the last curly brace.

It would appear the subscription plugin thinks the settings class is a payment gateway, and assumes it extends from WC_Payment_Gateway. Since this assumption is incorrect, it throws a fatal error.

@grumpyoldman-io
Copy link
Author

thank you for the well in depth explanation. I already had the quick fix in place so we're good for now. I will follow up in a few weeks.

@codeagencybe codeagencybe mentioned this issue Oct 10, 2016
ndijkstra added a commit that referenced this issue Mar 1, 2017
@joeyboth joeyboth mentioned this issue Jul 10, 2018
mmaymo pushed a commit that referenced this issue Jun 22, 2021
Add klarna payment method translations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants