[buddypress-trac] [BuddyPress] #4761: Modify ajax url to work properly with FORCE_SSL_ADMIN

buddypress-trac noreply at wordpress.org
Fri Mar 1 12:14:17 UTC 2013


#4761: Modify ajax url to work properly with FORCE_SSL_ADMIN
------------------------------------------+-----------------------
 Reporter:  will_c                        |       Owner:
     Type:  defect (bug)                  |      Status:  reopened
 Priority:  normal                        |   Milestone:  1.7
Component:  Core                          |     Version:  1.7
 Severity:  major                         |  Resolution:
 Keywords:  needs-patch needs-unit-tests  |
------------------------------------------+-----------------------
Changes (by boonebgorges):

 * keywords:  dev-feedback commit => needs-patch needs-unit-tests
 * status:  closed => reopened
 * resolution:  fixed =>
 * severity:  normal => major


Comment:

 The changes introduced in r6790 do not work properly when `BP_ROOT_BLOG`
 is defined as something other than the default. That's because
 `network_site_url()` will always point to the main site's wp-admin, and on
 some setups, some network users do not have access to any wp-admin URLs on
 the main site. Thus, for those users, AJAX requests fail silently. See
 #4850

 I'll try to figure something out (and write some tests so we don't get in
 this mess again). But in the meantime, if we can't figure out a solution
 we are confident with, we should revert r6790 for 1.7, since the HTTPS
 issue here is not a regression, but breaking AJAX for alternative root
 blog setups *is*.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4761#comment:8>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list