WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#25866 closed defect (bug) (fixed)

Ajaxurl should be defined in the customizer

Reported by: celloexpressions's profile celloexpressions Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 3.8 Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-patch
Focuses: Cc:

Description

Our documentation says that ajaxurl is defined everywhere in wp-admin. It is not in the customizer, and we need it for the suggest script for Twenty Fourteen's featured content tag name field. Very silly for themes to do this manually.

Attachments (1)

25866.diff​ (2.0 KB) - added by celloexpressions 11 years ago.
Define ajaxurl directly in the customizer <head>, as it is in the rest of the admin. Also remove the code that was adding it in Twenty Fourteen.

Download all attachments as: .zip

Change History (8)

#1 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 3.8

@celloexpressions
11 years ago

Define ajaxurl directly in the customizer <head>, as it is in the rest of the admin. Also remove the code that was adding it in Twenty Fourteen.

#2 @celloexpressions
11 years ago

  • Keywords has-patch added; needs-patch removed

#3 @lancewillett
11 years ago

  • Cc lancewillett added

#4 @rachelbaker
11 years ago

  • Cc rachel@… added

#6 @SergeyBiryukov
11 years ago

In 26082:

Define ajaxurl in the customizer.

props celloexpressions.
see #25866.

#7 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 26083:

Remove the code for setting ajaxurl from Twenty Fourteen.

props celloexpressions.
fixes #25866.

Note: See TracTickets for help on using tickets.