Adium

Ticket #10525 (closed defect: fixed)

Opened 4 months ago

Last modified 4 months ago

/me breaks emotification

Reported by: mathuaerknedam Assigned to: earthmkii
Priority: lowest Milestone: Adium 1.3
Component: Message View Version: 1.2.7
Severity: minor Keywords:
Cc: Patch: None
Pending: 0

Description

In r24468...

"/me :)", "/me :)", and "/me :) " all get displayed without emotification. Just to remove possible ambiguity, that's "/me<space>:)", "/me<space><space>:)", and "/me<space>:)<space>", or a smiley with no surrounding space, a smiley with a preceding space, and a smiley with a trailing space.

A smiley with space on space on both sides is emotified properly. All four get emotified properly when sent outside of /me.

Change History

07/22/2008 09:38:28 AM changed by jas8522

  • owner set to earthmkii.
  • priority changed from normal to lowest.
  • severity changed from normal to regression.
  • milestone set to Adium X 1.3.

sholt: up your alley?

I assume this wasn't the case in 1.2.7

07/22/2008 09:39:24 AM changed by jas8522

Wait... I would expect anything following /me to not be emotified anyway...

07/22/2008 09:57:37 AM changed by earthmkii

Ok I've checked this out, and it's definitely a bug in our emoticon replacement, seems related to #6440.

07/23/2008 01:14:44 PM changed by earthmkii

  • status changed from new to assigned.
  • version changed from 1.3svn to 1.2.7.

I can confirm this on 1.2.7

07/23/2008 01:38:12 PM changed by sholt

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

(In [24481]) Quick and dirty fix for emoticons bordered by * chars. Ugly ugly, but worky. Fixes #10525

07/23/2008 02:04:10 PM changed by Robby

  • milestone changed from Adium X 1.3 to SVN issues.

07/23/2008 02:08:04 PM changed by earthmkii

  • severity changed from regression to minor.
  • milestone changed from SVN issues to Adium X 1.3.

This bug exists in Adium 1.2.7.