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 353539 - Prefs for FolderExport not stored in gconf
Prefs for FolderExport not stored in gconf
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: General
CVS
Other Linux
: Normal minor
: ---
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2006-08-30 13:02 UTC by Stephane Delcroix
Modified: 2006-10-19 09:30 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Store settings for folder export (4.32 KB, patch)
2006-10-18 18:18 UTC, Thomas Van Machelen 		committed Details | Review

Description Stephane Delcroix 2006-08-30 13:02:34 UTC 
Keep default user values in GConf for
- Export to folder
- Export to Gallery
Comment 1 Stephane Delcroix 2006-08-30 13:09:21 UTC 
sorry, it's already done for Gallery. I change the title to reflect the new status.
Comment 2 Larry Ewing 2006-10-05 04:13:54 UTC 
agreed
Comment 3 Thomas Van Machelen 2006-10-18 18:18:22 UTC 
Created attachment 74968 [details] [review]
Store settings for folder export

Mr Delcroix & Mr Ewing, does this patch suit your needs?
Comment 4 Stephane Delcroix 2006-10-18 18:41:40 UTC 
Dear Meneer Van Machelen,

This patch fill perfectly the hole existing in the folder export code.

Best regards,

Mr Delcroix

Ah, if Thomas is reading also: 
Hi Thomas,
think about adding a changelog entry, so this job can be credited to you at commit time.

s
Comment 5 Thomas Van Machelen 2006-10-18 19:28:45 UTC 
Changelog entry:
2006-10-18  Thomas Van Machelen  <thomas.vanmachelen@gmail.com>

        * src/Preferences: add new gconf paths
	* src/FolderExport.cs: add code to store the export setting in gconf
                               fixes bgo 353539

Thank you very much.
Comment 6 Thomas Van Machelen 2006-10-19 09:27:02 UTC 
code is in cvs, closing this one.
Comment 7 Stephane Delcroix 2006-10-19 09:30:36 UTC 
yes, slightly modified version