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 349359 - Dates added with the calendar dialog aren't preserved
Dates added with the calendar dialog aren't preserved
Status: RESOLVED FIXED
Product: glom
Classification: Other
Component: data mode
1.0.x
Other Linux
: Normal normal
: ---
Assigned To: Murray Cumming
Murray Cumming
Depends on:
Blocks:
 
 
Reported: 2006-07-30 22:52 UTC by Ryan Paul
Modified: 2006-09-09 21:21 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for #349359 (1.33 KB, patch)
2006-09-08 09:02 UTC, Johannes Schmid
none Details | Review
349359_v2.patch (9.84 KB, patch)
2006-09-09 12:04 UTC, Murray Cumming
none Details | Review

Description Ryan Paul 2006-07-30 22:52:01 UTC
When I use the calendar control on the details tab to set a date value, it doesn't get saved. When I go back to the list view after setting a date with the calendar control, the value isn't there. In order to enter a date on the details tab, I have to type it in manually.
Comment 1 Murray Cumming 2006-07-31 08:27:19 UTC
Thanks. What version of Glom, and what distro is this?
Comment 2 Ryan Paul 2006-07-31 08:41:19 UTC
I'm using version 1.0.3 in Ubuntu.
Comment 3 Johannes Schmid 2006-09-07 12:16:06 UTC
I can comfirm this. Seems like the changed signal is somehow not emitted. I will try to fix this.
Comment 4 Johannes Schmid 2006-09-08 09:02:43 UTC
Created attachment 72405 [details] [review]
Patch for #349359

OK, so the problem is simply that the entry never get the focus and thus the focus_out_event used for saving never occurs. This patch fixes this by giving the entry the focus after the date was selected.

Patch was sponsered by Openismus Gmbh!
Comment 5 Murray Cumming 2006-09-09 12:04:50 UTC
Created attachment 72447 [details] [review]
349359_v2.patch

This patch just emits the edited signal instead. That seems more explicit, and it's what we do elsewhere, and it seems to work. I have committed this.
Comment 6 Murray Cumming 2006-09-09 12:06:51 UTC
Sorry, please ignore the changes to util.cc in that patch. That's a different work in progress.
Comment 7 Johannes Schmid 2006-09-09 21:08:26 UTC
Hi Murray!

I tried emitting the signal directly but this gave me some strage gda-warnings so I searched for another solution. But if this works for you it's OK!

Regards,
Johannes
Comment 8 Johannes Schmid 2006-09-09 21:15:12 UTC
In Detail, it gives me the following:

debug: Box_Data_List::refresh_data_from_database_blank(): before refresh_from_database_blank().

Regards,
Johannes
Comment 9 Johannes Schmid 2006-09-09 21:21:38 UTC
After some testing I think the message comes from some other part, so that should be no problem - sorry!