Patch Review please : 321770 - Import Roll History



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]