Wrapper Function for getting the currently displayed page url.

[chrisclayton]

We need a template function for returning the current page url as get_permalink() doesn't work with buddypress generated pages.

See here for history: http://buddypress.org/community/groups/how-to-and-troubleshooting/forum/topic/current-url/

[chrisclayton]

Not finished, needs work and advice.

[DJPaul]

I think the idea might have merit, but for implementation, I'd reconstruct the current URL through the $bp->canonical_stack. Maybe something like bp_rel_canonical() which I put into trunk a couple weeks ago.

[boonebgorges]

What about wp_guess_url()?

[chrisclayton]

Agreed. Knew there was a reason i wanted advice :) Don't think i was too confident in that patch.

Haven't tested it and i could be reading it wrong, but looking through the bp_rel_canonical function (thanks btw, somehow i missed that update) it already seems to detect every page possible so, wouldn't we just be repeating the same code in both? (or am i reading it wrong?)

Wouldn't moving the url building stuff into bp_get_permalink() and returning the url_stack and then changing bp_rel_canonical() to simply just echo the bp_get_permalink()?

echo "<link rel='canonical' href='" . bp_get_permalink() . "' />\n";

Work perfectly, while keeping the DRY (Dont Repeat Yourself) principle intact?

[boonebgorges]

Yes, you're right that the canonical logic should be separated out and not repeated. bp_get_permalink() may not be the best name for such a function, though, as it's not really parallel to get_permalink() (which is used inside of the post loop or can be fed a post/post_id, and does not refer to the current page).

[chrisclayton]

Yeah, i was just using bp_get_permalink to refer to it by until i came up with something better. :)

[boonebgorges]

(In [5985]) Some canonical URL reconfiguration:

• Introduces bp_get_canonical_url() and bp_get_requested_url(), to abstract some commonly used canonical logic
• Provides an easy way for plugins to access the current canonical URL, using bp_get_canonical_url(). Fixes #4141