Opened 13 years ago
Closed 13 years ago
#4263 closed enhancement (fixed)
Maybe update bp_core_set_uri_globals() for AJAX detection
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 1.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Core | Keywords: | |
Cc: |
Description
I noticed bp_core_set_uri_globals() has an "Ajax or not?" check. Do we need to update this for 1.6 now that the AJAXURL has changed to admin-ajax.php? I'm not sure what the purpose of this check is.
Attachments (1)
Change History (4)
Note: See
TracTickets for help on using
tickets.
I think we need this check, because proper path detection is necessary for determining current component, action, etc. Also, I think we need the wp-load.php logic for backward compatibility (themes that still register the old version of ajaxurl). But our primary check should be for WP's DOING_AJAX. See attached patch.