Ensure beakers still hook in on AJAX requests #1

Merged
merged 1 commit into from Oct 13, 2012

Projects

None yet

3 participants

@rmccue

Autosuggestion is currently broken (as can be seen on wpaustralia.org), due to the autosuggest never hooking in. The problem is that requests to wp-admin/admin-ajax.php have is_admin() set true, so the beaker constructor exits before it hooks the AJAX hook in.

This pull request fixes that for all beakers (); if you'd prefer that to be fixed just for the autosuggest, I'll simply override the constructor in the subclass and resubmit the pull request.

@paulgibbs
Owner

Thanks very much for the patch. I'll check it out this week

@errorstudent

work perfectly with this patch (rmccue@916b63f), thanks @rmccue & @paulgibbs

@rmccue

@paulgibbs Had any chance to check this out? It's been live on wpaustralia.org and working :)

@paulgibbs
Owner

Thanks for the patch

@paulgibbs paulgibbs merged commit d9247cf into paulgibbs:master Oct 13, 2012
@rmccue

🍰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment