get_permalink() returns wrong link outside The Loop

[fmorel90]

I'm using wp_logout_url(get_permalink()) in a sidebar widget to let users logout and return to the current page.
When viewing a single post, get_permalink() returns the posts' correct link.

However, on my home page, on archive pages, and on search results pages, get_permalink() returns the link of the last post in The Loop used to display posts in the page. I've checked both my theme and the default theme to see if I'd screwed up The Loop, but the default theme does this as well.

I am unsure if it worked correctly in 2.7. It's been a few months since I played with WordPress and I just started again with 2.8 a few days ago. However, I'm sure I would have noticed this before, so I'm putting this as a bug in 2.8.

If I take out The Loop on my homepage, it returns the link to my latest post.

[ryan]

What version of 2.8? Some related bugs have been fixed recently. Could you try with the latest nightly or latest svn?

[fmorel90]

I'm using beta 2. I'm away from laptop right now, but I'll try an update tonight or tomorrow morning.

[fmorel90]

I just downloaded a nightly and retried several pages. I still have the same problem.

[ryan]

get_permalink() does not create links to the current page. It only creates links to posts. On a page with multiple posts, returning a link to the last post is about the only thing it can do in this context.

[ryan]

esc_url( wp_logout_url( $_SERVER['REQUEST_URI'] ) )

You probably need something more like that. get_permalink() is not intended to be used like this.

[fmorel90]

Thanks, I'll try that tomorrow when I have time at home. I'm still new with Wordpress and PHP.

[fmorel90]

Thank you very much. That worked perfectly.

[iLobster]

ryan: your solution may work for the specific presented problem, but I think there needs to be a uniform way to pull the current page's permalink, or more basically ID (with the id you can access the permalink) outside the loop without running custom queries such as ​here or relying on different combinations of $_SERVER which are not always uniform and reliable.

My test case is a self submitting form where I prefer not to leave the action attribute empty. As the page ID is processed in the beginning of each page, it would make sense to copy it to another veritable that doesn't get overridden by the loop

[johnbillion]

iLobster, get_permalink() is not the function you should be using in this case. What you're really asking for is a function which returns the current URL.

I had thought there was a ticket open suggesting a get_current_url() function, but it looks like I'm mistaken. The code is quite simple though:

$current_url = ( is_ssl() ? 'https://' : 'http://' ) . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];