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 442187 - e_cal_open_default does not open default memo componant
e_cal_open_default does not open default memo componant
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: Memos
1.10.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-calendar-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2007-05-29 23:45 UTC by John Stowers
Modified: 2007-06-03 12:57 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
Sample program to illustrate the problem (2.24 KB, text/plain)
2007-05-29 23:46 UTC, John Stowers
  Details
Possible fix for the problem (406 bytes, patch)
2007-05-29 23:47 UTC, John Stowers
committed Details | Review

Description John Stowers 2007-05-29 23:45:35 UTC
Please describe the problem:
Title says it all. Sample program illustrating the problem attached, along with possible patch to fix the issue.

patch credit to varadhan



Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 John Stowers 2007-05-29 23:46:25 UTC
Created attachment 89028 [details]
Sample program to illustrate the problem
Comment 2 John Stowers 2007-05-29 23:47:11 UTC
Created attachment 89029 [details] [review]
Possible fix for the problem
Comment 3 Karsten Bräckelmann 2007-05-29 23:52:37 UTC
<nzjrs_> the patch applies equally well against 1.10 as it does against 1.12 but neither work corrrectly
Comment 4 Karsten Bräckelmann 2007-05-30 00:07:18 UTC
/me sighs

Maybe there's an issue with the locally patched e-d-s.

Futher comments as per IRC:  "by caling the same sequence of calls that the patch does, from my application, it now works, hence I determined that the patch would work"  "i.e. me manually calling e_cal_new_system_memos (); (as the patch does) instead of going via get_default() work perfectly"


Adding Varadhan to the Cc list. Leaving this up to you guys. :)  Feel free to accept the patch anyway, if it turns out it actually works despite comment 3.
Comment 5 Karsten Bräckelmann 2007-05-30 00:25:14 UTC
<nzjrs_> guenther|afk: wahoo I just confirmed that the patch actually works. Must have been me setting the env vars incorrectly

More news... Setting patch status to "reviewed", since there is no "commented on".

Varadhan, your patch. :-)
Comment 6 Karsten Bräckelmann 2007-05-30 00:30:46 UTC
Oh, and... Thanks for working on this, John. And Varadhan, of course.
Comment 7 Chenthill P 2007-05-30 08:19:44 UTC
The patch looks good. Please commit it to svn HEAD and stable branch. 
Comment 8 Chenthill P 2007-05-30 08:21:25 UTC
Thanks John, varadhan and guenther :)