Opened 4 years ago

Closed 4 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
Priority: normal Milestone:
Component: JavaScript Version: 2.7
Severity: normal Keywords: reporter-feedback needs-testing
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 4 years ago.

Download all attachments as: .zip

Change History (5)

projct — 4 years ago

comment:1   projct — 4 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

comment:2   DD32 — 4 years ago

  • Keywords has-patch added
  • Milestone set to 2.8

comment:3   Denis-de-Bernardy — 4 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.

comment:4   azaozz — 4 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.