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 525522 - Some composer header signals aren't hooked up (Window title doesn't reflect subject, ...)
Some composer header signals aren't hooked up (Window title doesn't reflect s...
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.22.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
evolution[composer]
: 518104 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-04-01 10:49 UTC by Michel Dänzer
Modified: 2008-04-03 16:33 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
Fix (2.88 KB, patch)
2008-04-01 10:52 UTC, Michel Dänzer
committed Details | Review

Description Michel Dänzer 2008-04-01 10:49:05 UTC
Please describe the problem:
subject_changed_cb and hdrs_changed_cb aren't hooked up anywhere and are thus never called. Apart from the most obvious symptom described below, this also causes the composer not to be considered dirty after changes to headers.

I'll attach a patch which fixes this.

Steps to reproduce:
1. File -> New -> Mail Message (Ctrl + N)
2. Modify the subject


Actual results:
The composer window title remains 'Compose Message'.

Expected results:
The composer window title should reflect the current subject.

Does this happen every time?
Yes.

Other information:
Comment 1 Michel Dänzer 2008-04-01 10:52:04 UTC
Created attachment 108406 [details] [review]
Fix

This fixes the problems for me.
Comment 2 Matthew Barnes 2008-04-01 11:29:38 UTC
*** Bug 518104 has been marked as a duplicate of this bug. ***
Comment 3 Milan Crha 2008-04-02 17:37:08 UTC
Good catch. Patch looks good and works fine, so just add a Changelog entry to the composer directory and commit to stable and trunk. Thanks.
Comment 4 Michel Dänzer 2008-04-02 21:26:18 UTC
(In reply to comment #3)
> Good catch. Patch looks good 

Thanks for the review.

> and works fine, so just add a Changelog entry to the composer directory 

Do I need to do that?

> and commit to stable and trunk.

Will the fix also be backported to 2.22, as this is a regression from 2.12?
Comment 5 Matthew Barnes 2008-04-02 21:33:54 UTC
Note: Bug is not present in the new composer code for 2.23.
Comment 6 André Klapper 2008-04-02 23:39:12 UTC
should get into Evolution 2.22.1, yepp. anybody feel free to commit.
Comment 7 Milan Crha 2008-04-03 13:40:25 UTC
Committed to gnome-2-22. Committed revision 35315.

I did write a Changelog entry for you (for this time), just do not blame if you want there something else :)
Comment 8 Michel Dänzer 2008-04-03 16:33:10 UTC
(In reply to comment #7)
> I did write a Changelog entry for you (for this time), just do not blame if you
> want there something else :)

:) Thanks! I'll try to remember including changelog entries in the future.