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

PHP fatal error when updating tax rules in WooCommerce #1649

Closed
marcinkrzeminski opened this issue Dec 17, 2024 · 1 comment · Fixed by #1654
Closed

PHP fatal error when updating tax rules in WooCommerce #1649

marcinkrzeminski opened this issue Dec 17, 2024 · 1 comment · Fixed by #1654
Assignees
Labels
Milestone

Comments

@marcinkrzeminski
Copy link
Collaborator

marcinkrzeminski commented Dec 17, 2024

Bug Report

This bug was reported via contact form on our website.

Expected Behavior

Describe what should happen.

Actual Behavior

Describe what actually happens.

[16-Dec-2024 16:45:35 UTC] PHP Fatal error: Uncaught ValueError: The arguments array must contain 4 items, 1 given in /home/mbzcooq/public_html/wp-content/plugins/stream/classes/class-log.php:127Stack trace:#0 /home/mbzcooq/public_html/wp-content/plugins/stream/classes/class-log.php(127): vsprintf()#1 [internal function]: WP_Stream\Log->log()#2 /home/mbzcooq/public_html/wp-content/plugins/stream/classes/class-connector.php(178): call_user_func_array()#3 /home/mbzcooq/public_html/wp-content/plugins/stream/connectors/class-connector-woocommerce.php(632): WP_Stream\Connector->log()#4 [internal function]: WP_Stream\Connector_Woocommerce->callback_woocommerce_tax_rate_updated()#5 /home/mbzcooq/public_html/wp-content/plugins/stream/classes/class-connector.php(127): call_user_func_array()#6 /home/mbzcooq/public_html/wp-includes/class-wp-hook.php(324): WP_Stream\Connector->callback()#7 /home/mbzcooq/public_html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters()#8 /home/mbzcooq/public_html/wp-includes/plugin.php(517): WP_Hook->do_action()#9 /home/mbzcooq/public_html/wp-content/plugins/woocommerce/includes/class-wc-tax.php(1110): do_action()#10 /home/mbzcooq/public_html/wp-content/plugins/woocommerce/includes/class-wc-ajax.php(2971): WC_Tax::_update_tax_rate()#11 /home/mbzcooq/public_html/wp-includes/class-wp-hook.php(324): WC_AJAX::tax_rates_save_changes()#12 /home/mbzcooq/public_html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters()#13 /home/mbzcooq/public_html/wp-includes/plugin.php(517): WP_Hook->do_action()#14 /home/mbzcooq/public_html/wp-admin/admin-ajax.php(192): do_action()#15 {main}thrown in /home/mbzcooq/public_html/wp-content/plugins/stream/classes/class-log.php on line 127

Steps to Reproduce the Problem

  1. Go to Settings / Taxes (you need to have manual taxes setup enabled).
  2. Add a tax rule if you don't have one. Click Save changes
  3. Edit the tax value and click Save changes
  4. The spinner will keep spinning, unless you refresh the site

Screenshots

Drag and drop screenshots in this area that help illustrate the problem.

System Information

  • Stream plugin version:
  • WordPress version:
  • PHP version:
  • Browser:
  • Computer operating system:
@marcinkrzeminski marcinkrzeminski added this to the 4.1.0 milestone Dec 17, 2024
@marcinkrzeminski marcinkrzeminski self-assigned this Dec 17, 2024
@marcinkrzeminski
Copy link
Collaborator Author

Can't reproduce it on two different setups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant