About: Joanne Masterson

Yes —
This is also the link generated by either ‘index.php’ or home_url() — anything I put in the code to create the logout link gets appended with the query string.

There are several places to logout (menu, forum).

All logouts get redirected to the membership options page, instead of home.

Here it is — thanks for asking.

Around line 67, the code sets the logout_link to $wpurl (which is ‘index.php’). This gets overwritten to become the membership options page (and a buch of MOP variables) no matter what I do:

function wps_add_loginlogout_link($items, $args)
{
	//defaults; change this as you see fit
	$defaults = array(
		'loginlout_position' => 999, //enter 0 for front, 999 for end
		'menu_id' => 'menu-item', //custom CSS
		'menu_class' => 'menu-item menu-item-type-custom menu-item-object-custom', //custom CSS
		'menu_location' =>'primary-public', //enter primary, secondary, or topnav for News
		'loginredirect' => false //enter true for redirect to wp-admin dashboard
	);

	//do nothing if not proper menu location
	if( $args->theme_location != $defaults['menu_location'] )
		return $items;

	//set redirect URL
	if( $defaults['loginredirect'] )
		$wpurl = 'wp-admin/index.php';
	else
		$wpurl = 'index.php';

	// split the menu items into an array using the ending <li> tag
	if ( $defaults['loginlout_position'] != 0 && $defaults['loginlout_position'] != 1 && $defaults['loginlout_position'] != 999 ) {
		$items = explode('</li>',$items);
	}

	if(is_user_logged_in())
	{
		$newitem = '<li id="'.$defaults['menu_id'].'" class="'.$defaults['menu_class'].'"><a title="Logout" href="'. wp_logout_url($wpurl) .'">Logout</a></li>';
		if ( $defaults['loginlout_position'] == 0 || $defaults['loginlout_position'] == 1 )
			$newitems = $newitem.$items;
		elseif ( $defaults['loginlout_position'] == 999 )
			$newitems = $items.$newitem;
		else
			$newitem = '<li id="'.$defaults['menu_id'].'" class="'.$defaults['menu_class'].'">' . $args->before . '<a title="Logout" href="'. wp_logout_url('index.php') .'">' . $args->link_before . 'Logout' . $args->link_after . '</a>' . $args->after; // no </li> needed this is added later
	}
	else
	{
		$newitem = '<li id="'.$defaults['menu_id'].'" class="'.$defaults['menu_class'].'"><a title="Login" href="'. wp_login_url($wpurl) .'">Login</a></li>';
		if ( $defaults['loginlout_position'] == 0 || $defaults['loginlout_position'] == 1 )
			$newitems = $newitem.$items;
		elseif ( $defaults['loginlout_position'] == 999 )
			$newitems = $items.$newitem;
		else
			$newitem = '<li id="'.$defaults['menu_id'].'" class="'.$defaults['menu_class'].'">' . $args->before . '<a title="Login" href="'. wp_login_url('index.php') .'">' . $args->link_before . 'Login' . $args->link_after . '</a>' . $args->after; // no </li> needed this is added later
	}

	if ( $defaults['loginlout_position'] != 0 && $defaults['loginlout_position'] != 1 && $defaults['loginlout_position'] != 999 ) {
		$newitems = array();

		// loop through the menu items, and add the new link at the right position
		foreach($items as $index => $item)
		{
		// array indexes are always one less than the position (1st item is index 0)
			if($index == $defaults['loginlout_position']-1)
			{
				$newitems[] = $newitem;
			}
			$newitems[] = $item;
		}

		// finally put all the menu items back together into a string using the ending <li> tag and return
		$newitems = implode('</li>',$newitems);
	}

	return $newitems;
}
add_filter('wp_nav_menu_items', 'wps_add_loginlogout_link', 10, 2);

So clean and easy to read and find stuff. Good job.