Context Navigation

← Previous Ticket
Next Ticket →

#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

Attachment 24989.patch added

@carldanley
13 years ago

Attachment 24989.2.patch added

#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.

Download in other formats:

Make WordPress Core

Context Navigation

#24989 closed defect (bug) (fixed)

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

Description

Attachments (2)

Change History (6)

#1 @ocean90 13 years ago

#2 @rmccue 13 years ago

@ocean90 13 years ago

@carldanley 13 years ago

#3 @helen 13 years ago

#4 @ocean90 13 years ago