Adium

Ticket #9511 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Crash when selecting "Undo Typing" or "Redo Typing" after closing a conversation tab

Reported by: 4ndy Owned by: evands
Milestone: Adium 1.2.4 Component: Adium UI
Version: 1.2.3 Severity: normal
Keywords: crash undo redo tab Cc:
Patch Status:

Description

After typing text into a conversation tab, if the tab is then closed, the "Undo Typing" option (or "Redo Typing" if "Undo Typing" action has been performed) is still available. If selected, or if the hotkey (command-z) is used, Adium will crash.

Note: The"Undo" option is not available if the last tab (and therefore the window) has been closed.

Using Adium 1.2.3, OS X 10.4.11, AOL Instant Messenger account.

Attachments

Adium.crash.log (93.1 KB) - added by 4ndy 2 years ago.

Change History

Changed 2 years ago by 4ndy

Changed 2 years ago by Robby

  • milestone set to Adium X 1.2.4

I can reproduce this in 1.2.3.

Changed 2 years ago by Robby

and in 1.3svn, also.

Changed 2 years ago by jas8522

  • milestone changed from Adium X 1.2.4 to Adium X 1.2.5

We've got enough for 1.2.4 already, and this is not a regression - it's been around for a while, so no need to hold up an imminent release for it.

Changed 2 years ago by evands

  • owner changed from nobody to evands
  • status changed from new to assigned
  • milestone changed from Adium X 1.2.5 to Adium X 1.2.4

Yup, but it offended my tired sensibilities ;)

Changed 2 years ago by evands

  • status changed from assigned to closed
  • resolution set to fixed

(In [22966]) Each message view controller now provides a unique undo manager to its message entry text view. If we don't do this, we end up with a single undo manager for the whole window per the documentation on the undo manager hierarchy.

This fixes a number of obscure undo/redo bugs in the message window, including crashes after closing a tab and invoking undo/redo (fixes #9511) and undo/redo working across multiple tabs (only one of which is visible) which was just weird.

Changed 2 years ago by evands

(In [22967]) Merged [22966]: Each message view controller now provides a unique undo manager to its message entry text view. If we don't do this, we end up with a single undo manager for the whole window per the documentation on the undo manager hierarchy.

This fixes a number of obscure undo/redo bugs in the message window, including crashes after closing a tab and invoking undo/redo (fixes #9511) and undo/redo working across multiple tabs (only one of which is visible) which was just weird.

Note: See TracTickets for help on using tickets.