Woocommerce Product Qty Button - Possible Bug

[antforager]

I have been seeing a couple errors in my logs. I think this is related to this change: 13ae924#diff-f0aa8d9fc7d54f3f6b7d108f02942537782aa9abd84f169e4ca0075e584e35b0. It looks like there is an 'input_value" introduced and it sometimes doesn't exist.

First one:
Crashing here: https://github.com/bootscore/bootscore/blob/main/woocommerce/inc/wc-qty-btn.php#L31

Warning: Undefined array key "input_value" in /var/www/wp-content/themes/bootscore/woocommerce/inc/wc-qty-btn.php on line 31

Stack Trace
1.	
{closure}(Array(12))
/var/www/wp-includes/class-wp-hook.php:326
2.	
WP_Hook->apply_filters(Array(12), Array(2))
/var/www/wp-includes/plugin.php:205
3.	
apply_filters('woocommerce_quantity_i...', Array(12), WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/includes/wc-template-functions.php:4476
4.	
wc_get_quantity_input_args(Array(0), WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Utilities/QuantityLimits.php:53
5.	
Automattic\WooCommerce\StoreApi\Utilities\QuantityLimits->get_add_to_cart_limits(WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Schemas/V1/ProductSchema.php:569
6.	
Automattic\WooCommerce\StoreApi\Schemas\V1\ProductSchema->get_item_response(WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Routes/V1/Products.php:81
7.	
Automattic\WooCommerce\StoreApi\Routes\V1\Products->get_route_response(WP_REST_Request)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Routes/V1/AbstractRoute.php:126
8.	
Automattic\WooCommerce\StoreApi\Routes\V1\AbstractRoute->get_response_by_request_method(WP_REST_Request)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Routes/V1/AbstractRoute.php:98
9.	
Automattic\WooCommerce\StoreApi\Routes\V1\AbstractRoute->get_response(WP_REST_Request)
/var/www/wp-includes/rest-api/class-wp-rest-server.php:1292
10.	
WP_REST_Server->respond_to_request(WP_REST_Request, '/wc/store/products', Array(8), null)
/var/www/wp-includes/rest-api/class-wp-rest-server.php:1125
11.	
WP_REST_Server->dispatch(WP_REST_Request)
/var/www/wp-includes/rest-api/class-wp-rest-server.php:439
12.	
WP_REST_Server->serve_request('/wc/store/products')
/var/www/wp-includes/rest-api.php:459
13.	
rest_api_loaded(WP)
/var/www/wp-includes/class-wp-hook.php:324
14.	
WP_Hook->apply_filters(null, Array(1))
/var/www/wp-includes/class-wp-hook.php:348
15.	
WP_Hook->do_action(Array(1))
/var/www/wp-includes/plugin.php:565
16.	
do_action_ref_array('parse_request', Array(1))
/var/www/wp-includes/class-wp.php:418
17.	
WP->parse_request('')
/var/www/wp-includes/class-wp.php:818
18.	
WP->main('')
/var/www/wp-includes/functions.php:1342
19.	
wp()
/var/www/wp-blog-header.php:16
20.	
require('/var/www/wp-blog-heade...')
/var/www/index.php:17

Second one:
Crashing here: https://github.com/bootscore/bootscore/blob/main/woocommerce/inc/wc-qty-btn.php#L37

Warning: Undefined array key "input_value" in /var/www/wp-content/themes/bootscore/woocommerce/inc/wc-qty-btn.php on line 37

Last 24 Hours
Stack Trace
1.	
{closure}(Array(12))
/var/www/wp-includes/class-wp-hook.php:326
2.	
WP_Hook->apply_filters(Array(12), Array(2))
/var/www/wp-includes/plugin.php:205
3.	
apply_filters('woocommerce_quantity_i...', Array(12), WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/includes/wc-template-functions.php:4476
4.	
wc_get_quantity_input_args(Array(0), WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Utilities/QuantityLimits.php:53
5.	
Automattic\WooCommerce\StoreApi\Utilities\QuantityLimits->get_add_to_cart_limits(WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Schemas/V1/ProductSchema.php:569
6.	
Automattic\WooCommerce\StoreApi\Schemas\V1\ProductSchema->get_item_response(WC_Product_Variation)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Routes/V1/Products.php:81
7.	
Automattic\WooCommerce\StoreApi\Routes\V1\Products->get_route_response(WP_REST_Request)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Routes/V1/AbstractRoute.php:126
8.	
Automattic\WooCommerce\StoreApi\Routes\V1\AbstractRoute->get_response_by_request_method(WP_REST_Request)
/var/www/wp-content/plugins/woocommerce/src/StoreApi/Routes/V1/AbstractRoute.php:98
9.	
Automattic\WooCommerce\StoreApi\Routes\V1\AbstractRoute->get_response(WP_REST_Request)
/var/www/wp-includes/rest-api/class-wp-rest-server.php:1292
10.	
WP_REST_Server->respond_to_request(WP_REST_Request, '/wc/store/products', Array(8), null)
/var/www/wp-includes/rest-api/class-wp-rest-server.php:1125
11.	
WP_REST_Server->dispatch(WP_REST_Request)
/var/www/wp-includes/rest-api/class-wp-rest-server.php:439
12.	
WP_REST_Server->serve_request('/wc/store/products')
/var/www/wp-includes/rest-api.php:459
13.	
rest_api_loaded(WP)
/var/www/wp-includes/class-wp-hook.php:324
14.	
WP_Hook->apply_filters(null, Array(1))
/var/www/wp-includes/class-wp-hook.php:348
15.	
WP_Hook->do_action(Array(1))
/var/www/wp-includes/plugin.php:565
16.	
do_action_ref_array('parse_request', Array(1))
/var/www/wp-includes/class-wp.php:418
17.	
WP->parse_request('')
/var/www/wp-includes/class-wp.php:818
18.	
WP->main('')
/var/www/wp-includes/functions.php:1342
19.	
wp()
/var/www/wp-blog-header.php:16
20.	
require('/var/www/wp-blog-heade...')
/var/www/index.php:17

[crftwrk]

Please check #1103 and let me know if that fixes this bug.

[antforager]

Wow! You guys are one step ahead of me!

So, here's my situation: I had to roll back to 6.21 because of the GoDaddy conflict when you try to remove something from the mini cart and you get the divide by zero exception. (https://github.com/orgs/bootscore/discussions/1057).

I have been so busy lately, I put that on the back burner. I think I might pull down 6.21 and cherry pick [87aeb43] and add it to my theme. Actually, I'm not sure I know how to make a wordpress theme zip file. Do I just zip the files up?

[crftwrk]

You can simply replace the changed file in your theme or download the PR branch https://github.com/neralex/bootscore/archive/refs/heads/woocommerce-quantity-php8-fix.zip, unzip, rename folder to bootscore, zip and upload file in the backend. That's it.