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
Add default values for admin address filters #44704
Add default values for admin address filters #44704
Conversation
…mmerce_admin_billing_fields" filters
Hi @wavvves, Apart from reviewing the code changes, please make sure to review the testing instructions as well. You can follow this guide to find out what good testing instructions should look like: |
Test Results SummaryCommit SHA: 3efde4d
To view the full API test report, click here. To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing checks out, thanks for the blazing fast PR @senadir 💪🏼
* add default values for "woocommerce_admin_shipping_fields" and "woocommerce_admin_billing_fields" filters * use null instead
* add default values for admin address filters (#44704) * add default values for "woocommerce_admin_shipping_fields" and "woocommerce_admin_billing_fields" filters * use null instead * Prep for cherry pick 44704 --------- Co-authored-by: Seghir Nadir <nadir.seghir@gmail.com> Co-authored-by: WooCommerce Bot <no-reply@woo.com>
* add default values for "woocommerce_admin_shipping_fields" and "woocommerce_admin_billing_fields" filters * use null instead
* add default values for admin address filters (#44704) * add default values for "woocommerce_admin_shipping_fields" and "woocommerce_admin_billing_fields" filters * use null instead * Prep for cherry pick 44704 --------- Co-authored-by: Seghir Nadir <nadir.seghir@gmail.com> Co-authored-by: WooCommerce Bot <no-reply@woo.com>
Submission Review Guidelines:
Changes proposed in this Pull Request:
Recently in the work we did for additional fields, we expanded some WooCommerce Order filters like
woocommerce_admin_shipping_fields
andwoocommerce_admin_billing_fields
filters and added 2 new arguments, we also consume them in our work.This created an issue for other plugins who recall those hooks with the old number of parameters.
To fix this, I added default values to where we consume them.
Issue reported here https://wordpress.org/support/topic/fatal-error-after-woocommerce-update-9
How to test the changes in this Pull Request:
Changelog entry
Significance
Type
Message
Add default values for 'woocommerce_admin_billing_fields' and 'woocommerce_admin_shipping_fields' filters
Comment