Make WordPress Core

Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#2282 closed defect (bug) (duplicate)

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

Reported by: abhay's profile abhay Owned by: abhay's profile 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 19 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)

#1 @zamoose
19 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.

#2 @abhay
19 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.

@abhay
19 years ago

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

#3 @abhay
19 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.

#4 @westi
19 years ago

  • Keywords bg|commit added

Patch looks good to me.

Marking for commit

#5 @abhay
19 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

#6 @(none)
18 years ago

  • Milestone 2.0.1 deleted

Milestone 2.0.1 deleted

Note: See TracTickets for help on using tickets.