After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 597049 - "<email@example.com>" trimmed to ">"
"<email@example.com>" trimmed to ">"
Status: RESOLVED OBSOLETE
Product: empathy
Classification: Core
Component: Chat themes
2.28.x
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
Depends on: 599640
Blocks:
 
 
Reported: 2009-10-01 21:20 UTC by Antono Vasiljev
Modified: 2010-01-18 10:59 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed fix (2.49 KB, patch)
2009-11-24 17:32 UTC, Gustavo Noronha (kov) 		rejected Details | Review

Description Antono Vasiljev 2009-10-01 21:20:27 UTC 
Virtually every Adium theme has such bug:

Steps:

Try to sent "<test@email.com>" and you'll see only ">" 
Buddy receives as expected "<test@email.com>".
The same bug occurs if someone sends you "<test@email.com>".

Everything ok with every theme itself (checked on Adium).
Everything ok with default Empathy theme.
I can see "<test@email.com>" in logs and system notification bubbles.
Comment 1 Guillaume Desmottes 2009-10-02 10:47:06 UTC 
Can reproduce with master.
Comment 2 Gustavo Noronha (kov) 2009-11-24 17:32:02 UTC 
Created attachment 148404 [details] [review]
proposed fix
Comment 3 Xavier Claessens 2009-11-24 17:49:09 UTC 
This bug is almost fixed in my "escape" branch. I added test cases for this.

The remaining issue is the "<" is made part of the email addresse.

http://git.collabora.co.uk/?p=user/xclaesse/empathy.git;a=commitdiff;h=b253440c84091dd88e619171330e895507abe40d

I make this bug depends on bug #599640
Comment 4 Xavier Claessens 2009-11-24 17:49:41 UTC 
Review of attachment 148404 [details] [review]:

This is done properly in the "escape" branch.
Comment 5 Gustavo Noronha (kov) 2009-11-24 18:04:13 UTC 
Thanks Xavier! I rebased my work on your branch, and pushed it here: http://git.collabora.co.uk/?p=user/kov/empathy.git;a=shortlog;h=refs/heads/escape-gt-lt

Want to merge that into your branch? Anything you would like me to change?
Comment 6 Omer Akram 2010-01-17 12:18:45 UTC 
Its fixed in 2.29.5 please mark this fixed