Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#11296 closed defect (bug) (fixed)

Impossible to cancel deletion of new Categories/Tags

Reported by: caesarsgrunt's profile caesarsgrunt Owned by: azaozz's profile azaozz
Milestone: 2.9 Priority: normal
Severity: normal Version: 2.9
Component: JavaScript Keywords: has-patch commit
Focuses: Cc:

Description

  1. Create a new Category.
  2. Without reloading the page, click the Delete link for the new Category.
  3. In the JS confirm() dialog, click Cancel.
  4. The Category is deleted anyway'''

Also applies to Tags.

Trunk (12304).

Attachments (1)

11296.diff (486 bytes) - added by caesarsgrunt 14 years ago.

Download all attachments as: .zip

Change History (6)

#1 @caesarsgrunt
14 years ago

Sorry about the formatting there.

Also, should have said; this is in Firefox 3.5.5. Haven't tested other browsers yet, but it seems likely to be a simple JS bug which would affect all browsers.

@caesarsgrunt
14 years ago

#2 @caesarsgrunt
14 years ago

  • Keywords has-patch commit added

11269.diff fixes the issue.

Seems I was wrong about Tags; it's Categories and Link Categories which are affected by this.

The patch fixes both.

#3 @azaozz
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [12313]) Fix canceling when deleting newly added categories and link categories, fixes #11296

#4 follow-up: @azaozz
14 years ago

Good catch, thought I fixed this some time ago. Better to use jQuery.live() there to avoid dealing with multiple repeating events.

#5 in reply to: ↑ 4 @caesarsgrunt
14 years ago

Replying to azaozz:

Good catch, thought I fixed this some time ago. Better to use jQuery.live() there to avoid dealing with multiple repeating events.

I've never come across jQuery.live() before, actually. I didn't think there were any core jQuery functions I didn't know about! Looks good; thanks for telling me. :-)

Note: See TracTickets for help on using tickets.