WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#2282 closed defect (bug) (duplicate)

Ability to add categories on the fly in post.php lost [from: wp-testers]

Reported by: abhay Owned by: abhay
Milestone: Priority: normal
Severity: major Version: 2.0
Component: Administration Keywords: duplicate
Focuses: Cc:

Description

This seems to have happenned at revision: [3422] when the javascript was moved out of admin-header and into separate files.

Attachments (1)

move-sack.patch (669 bytes) - added by abhay 9 years ago.
move variable that eventually calls sack if list_js or cat_js is being used.

Download all attachments as: .zip

Change History (7)

comment:1 @zamoose9 years ago

I can verify this issue across 4 WP 2.0.1-alpha blogs, all running different (or, in one case, no) plugins, meaning that this isn't likely due to outside forces.

comment:2 @abhay9 years ago

  • Keywords bg|has-patch added
  • Owner changed from anonymous to abhay

Looks like a simple enough fix. SACK is not being called. Just had to move one if statement. Patch has been attached.

@abhay9 years ago

move variable that eventually calls sack if list_js or cat_js is being used.

comment:3 @abhay9 years ago

  • Keywords admin-header sack-js added; cat-js removed

Just wanted to change the tags but also wanted to mention how awful the description is in that patch of mine. My apologies.

Please add commit tag after checking.

comment:4 @westi9 years ago

  • Keywords bg|commit added

Patch looks good to me.

Marking for commit

comment:5 @abhay9 years ago

  • Keywords duplicate added; admin-header sack-js wp-testers bg|has-patch bg|commit removed
  • Resolution set to duplicate
  • Status changed from new to closed

ugh. looks like donncha had marked this patch already but no one set commit. i'll defer to the earlier ticket #2262

comment:6 @anonymous8 years ago

  • Milestone 2.0.1 deleted

Milestone 2.0.1 deleted

Note: See TracTickets for help on using tickets.