function wp_logout_url urlencode causes issues in items wrap nav walker.

[BackuPs]

Hi

The function wp_logout_url returns a encoded logout url. This used to work before without any issues.

string(128) "http://wprtdev.tst/wp-login.php?action=logout&_wpnonce=72f1be58e4&redirect_to=http%3A%2F%2Fwprtdev.tst%2F%3Floggedout%3Dtrue"

But if you parse this url as a list element in the menu walker you get a warning and error

<b>Warning</b>:  sprintf(): Too few arguments in <b>/var/docs/wprtdev.tst/public/wp-includes/nav-menu-template.php</b> on line <b>261</b><br />

because the walker uses %3$s for the items wrap and the encoded url has %3Afor : and %3F for ?

It is not possible to add a logout url into the items wrap like this

'items_wrap'=>'<ul id="%1$s" class="%2$s">%3$s<li id="woo-menu-logon-main" class="menu-item menu-item-type-post_type"><a href="http://wprtdev.tst/wp-login.php?action=logout&amp;_wpnonce=72f1be58e4&amp;redirect_to=http%3A%2F%2Fwprtdev.tst%2Fblog%3Floggedout%3Dtrue" title="Logout from shop" itemprop="url"><i class="wooicon icon icon-exit"></i>Logout</a></li>',

How to fix this?

Thank you !

[BackuPs]

To add more details to this.

Your own coding example adds a extra list element this way at wordpress.org

<?php
wp_nav_menu( array(
    'theme_location' => 'primary',
    'items_wrap'     => '<ul><li id="item-id"><?php __( 'Menu:', 'textdomain' ); ?></li>%3$s</ul>'
) );

To be found here. ​https://developer.wordpress.org/reference/functions/wp_nav_menu/

However if you add a extra menu item with a url that url cannot be encoded because the sprintf() function uses identifiers and a encoded url has codes that look like a identifier.

So sprintf also acting on %3F and the other ones in the encoded url causing the sprintf function to fail

How to work around this?

[audrasjb]

Hi @BackuPs thanks for the ticket and sorry for the delay,

Could you provide an example of the code used to generate your menu? so we could try to reproduce the issue.

Thanks,
Jb

[BackuPs]

Hi

The example is in my initial report of the issue.

Just try the following. But try this

<?php
$logout_url=wp_logout_url();

// now call 

                return wp_nav_menu( array(
                                                'theme_location' => 'top-menu',
                                                'container' => 'nav',
                                                'container_id' => 'navigation',
                                                'container_class' => 'jqueryslidemenu',
                                                'menu_id' => 'top-bar-menu',   
                                                'fallback_cb' => '',
                                                'items_wrap'=>'<ul id="%1$s" class="%2$s">%3$s<li id="woo-menu-logon-xxxx" class="menu-item menu-item-type-post_type"><a href="'.$logout_url.'" title="Logout from shop">Logout</a></li></ul>',
                                        ));

Note: Change the theme_location to the correct menu location !