• Resolved jlev00

    (@jlev00)


    Hi – I’m getting errors related to “Kadence WooCommerce Email Designer” (https://www.ads-software.com/plugins/kadence-woocommerce-email-designer/) when I try to set a product back to “in stock” eg

    [Sun Dec 30 07:57:24.518072 2018] [proxy_fcgi:error] [pid 4530] [client 127.0.0.1:50546] AH01071: Got error 'PHP message: PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function Kadence_Woomail_Designer::add_email_header(), 1 passed in /home/xxxxxx.cloudwaysapps.com/xxxxxx/public_html/wp-includes/class-wp-hook.php on line 286 and exactly 2 expected in /home/xxxxxx.cloudwaysapps.com/xxxxxx/public_html/wp-content/plugins/kadence-woocommerce-email-designer/kadence-woocommerce-email-designer.php:466\nStack trace:\n#0 /home/xxxxxx.cloudwaysapps.com/xxxxxx/public_html/wp-includes/class-wp-hook.php(286): Kadence_Woomail_Designer->add_email_header('xxxxxx...')\n#1 /home/xxxxxx.cloudwaysapps.com/xxxxxx/public_html/wp-includes/class-wp-hook.php(310): WP_Hook->apply_filters('', Array)\n#2 /home/xxxxxx.cloudwaysapps.com/xxxxxx/public_html/wp-includes/plugin.php(453): WP_Hook->do_action(Array)\n#3 /home/xxxxxx.cloudwaysapps.com/xxxxxx/public_html/wp-content/plugins/woocommerce-product-stock-alert/templates/emails/stock_alert_email.p...\n', referer: https://xxxxxx.com/wp-admin/post.php?post=xxxxxx&action=edit

    I could be wrong but this looks similar to the error message here:

    https://www.ads-software.com/support/topic/causes-an-php-fatal-error-when-placing-an-order/

    Would appreciate your thoughts. I’m in a bit of a pickle because both are great plugins and work well independently but I’m not sure what the standard is supposed to be.

    Let me know if you’d like more details from my side.

    edit: this is probably a better support thread to link to: https://www.ads-software.com/support/topic/fatal-error-line-416/

    Same error, different plugin.

    • This topic was modified 6 years, 2 months ago by jlev00.
Viewing 15 replies - 1 through 15 (of 15 total)
  • Hi @jlev00, thanks for reporting this issue.

    Our team will add fix for this issue in our next update.

    We will keep you posted.

    Thread Starter jlev00

    (@jlev00)

    Fantastic! Thank you very much!!

    @dualcube I think this is just a matter of adding $email to
    do_action( 'woocommerce_email_header', $email_heading, $email ); ?>

    Currently it does not have (should be on the 3 templates), hence the error:
    https://plugins.trac.www.ads-software.com/browser/woocommerce-product-stock-alert/trunk/templates/emails/stock_alert_admin_email.php#L13

    Do you have this project on github?

    Thanks

    Hi @joaovpmamede, Yes you can pass second variable $email in woocommerce_email_header action simultaneously also update the class for that template file in get_content_html and get_content_plain function for email.

    @dualcube yup but this needs to be fixed on the plugins’ code.

    @joaovpmamede, we will add the support in our next update.

    We will keep you posted.

    Thread Starter jlev00

    (@jlev00)

    Hi – I just saw you pushed a new version – thank you.

    Was this fix included?

    Thread Starter jlev00

    (@jlev00)

    Hi – just an update to my post above, I finally got around to testing but it looks like the fix hasn’t been included yet. Still getting an error and no emails sent.

    @jlev00, we haven’t added the support in our last version.

    However, we have already started working on this issue. We will keep you posted.

    Hello,
    Have you found a way to solve the problem?
    @dualcube you say add a variable and modify classes. Can you tell me more about it?

    • This reply was modified 5 years, 7 months ago by wingz44.

    @wingz44, @jlev00, @joaovpmamede , we tried to recreate this on our end. However, we are not able to recreate this.

    Can you please share the exact flow we need to follow, so we can fix this.

    Looking for your co-operation regarding the same.

    Thread Starter jlev00

    (@jlev00)

    Hi –

    1) Install WordPress/Woocommerce
    2) Install Storefront and Test Products
    3) Install & Activate WooCommerce Product Stock Alert
    4) Install & Activate Kadence WooCommerce Email Designer
    5) Find a test product. Turn on Stock Management, set stock qty to zero.
    6) View product
    7) Enter email address and click “get an alert”

    The number of interested persons will be updated, but the action will never complete.

    Errors can be found in WooCommerce–>Status–>Logs

    I created a site and product with the above steps here where you can view the “get an alert” action never completing:

    https://woocommerce-128292-928254.cloudwaysapps.com/product/beanie

    Let me know if you would like a login.

    Thread Starter jlev00

    (@jlev00)

    Guys? ??

    How’s the testing going?

    @joaovpmamede I tried editing the three files directly as you suggested and the error is gone but the only email sent is the “back in stock” one. Also seems to be some kind of issue with caching but not sure on this.

    edit: my bad, looks like it is working!

    • This reply was modified 5 years, 7 months ago by jlev00.

    I need help with this, too. I’ve edited the three templates as suggested and I now get the admin alert when a customer has entered an e-mail address, but when I re-stock the product, no e-mail is sent.

    I get the following debug message:

    Fatal error: Uncaught ArgumentCountError: Too few arguments to function Kadence_Woomail_Designer::add_email_header(), 1 passed in /home/NNN/public_html/wp-includes/class-wp-hook.php on line 286 and exactly 2 expected in /home/NNN/public_html/wp-content/plugins/kadence-woocommerce-email-designer/kadence-woocommerce-email-designer.php:538 Stack trace: #0 /home/NNN/public_html/wp-includes/class-wp-hook.php(286): Kadence_Woomail_Designer->add_email_header(‘A product…’) #1 /home/NNN/public_html/wp-includes/class-wp-hook.php(310): WP_Hook->apply_filters(”, Array) #2 /home/NNN/public_html/wp-includes/plugin.php(453): WP_Hook->do_action(Array) #3 /home/NNN/public_html/wp-content/themes/cheerup-child/woocommerce/emails/stock_alert_email.php(13): do_action(‘woocommerce_ema…’, ‘A product…’) #4 /home/NNN/public_html/wp-content/plugins/woocommerce/includes/wc-core-functions.php(249): include(‘/home/NNN/…’) #5 /home/NNN/public_html/wp-content/plugins/woocommerce-product- in /home/NNN/public_html/wp-content/plugins/kadence-woocommerce-email-designer/kadence-woocommerce-email-designer.php on line 538

    Any suggestions how to solve this?

    Best,
    Johannes

Viewing 15 replies - 1 through 15 (of 15 total)
  • The topic ‘Errors with Kadence WooCommerce Email Designer’ is closed to new replies.