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

Obcasny problem s undefined country pri kontrole ICO/DIC #27

Closed
morvy opened this issue Nov 18, 2020 · 1 comment
Closed

Obcasny problem s undefined country pri kontrole ICO/DIC #27

morvy opened this issue Nov 18, 2020 · 1 comment

Comments

@morvy
Copy link
Contributor

morvy commented Nov 18, 2020

Obcas sa stane, ze pri odosielani objednavky nie je definovane billing_country. Predpokladam, ze su to pripady ked zakaznik iba odosle objednavku bez vyplnenych povinnych poli, no potom by nemala byt kontrola ostatnych poli ani spustena, kedze nema s cim porovnat.

Myslim, ze by stacilo pridat podmienku if (!isset($_POST['billing_country'])) { return false; }

ErrorException: Notice: Undefined index: billing_country
#13 /wp-content/plugins/woolab-ic-dic/includes/filters-actions.php(144): woolab_icdic_checkout_field_process
#12 /wp-includes/class-wp-hook.php(285): WP_Hook::apply_filters
#11 /wp-includes/class-wp-hook.php(311): WP_Hook::do_action
#10 /wp-includes/plugin.php(478): do_action
#9 /wp-content/plugins/woocommerce/includes/class-wc-checkout.php(1092): WC_Checkout::process_checkout
#8 /wp-content/plugins/woocommerce/includes/class-wc-form-handler.php(353): WC_Form_Handler::checkout_action
#7 /wp-includes/class-wp-hook.php(287): WP_Hook::apply_filters
#6 /wp-includes/class-wp-hook.php(311): WP_Hook::do_action
#5 /wp-includes/plugin.php(478): do_action
#4 /wp-settings.php(562): require_once
#3 /wp-config.php(108): require_once
#2 /wp-load.php(37): require_once
#1 /wp-blog-header.php(13): require
#0 /index.php(17): null

$country = $_POST['billing_country'];

@vyskoczilova
Copy link
Owner

@morvy Díky, přidáno, jdu to poslat na wordpress.org

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