From: shivamv <shivam.v2@samsung.com>
Date: Tue, 21 Sep 2021 10:38:50 +0000 (+0530)
Subject: Fixed SVACE issues
X-Git-Tag: submit/tizen/20210923.162342~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F59%2F264459%2F2;p=profile%2Fiot%2Fapps%2Fdotnet%2Fmusic-player.git

Fixed SVACE issues

Change-Id: Ia7745520193ff4085329a1c34ca6dfab1a4c3225
Signed-off-by: shivamv <shivam.v2@samsung.com>
---

diff --git a/music-player/MediaContent/TrackContents.cs b/music-player/MediaContent/TrackContents.cs
index 97b3b18..f57e80e 100755
--- a/music-player/MediaContent/TrackContents.cs
+++ b/music-player/MediaContent/TrackContents.cs
@@ -44,6 +44,10 @@ namespace MusicPlayer.Media
             foreach (string id in trackList)
             {
                 MediaInfo info = mediaInfo.SelectMedia(id);
+                if (info == null)
+                {
+                    continue;
+                }
                 string filePath = @info.Path;
                 try
                 {
@@ -85,7 +89,10 @@ namespace MusicPlayer.Media
             foreach (string id in trackList)
             {
                 MediaInfo info = mediaInfo.SelectMedia(id);
-                Tizen.Log.Debug(AppConstants.LogTag, "Track Path: " + info.Path);
+                if (info == null)
+                {
+                    continue;
+                }
                 string filePath = @info.Path;
                 try
                 {
diff --git a/music-player/Models/SearchModel.cs b/music-player/Models/SearchModel.cs
index d9057ab..56e029b 100755
--- a/music-player/Models/SearchModel.cs
+++ b/music-player/Models/SearchModel.cs
@@ -15,19 +15,19 @@ namespace MusicPlayer.Models
             ArtistDetail = artist;
             TrackDetail = track;
             ItemType = type;
-            if(type ==0)
+            if(type ==0 && artist != null)
             {
                 Thumbnail = artist.AlbumArtPath;
                 Title = artist.ArtistName;
                 SubTitle = "";
             }
-            else if (type == 1)
+            else if (type == 1 && musicAlbum != null)
             {
                 Thumbnail = musicAlbum.AlbumArtPath;
                 Title = musicAlbum.AlbumName;
                 SubTitle = musicAlbum.ArtistName;
             }
-            else if (type == 2)
+            else if (type == 2 && track != null)
             {
                 Thumbnail = track.ThumbnailPath;
                 Title = track.TrackTitle;
diff --git a/music-player/ViewModels/LyricsViewModel.cs b/music-player/ViewModels/LyricsViewModel.cs
index 6ffff18..b9db5a3 100755
--- a/music-player/ViewModels/LyricsViewModel.cs
+++ b/music-player/ViewModels/LyricsViewModel.cs
@@ -52,6 +52,7 @@ namespace MusicPlayer.ViewModels
                 var metadataExtractor = new MetadataExtractor(path);
                 Metadata metadata = metadataExtractor.GetMetadata();
                 lyrics = metadata.UnsyncLyrics;
+                metadataExtractor.Dispose();
             }
             catch(System.IO.FileNotFoundException ex)
             {
diff --git a/music-player/ViewModels/PlayerViewModel.cs b/music-player/ViewModels/PlayerViewModel.cs
index d4ab854..f2882c0 100755
--- a/music-player/ViewModels/PlayerViewModel.cs
+++ b/music-player/ViewModels/PlayerViewModel.cs
@@ -257,7 +257,11 @@ namespace MusicPlayer.ViewModels
 
         public void StartShuffledTrack()
         {
-            SetCurrentTrack(playingListViewModel.FirstShuffledTrack());
+            Track track = playingListViewModel.FirstShuffledTrack();
+            if (track != null)
+            {
+                SetCurrentTrack(track);
+            }
         }
 
         private void UpdatePlayingStatus(PlayingStatus status)
@@ -438,14 +442,16 @@ namespace MusicPlayer.ViewModels
                 Tizen.Log.Info(AppConstants.LogTag, "Palette or palatte values not valid, adding default gradient");
                 ImageVisual imageVisual = CreateImageVisual();
                 PlayerBackground = imageVisual.OutputVisualMap;
-                return;
+                imageVisual.Dispose();
             }
             else
             {
                 Tizen.Log.Info(AppConstants.LogTag, "setting palette color");
                 GradientVisual gradientVisual = CreateGradientVisual(stopColor);
                 PlayerBackground = gradientVisual.OutputVisualMap;
+                gradientVisual.Dispose();
             }
+            stopColor.Dispose();
         }
     }
 }
diff --git a/music-player/Views/SearchView.cs b/music-player/Views/SearchView.cs
index 44b713a..3a6ed41 100755
--- a/music-player/Views/SearchView.cs
+++ b/music-player/Views/SearchView.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Collections.Generic;
 using Tizen.NUI.Components;
 using Tizen.NUI.BaseComponents;
 using Tizen.NUI;
@@ -312,5 +313,71 @@ namespace MusicPlayer.Views
                 noItemFound = true;
             }
         }
+
+        protected override void Dispose(DisposeTypes type)
+        {
+            if (Disposed)
+            {
+                return;
+            }
+            if (type == DisposeTypes.Explicit)
+            {
+                if (searchBox != null)
+                {
+                    List<View> children = searchBox.Children;
+                    foreach (View child in children)
+                    {
+                        searchBox.Remove(child);
+                        child.Dispose();
+                    }
+                    textField = null;
+                    crossButton = null;
+                }
+
+                if (topView != null)
+                {
+                    List<View> children = topView.Children;
+                    foreach (View child in children)
+                    {
+                        topView.Remove(child);
+                        child.Dispose();
+                    }
+                    backButton = null;
+                    searchBox = null;
+                    base.Remove(topView);
+                    topView.Dispose();
+                    topView = null;
+                }
+
+                if (!noItemFound && collectionView != null)
+                {
+                    base.Remove(collectionView);
+                }
+                else if(noItemFound && noItemView != null)
+                {
+                    base.Remove(noItemView);
+                }
+
+                if (collectionView != null)
+                {
+                    collectionView.Dispose();
+                    collectionView = null;
+                }
+
+                if (noItemView != null)
+                {
+                    List<View> children = noItemView.Children;
+                    foreach (View child in children)
+                    {
+                        noItemView.Remove(child);
+                        child.Dispose();
+                    }
+                    noItemView.Dispose();
+                    noItemView = null;
+                }
+            }
+
+            base.Dispose(type);
+        }
     }
 }
diff --git a/packaging/org.tizen.MusicPlayer-1.0.0.tpk b/packaging/org.tizen.MusicPlayer-1.0.0.tpk
index 68af2f8..a542c52 100755
Binary files a/packaging/org.tizen.MusicPlayer-1.0.0.tpk and b/packaging/org.tizen.MusicPlayer-1.0.0.tpk differ