change SetImage to method instead of property to avoid unintended use

Author: taooceros

Flow.Launcher/ViewModel/ResultViewModel.cs

@@ -19,7 +19,7 @@ public ResultViewModel(Result result, Settings settings)
             if (result != null)
             {
                 Result = result;
-                Image = new Lazy<ImageSource>(() => SetImage);
+                Image = new Lazy<ImageSource>(SetImage);
             }
 
             Settings = settings;
@@ -41,37 +41,34 @@ public ResultViewModel(Result result, Settings settings)
 
         public Lazy<ImageSource> Image { get; set; }
 
-        private ImageSource SetImage
+        private ImageSource SetImage()
         {
-            get
+            var imagePath = Result.IcoPath;
+            if (string.IsNullOrEmpty(imagePath) && Result.Icon != null)
             {
-                var imagePath = Result.IcoPath;
-                if (string.IsNullOrEmpty(imagePath) && Result.Icon != null)
+                try
                 {
-                    try
-                    {
-                        return Result.Icon();
-                    }
-                    catch (Exception e)
-                    {
-                        Log.Exception($"|ResultViewModel.Image|IcoPath is empty and exception when calling Icon() for result <{Result.Title}> of plugin <{Result.PluginDirectory}>", e);
-                        imagePath = Constant.MissingImgIcon;
-                    }
+                    return Result.Icon();
                 }
+                catch (Exception e)
+                {
+                    Log.Exception($"|ResultViewModel.Image|IcoPath is empty and exception when calling Icon() for result <{Result.Title}> of plugin <{Result.PluginDirectory}>", e);
+                    imagePath = Constant.MissingImgIcon;
+                }
+            }
 
-                if (ImageLoader.CacheContainImage(imagePath))
-                    // will get here either when icoPath has value\icon delegate is null\when had exception in delegate
-                    return ImageLoader.Load(imagePath);
-                else
+            if (ImageLoader.CacheContainImage(imagePath))
+                // will get here either when icoPath has value\icon delegate is null\when had exception in delegate
+                return ImageLoader.Load(imagePath);
+            else
+            {
+                Task.Run(() =>
                 {
-                    Task.Run(() =>
-                    {
-                        Image = new Lazy<ImageSource>(() => ImageLoader.Load(imagePath));
-                        OnPropertyChanged(nameof(Image));
-                    });
+                    Image = new Lazy<ImageSource>(() => ImageLoader.Load(imagePath));
+                    OnPropertyChanged(nameof(Image));
+                });
 
-                    return ImageLoader.LoadDefault();
-                }
+                return ImageLoader.LoadDefault();
             }
         }