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:
  Show dependency tree
 
Reported: 2006-07-30 22:52 UTC by Ryan Paul
Modified: 2006-09-09 21:21 UTC (History)
1 user (show)

See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for #349359 (1.33 KB, patch)
2006-09-08 09:02 UTC, Johannes Schmid
none Details | Diff | Review
349359_v2.patch (9.84 KB, patch)
2006-09-09 12:04 UTC, Murray Cumming
none Details | Diff | 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!

Note You need to log in before you can comment on or make changes to this bug.