Re: [Banshee-List] banshee r2984 - in trunk/banshee: . src/Core/Banshee.Services src/Core/Banshee.Services/Banshee.Collection.Database src/Core/Banshee.Services/Banshee.Database src/Core/Hyena src/Core/Hyena/Hyena.Data.Sqlite

["Gabriel Burt" <gabriel burt gmail com>]

On Jan 17, 2008 11:41 PM,  <scottp svn gnome org> wrote:
==============================================================================
> --- trunk/banshee/src/Core/Banshee.Services/Banshee.Collection.Database/LibraryAlbumInfo.cs     (original)
> +++ trunk/banshee/src/Core/Banshee.Services/Banshee.Collection.Database/LibraryAlbumInfo.cs     Fri Jan 18 05:41:46 2008
> @@ -31,7 +31,6 @@
>
>  using Mono.Unix;
>
> -using Hyena.Data;
>  using Hyena.Data.Sqlite;
>
>  using Banshee.Database;
> @@ -39,10 +38,11 @@
>
>  namespace Banshee.Collection.Database
>  {
> +    [DatabaseTable("CoreAlbums", 1)]
>      public class LibraryAlbumInfo : AlbumInfo, IDatabaseItem
>      {
>          private static BansheeModelProvider<LibraryAlbumInfo> provider = new BansheeModelProvider<LibraryAlbumInfo> (
> -            "CoreAlbums", ServiceManager.DbConnection
> +            ServiceManager.DbConnection
>          );

Seems like the DatabaseTable attribute is quite a bit more cumbersome
than just passing the table name and requires more code.

The rest (splitting ICacheableDatabaseModel and ModelProvider) is just
what I was thinking - good job!

Gabriel