mpserviceplugins/mpxsqlitedbhgplugin/src/mpxdbalbum.cpp
changeset 55 f3930dda3342
parent 51 560ce2306a17
--- a/mpserviceplugins/mpxsqlitedbhgplugin/src/mpxdbalbum.cpp	Tue Aug 24 03:36:14 2010 -0500
+++ b/mpserviceplugins/mpxsqlitedbhgplugin/src/mpxdbalbum.cpp	Fri Sep 03 19:58:01 2010 -0500
@@ -1063,14 +1063,11 @@
 // ----------------------------------------------------------------------------
 //
 TBool CMPXDbAlbum::NeedToUpdateArt(const TDesC& aDeletedSongArt, const TDesC& aCurrentAlbumArt)
-    {    
-    if (aDeletedSongArt.Length() == 0)
-        {
-        // Deleted song's art has default album art
-        return EFalse;
-        }
+    { 
+    TBool needToUpdateArt = EFalse;
+    
 #ifdef ABSTRACTAUDIOALBUM_INCLUDED
-    else
+    TBool isNonEmbedded = EFalse;
     if (aDeletedSongArt.Length() > 0)
         {
         TParsePtrC parse(aDeletedSongArt);
@@ -1080,20 +1077,21 @@
         if (ext.CompareF(KAbstractAlbumExt) == 0) 
             {     
             // Deleted song's art is Non-embedded album art
-            return EFalse;
+            isNonEmbedded = ETrue;
             }
         }
-    else
-#endif   
-    if (aDeletedSongArt.Length() > 0 && aCurrentAlbumArt.Length() > 0 && aDeletedSongArt.CompareF(aCurrentAlbumArt) == 0)
+    if (!isNonEmbedded)
         {
-        // Deleted song's art is Embedded album art and it is the same as Album's current art
-        return ETrue;
+#endif   // ABSTRACTAUDIOALBUM_INCLUDED
+        if (aDeletedSongArt.Length() > 0 && aCurrentAlbumArt.Length() > 0 && aDeletedSongArt.CompareF(aCurrentAlbumArt) == 0)
+            {
+            // Deleted song's art is Embedded album art and it is the same as Album's current art
+            needToUpdateArt = ETrue;
+            }
+#ifdef ABSTRACTAUDIOALBUM_INCLUDED            
         }
-    else
-        {
-        return EFalse;
-        }
+#endif // ABSTRACTAUDIOALBUM_INCLUDED
+    return needToUpdateArt;
     }
 
 // ----------------------------------------------------------------------------