Patch Review please : 321770 - Import Roll History
- From: Bengt Thuree <bengt thuree com>
- To: f-spot-list <f-spot-list gnome org>
- Subject: Patch Review please : 321770 - Import Roll History
- Date: Fri, 04 Aug 2006 00:18:03 +1000
Hi Guys
New patch submitted for bug 321770 - Keeping track of the last 20 import
rolls.
The main changes :
* New gui by Stephane.
* Using gconf to store the number of rolls to keep track of.
Bug : http://bugzilla.gnome.org/show_bug.cgi?id=321770
Patch : http://bugzilla.gnome.org/attachment.cgi?id=70137&action=view
It applies cleanly to CVS, and it also updates the photos.db to add one
extra table (imports_data)
Looking forward to see some review of this patch.
/Bengt
Changelog as below:
> 2006-08-03 Bengt Thuree <bengt thuree com>
>
> * : The below changes are for bug #321770
> * src/ImportStore.cs: Reworked this class. Creating two tables,
> imports and imports_data, to track the last 10 or so imports (fixes #321770).
>
> * src/FileImportBackend.cs: Adding the import session for each
> imported photo, as well as Removing on cancel, or Finish on finish.
>
> * src/CameraFileSelectionDialog.cs:
> * src/ImportCommand.cs:
> * src/MainWindow.cs: Added ImportStore parameter, as well as three menu entries
>
> * src/Updater.cs: Creating the extra imports_data table (to keep import id and photo id)
>
> * src/PhotoQuery.cs: Added query on last import session id.
>
> * src/QueryDisplay.cs: Added indication of Import Roll filter, as well as clearing it.
>
> * src/PhotoStore.cs: Added PhotoStore.ImportIdRange, as well as sql filter on last import session id.
> Minor modifications on creating a query to handle (AND or WHERE text)
>
> * src/f-spot.glade:
> * src/LastRollDialog.cs: Added a dialog for the Last Imported Roll history and selection.
>
> * src/Preferences.cs: Added gconf parameter for how many import rolls to store.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]