WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#24989 closed defect (bug) (fixed)

Cleanup for `/wp-admin/js/cat.js`

Reported by: carldanley Owned by: ocean90
Priority: normal Milestone: 3.7
Component: General Version:
Severity: normal Keywords:
Cc: Focuses:

Description

Can this functionality be moved to a WP utilities file or something? Seems a little crazy to include a file for 5 lines of JS.

Attachments (2)

24989.patch​ (949 bytes ) - added by ocean90 13 years ago.
24989.2.patch​ (2.8 KB ) - added by carldanley 13 years ago.

Download all attachments as: .zip

Change History (6)

#1 @ocean90
13 years ago

  • Milestone Awaiting Review → 3.7

The handle of the file is ajaxcat. Seems like it's not even used in core.

There is also a typo in the ID selector: $('#jaxcat'). Coudn't find any reference to jaxcat or ajaxcat.

So, let's remove it?

#2 @rmccue
13 years ago

This was made redundant in r6588 as part of #5618 for 2.5. The last core usage was removed in r6695. (via git log -Sjaxcat -p)

+1 for removal.

@ocean90
13 years ago

@carldanley
13 years ago

#3 @helen
13 years ago

ocean90 - seems like you removed the bad ref from $_old_files but didn't replace it.

#4 @ocean90
13 years ago

  • Owner set to ocean90
  • Resolution → fixed
  • Status new → closed

In 25020:

Remove unused cat JavaScript file. props carldanley. fixes #24989.

Note: See TracTickets for help on using tickets.