[buddypress-trac] [BuddyPress Trac] #7840: bp_is_messages_compose_screen not working with BP Nouveau
buddypress-trac
noreply at wordpress.org
Mon May 21 09:58:57 UTC 2018
#7840: bp_is_messages_compose_screen not working with BP Nouveau
-------------------------------------------------+-------------------------
Reporter: sbrajesh | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.0.1
Component: Templates | Version: 3.0.0
Severity: normal | Resolution:
Keywords: has-patch needs-testing reporter- |
feedback |
-------------------------------------------------+-------------------------
Comment (by imath):
Thanks @DJPaul
I think our trouble is more linked to the fact, you won't get the fragment
using `$_SERVER['REQUEST_URI']` for instance. So I took a deeper look at
the Backbone history object. And we could do 7840.pushState.patch. The
main trouble is that in this case we need to make sure a trailing slash is
not added to the url within `bp_redirect_canonical()`. This also means, we
should remove this trailing slash if the URL contains one. I'm going to
look further on this road.
I think, It would do part of the job, but for @sbrajesh trouble, the
template redirect hook won't be fired if the click comes from the UI.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7840#comment:13>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list