[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