Re: [Muine] Some thoughts on playlist management

[Tom Coleman <tmcol bigpond net au>]

Alright, alright, I'll do it.

Or more accurately, I've done it. Its not too complicated, and I think
it works nicely. 

The only issue is that because the individual songs are no long part of
an album, they don't have an album photo attached to them. I don't think
there is a very clean way make it so they do...

Oh well, they wouldn't have with the retagging solution anyway.

Cheers,
Tom

On Fri, 2005-01-07 at 17:56 +0200, Yo'av Moshe wrote:
> I'm certanialy voting for this one. I'm having this problem too, and
> it's seems like the only solution that won't harm no-one.
> 
> Now we only need someone to write a patch for it... :)
> 
> Yo'av.
> 
> 
> On Sat, 08 Jan 2005 02:08:52 +1100, Tom Coleman <tmcol bigpond net au> wrote:
> > Hmm...
> > 
> > Well although this will certainly fix the problem, I don't really like
> > the idea of having a significant portion of my collection improperly
> > tagged.
> > 
> > What about the possibility of having a gconf key specifying a directory
> > under which muine won't group songs into albums? I realise its fairly
> > clumsy, but it seems the best solution to me, and having it in gconf
> > will mean it will be hidden to those who don't know why its there.
> > 
> > Cheers,
> > Tom
> > 
> > On Fri, 2005-01-07 at 14:57 +0100, Jorn Baayen wrote:
> > > Hi,
> > >
> > > Yea, this is a known problem, but from the Muine side there is nothing
> > > that can be done about it I think. Single-track albums are valid and do
> > > exists, so having a certain minimum of tracks won't do.
> > >
> > > The best I can suggest is hacking up a simple script that will strip
> > > album tags off all your single songs.
> > >
> > > HTH,
> > >
> > > Jorn
> > >
> > > On Fri, 2005-01-07 at 20:19 +1100, Tom Coleman wrote:
> > > > Hi all,
> > > >
> > > > I've always really enjoyed using muine because my listening habits are
> > > > very album based, and muine fits my use case very well (I'm sure it
> > > > works well more many others too).
> > > >
> > > > However, for whatever reason, I have a collection of songs that aren't
> > > > whole albums. Just random tracks that I liked and picked up along the
> > > > way...
> > > >
> > > > The problem that this causes is that the Album list ends up having all
> > > > these albums in it consisting of single (or a couple) of tracks. This
> > > > means that its harder to find the 'real' albums, plus it can be a source
> > > > of disappointment :) For me it would be more appropriate if those tracks
> > > > were only to appear in the Song list...
> > > >
> > > > As it is I don't see any good way to fix this problem.. here's a couple
> > > > of things I thought of
> > > > 1) Define an album as having at least X tracks. Clumsy and generally
> > > > bad, I'd say..
> > > > 2) Use the 'total tracks' tag. However this is often not defined, plus
> > > > it would probably be quite annoying if an album didn't appear just
> > > > because you were missing one song (for whatever reason)
> > > > 3) Define a seperate directory where 'single tracks' are placed.
> > > > Although I think its clumsy, and perhaps asks too much from the user, I
> > > > personally would be willing to do it..
> > > >
> > > > Does anyone have any better ideas? (I'll admit mine aren't very good :)
> > > >
> > > > Regards,
> > > > Tom
> > > >
> > > > _______________________________________________
> > > > muine-list mailing list
> > > > muine-list gnome org
> > > > http://mail.gnome.org/mailman/listinfo/muine-list
> > >
> > 
> > _______________________________________________
> > muine-list mailing list
> > muine-list gnome org
> > http://mail.gnome.org/mailman/listinfo/muine-list
> >

? non_album_folders.patch
Index: data/muine.schemas.in
===================================================================
RCS file: /cvs/gnome/muine/data/muine.schemas.in,v
retrieving revision 1.16
diff -a -u -r1.16 muine.schemas.in
--- data/muine.schemas.in	13 Dec 2004 14:59:58 -0000	1.16
+++ data/muine.schemas.in	7 Jan 2005 17:45:48 -0000
@@ -135,6 +135,18 @@
       </locale>
     </schema>
     <schema>
+      <key>/schemas/apps/muine/non_album_folders</key>
+      <applyto>/apps/muine/non_album_folders</applyto>
+      <owner>muine</owner>
+      <type>list</type>
+      <list_type>string</list_type>
+      <default>[]</default>
+      <locale name="C">
+        <short>Non album folders</short>
+        <long>Folders that don't contain entire albums.</long>
+      </locale>
+    </schema>
+    <schema>
       <key>/schemas/apps/muine/default_import_folder</key>
       <applyto>/apps/muine/default_import_folder</applyto>
       <owner>muine</owner>
Index: src/SongDatabase.cs
===================================================================
RCS file: /cvs/gnome/muine/src/SongDatabase.cs,v
retrieving revision 1.55
diff -a -u -r1.55 SongDatabase.cs
--- src/SongDatabase.cs	7 Jan 2005 03:01:44 -0000	1.55
+++ src/SongDatabase.cs	7 Jan 2005 17:45:50 -0000
@@ -32,6 +32,10 @@
 		private const string GConfKeyWatchedFolders = "/apps/muine/watched_folders";
 		private readonly string [] GConfDefaultWatchedFolders = new string [0];
 
+                private const string GConfKeyNonAlbumFolders = "/apps/muine/non_album_folders";
+                private readonly string [] GConfDefaultNonAlbumFolders = new string [0];
+                private string [] non_album_folders = new string [0];
+    
 		private Hashtable songs;
 		public Hashtable Songs {
 			get { return songs; }
@@ -92,6 +96,9 @@
 			db.Load ();
 
 			loading = false;
+
+                        /* block albums from being added to non-album folders */
+                        non_album_folders = (string []) Config.Get (GConfKeyNonAlbumFolders, GConfDefaultNonAlbumFolders);
 			
 			/* add file monitors */
 			string [] folders = (string []) Config.Get (GConfKeyWatchedFolders, GConfDefaultWatchedFolders);
@@ -201,6 +208,11 @@
 				return;
 
 			string key = song.AlbumKey;
+
+                        // check that we aren't blocking this song as an album
+                        foreach (string folder in non_album_folders)
+                                if (key.StartsWith (folder))
+                                        return;
 
 			Album album = (Album) Albums [key];