Opened 14 years ago
Closed 14 years ago
#4263 closed enhancement (fixed)
Maybe update bp_core_set_uri_globals() for AJAX detection
| Reported by: | DJPaul | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.6 |
| Component: | Core | Version: | |
| Severity: | normal | 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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.