Adium

Opened 15 years ago

Closed 15 years ago

Last modified 11 years ago

#468 closed defect (fixed)

Growl Notification and Contact Alert Awayness

Reported by: spam@weks.net Owned by: somebody
Milestone: Adium 1.3 Component: Adium Core
Version: Severity: normal
Keywords: Cc: evan@…
Patch Status:

Description

I currently have the following set (and no others) on a user:

  • Contact returns from away
    • Display a Growl notification
    • Play the sound "..."
  • Contact signs on
    • Display a Growl notification
    • Play the shound "..."

When the user sets herself away, I get a Growl notification that she "is back." No.. no, she's not.

Change History (14)

comment:1 Changed 15 years ago by Evan Schoenberg

Milestone: Adium X 0.86

comment:2 Changed 15 years ago by Evan Schoenberg

Should confirm and fix if appropriate for 0.86

comment:3 Changed 15 years ago by Diatribe

I have something similar happening, which sounds like the same problem.

  1. I get a "returned from idle" right before a contact goes away.
  1. I get a "contact came back" right before he signs off or goes away.

I get these events even though they are wrong and I get them independent of the event system used (i.e. Growl, sounds, etc.)

Oh, and it seems that #556 is a dupe of this.

comment:4 Changed 15 years ago by Chris Forsythe

field_haspatch: 0
Milestone: Adium X 0.87Adium X 1.0

.8x is only for security issues now, moving to 1.0

comment:5 Changed 15 years ago by Colin Barrett

Cc: evan@… added
Owner: changed from anybody to Colin Barrett

This sounds like an issue with the event coalescing. Evan, think you could find a couple minutes to explain that to me again?

comment:6 Changed 15 years ago by Evan Schoenberg

The OP's description doesn't sound like desired behavior at all -- the away event is being displayed as a returned event. Straight bug... haven't seen it myself. Does it really exist?

Diatribe's bug above sounds like it might be protocol-specific.. for example, if a contact can't be both idle and away and went from idle to away, perhaps idle is being cleared and then away set. The 'contact came back' is from the away status being cleared when the contact signs off... should be suppressed somehow.

And as far as the actual event coalescing which was supposed to be able to suppress a 'contact went away' in favor of the away message if possible... I'm not convinced it works at all, especially with the delay from AIM we have to deal with now ;)

comment:7 Changed 15 years ago by Colin Barrett

There's a delay from AIM? And it's too bad that the coalescing doesn't work; that was a neat feature.

comment:8 Changed 15 years ago by Colin Barrett

Owner: changed from Colin Barrett to Ivan

comment:9 Changed 15 years ago by Chris Forsythe

Milestone: Adium X 1.0
Resolution: worksforme
Status: newclosed

Can't duplicate, closing.

comment:10 Changed 12 years ago by Zachary West

Resolution: worksformefixed

(In [23990]) Fix the world's most oddly placed bang, fixing a longstanding issue where the wrong event would fire for a contact. Fixes #468. Fixes #5551.

comment:11 Changed 12 years ago by Zachary West

Milestone: Adium X 1.3
Patch Status: None
pending: 0

Nothing like the satisfaction of fixing your own 3 year old ticket.

comment:12 Changed 12 years ago by Zachary West

(In [23997]) Fix the case of a metacontact which was available on connect going away causing the "come back" event to be fired. This would occur when the cache had a nil-entry and an event fires causing the cache to be updated before the meta contact is fully away, which triggers the "came back" event. Refs #468.

comment:13 Changed 12 years ago by Evan Schoenberg

(In [24152]) Backported [23997]: Fix the case of a metacontact which was available on connect going away causing the "come back" event to be fired. This would occur when the cache had a nil-entry and an event fires causing the cache to be updated before the meta contact is fully away, which triggers the "came back" event. Refs #468.

Backported [23990]: Fix the world's most oddly placed bang, fixing a longstanding issue where the wrong event would fire for a contact. Fixes #468. Fixes #5551.

comment:14 Changed 11 years ago by Zachary West

Component: Core AdiumAdium Core
Note: See TracTickets for help on using tickets.