WordPress.org

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#8653 closed defect (bug) (invalid)

[PATCH] wp-admin/js/common.js uses wpAjax() but does not specify wp-ajax-response as a dependency

Reported by: projct Owned by: azaozz
Milestone: Priority: normal
Severity: normal Version: 2.7
Component: JavaScript Keywords: reporter-feedback needs-testing
Focuses: Cc:

Description

Form validation code and some other bits in common.js require wp-ajax-response.js for the wpAjax() call. Does not currently manifest as a javascript error but the dependency graph is wrong.

Attachments (1)

wp-ajax-response.diff (707 bytes) - added by projct 12 years ago.

Download all attachments as: .zip

Change History (5)

#1 @projct
12 years ago

  • Summary changed from wp-admin/js/common.js uses wpAjax() but does not specify wp-ajax-response as a dependency to [PATCH] wp-admin/js/common.js uses wpAjax() but does not specify wp-ajax-response as a dependency

#2 @DD32
12 years ago

  • Keywords has-patch added
  • Milestone set to 2.8

#3 @Denis-de-Bernardy
12 years ago

  • Component changed from Administration to JavaScript
  • Keywords reporter-feedback needs-testing added; has-patch removed
  • Owner changed from anonymous to azaozz

patch is broken, probably invalid, too.

#4 @azaozz
12 years ago

  • Milestone 2.8 deleted
  • Resolution set to invalid
  • Status changed from new to closed

Currently wp-ajax-response.js is not needed for common.js.

Note: See TracTickets for help on using tickets.