eclev91

Yep, did that, issue persisted.

Disabled “Default customer location” and that resolved the issue, which is to say, the label updated to say “Enter address for shipping options” (or something roughly equivalent to that). Re-enabled with “Geolocation (with page cache support)” and the issue came back. So I’m pretty sure that’s the issue.

In fact, once I re-enabled that, I even got it to figure out I was in Oregon. So now it has the State and Country, but that’s still not enough for it to generate delivery options.

@doublezed2 I’m working on a staging environment for you.

Hi @doublezed2 , I appreciate your recommendation, but we’re going to switch back to shortcodes at this time. We simply don’t have the bandwidth to troubleshoot an issue we’re having trouble recreating, and if your team is unaware of any larger issues, then this ticket at least makes a nice data point if your team has other reports. I have been keeping an eye on this – https://github.com/Automattic/woocommerce-payments/issues/9939

On the caching, I do have a couple thoughts:

• You said “the JavaScript responsible for handling the button click action is not working properly”. That could certainly be true, but given draft orders are successfully generated without billing addresses, it seems like JS-driven AJAX requests are working just fine. I think it’s more likely that there’s some kind of validation error on the billing fields (which shouldn’t be validated at all when the “use same address” checkbox is checked), and if there are inline validation messages, they’re hidden to the end user because the “use same address” checkbox is checked.
• Opening the site in an incognito window should give me the cache-iest of cached versions of the page, so you would think we would be able to reproduce if it were a cache issue.
• It’s unclear to me why checkout is so dependent on Javascript that it might fail completely. I would hope, given checkout is a single point of failure for eCommerce sites, that the checkout form would more degrade more gracefully.

Deprecated: strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in .../wp-includes/functions.php on line 7300
Call Stack
#	Time	Memory	Function	Location
1	0.0010	408416	{main}( )	.../index.php:0
2	0.0012	445136	require_once( '...wp-admin/admin.php )	.../index.php:10
3	0.3560	37487320	require( '...wp-admin/menu.php )	.../admin.php:158
4	0.8579	40405536	require_once( '...wp-admin/includes/menu.php )	.../menu.php:412
5	0.8596	40444528	do_action( $hook_name = 'admin_menu', ...$arg = variadic('') )	.../menu.php:161
6	0.8596	40444744	WP_Hook->do_action( $args = [0 => ''] )	.../plugin.php:517
7	0.8596	40444744	WP_Hook->apply_filters( $value = '', $args = [0 => ''] )	.../class-wp-hook.php:348
8	0.8623	40567392	Tribe__Tickets__Commerce__PayPal__Orders__Report->register_orders_page( '' )	.../class-wp-hook.php:324
9	0.8623	40567640	add_submenu_page( $parent_slug = NULL, $page_title = 'PayPal Orders', $menu_title = 'PayPal Orders', $capability = 'edit_posts', $menu_slug = 'tpp-orders', $callback = [0 => class Tribe__Tickets__Commerce__PayPal__Orders__Report { public $orders_page = NULL; public $orders_table = NULL }, 1 => 'orders_page_inside'], $position = ??? )	.../Report.php:163
10	0.8623	40567640	plugin_basename( $file = NULL )	.../plugin.php:1478
11	0.8623	40567640	wp_normalize_path( $path = NULL )	.../plugin.php:769
12	0.8623	40567640	wp_is_stream( $path = NULL )	.../functions.php:2182
13	0.8623	40567640	strpos( $haystack = NULL, $needle = '://' )	.../functions.php:7300

This is me copying and pasting the XDebug output. But I’ve linked the line in your source code that is problematic. Regardless of whether or not WordPress gracefully handles what comes next (see above – it does not), you’re calling add_submenu_page in an unexpected and unintended way.

FWIW, the workarounds in the linked article are more complicated than they need to be. It has nothing to do with space or semi-colons. Just getting the browser to repaint the field will fix it. I went ahead and hooked into admin_print_scripts to get this working for us in the meantime:

add_action('admin_print_scripts', function() {
    ?>
    <script>
        document.addEventListener('DOMContentLoaded', function() {
        var emailInput = document.getElementById('customer_email');
        if (emailInput && navigator.userAgent.indexOf('Safari') != -1 && navigator.userAgent.indexOf('Chrome') == -1) {
            var emails = emailInput.value;
            emailInput.value = emails;
        }
    });
    </script>
    <?php 
});

• This reply was modified 1 year, 11 months ago by eclev91.