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 415608 - f-spot should remember settings of recursive import and copy photos
f-spot should remember settings of recursive import and copy photos
Status: RESOLVED DUPLICATE of bug 357398
Product: f-spot
Classification: Other
Component: Import
CVS
Other Linux
: Normal normal
: ---
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2007-03-07 07:17 UTC by Patrick Wagstrom
Modified: 2007-03-07 14:17 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch to implement feature (585 bytes, patch)
2007-03-07 07:22 UTC, Patrick Wagstrom 		none Details | Review

Description Patrick Wagstrom 2007-03-07 07:17:45 UTC 
It's a little obnoxious how I need to tell F-Spot every time I import photos whether or not it should be recursive and it should copy photos.  Wouldn't it be nice if the F-Spot import dialog just remembered the state of the checkboxes?  Well, this patch restores balance to the force by making the import remember the state of the checkboxes.
Comment 1 Patrick Wagstrom 2007-03-07 07:22:23 UTC 
Created attachment 84141 [details] [review]
patch to implement feature

this patch implements the feature.  Would appreciate review as I required ImportCommand to utilized the FSpot assembly to get and store GConf values.
Comment 2 Thomas Van Machelen 2007-03-07 08:23:00 UTC 
Thanks for the bug report. This particular bug has already been reported into our bug tracking system, but please feel free to report any further bugs you find.

Just changing the default value for the checkbox isn't really fixing the issue.  Others probably prefer the recursive option enabled.

*** This bug has been marked as a duplicate of 357398 ***
Comment 3 Patrick Wagstrom 2007-03-07 14:09:53 UTC 
Not to be rude, but did you look at what this patch does?  This patch does NOT change the default.  What it does is it REMEMBERS the state of the checkboxes.  If you're going to automatically reject a patch, please look at what it does first.

I'm posting the patch to the bug #357398 because there is no patch and this patch does exactly what that bug talks about.
Comment 4 Thomas Van Machelen 2007-03-07 14:15:55 UTC 
Patrick,

Sorry, i didn't mean to be rude, but in the patch that is attached here, I only see a change the f-spot.glade file; the patch you attached to 357398 is completelty different...
Comment 5 Patrick Wagstrom 2007-03-07 14:17:34 UTC 
Yeah, I was braindead when I submitted the patch last night.  It was late, I had been out, yada yada.  May bad completely on that one.  I should be beaten with a cluestick.