Philip Meyer;512813 Wrote: 
> Setting ALBUMARTIST=Various Artists doesn't make an album a compilation
> (unless something has changed).

It may be new, but I think it makes a lot of sense.  A lot of people
have tagged compilations like this, so it short circuits the logic that
says _any_ ALBUMARTIST means not a compilation.


Code:
--------------------
    
  # Run another query to automatically set all albums with an ARTIST contributor
  # named "Various Artists" (id $vaObjId) to compilation = 1
  $self->dbh->do( qq{
        UPDATE albums
        SET    compilation = 1
        WHERE id IN (
                SELECT album FROM contributor_album
                WHERE role = ?
                AND   contributor = ?
        )
  }, undef, $role, $vaObjId );
  
--------------------


But again, if the same logic is applied when scanning each track, I'm
not sure why it would need to be applied library-wide at the end of a
scan.  Incremental scanning (if done correctly) should avoid these sort
of wrap up queries.


-- 
JJZolx

Jim
------------------------------------------------------------------------
JJZolx's Profile: http://forums.slimdevices.com/member.php?userid=10
View this thread: http://forums.slimdevices.com/showthread.php?t=74700

_______________________________________________
beta mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/beta

Reply via email to