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

woocommerce_emails_attach_downloadables() must be an instance of WC_Order #60

Open
mwangiwambui opened this issue Apr 10, 2023 · 4 comments

Comments

@mwangiwambui
Copy link

Getting the below error due to woocommerce conflicting with the mpesa plugin. This conflict makes the registration and forget password functionality not work on my wordpress website. See full error log below:

2023-04-10T17:11:53+00:00 CRITICAL Uncaught TypeError: Argument 3 passed to Osen\Woocommerce\Utilities::woocommerce_emails_attach_downloadables() must be an instance of WC_Order, instance of WP_User given, called in /home/site/public_html/wp-includes/class-wp-hook.php on line 310 and defined in /home/site/public_html/wp-content/plugins/osen-wc-mpesa-3.0.1rc/src/Utilities.php:110
Stack trace:
#0 /home/site/public_html/wp-includes/class-wp-hook.php(310): Osen\Woocommerce\Utilities->woocommerce_emails_attach_downloadables(Array, 'customer_reset_...', Object(WP_User))
#1 /home/site/public_html/wp-includes/plugin.php(205): WP_Hook->apply_filters(Array, Array)
#2 /home/site/public_html/wp-content/plugins/woocommerce/includes/emails/class-wc-email.php(461): apply_filters('woocommerce_ema...', Array, 'customer_reset_...', Object(WP_User), Object(WC_Email_Customer_Reset_Password))
#3 /home/site/public_html/wp-content/plugins/woocommerce/includes/emails/class-wc-email-customer-reset-password.php(114): WC_Email->get_attachme in /home/site/public_html/wp-content/plugins/osen-wc-mpesa-3.0.1rc/src/Utilities.php on line 110

@maukoese
Copy link
Contributor

I'll try update the code here, but meantime try downloading from the source.

@edwanyoike
Copy link

@mwangiwambui were you able to fix this?

@maukoese
Copy link
Contributor

Everything should be working fine now.

@edwanyoike
Copy link

hey @maukoese did you push a commit?

Thelastpoet added a commit to Thelastpoet/osen-wc-mpesa that referenced this issue Aug 16, 2023
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

3 participants