Make pinned pipes actually persistent

Author: grayed

Services/Settings.cs

@@ -76,6 +76,7 @@ public Settings()
             ColumnWidths.ToObservableChangeSet<ObservableDictionary<string, int>, KeyValuePair<string, int>>()
                 .CastToObject()
                 .Concat(this.WhenAnyPropertyChanged().ToObservableChangeSet().CastToObject())
+                .Concat(PinnedNames.ToObservableChangeSet<ObservableSet<string>, string>().CastToObject())
                 .Throttle(TimeSpan.FromSeconds(1))
                 .Subscribe(_ => Save());
         }

ViewModels/PipeExplorerViewModel.cs

@@ -163,7 +163,7 @@ public PipeExplorerViewModel()
                 .Select(v => (ImageSource)App.Current.FindResource(v ? "AppIconActive" : "AppIcon"))
                 .Subscribe(v => AppIcon = v);
 
-            pipesCache.AddOrUpdate(Native.GetPipes().Select(p => new PipeViewModel(p, false)));
+            pipesCache.AddOrUpdate(Native.GetPipes().Select(p => new PipeViewModel(p, settings.PinnedNames.Contains(p.Name), false)));
             IsRunning = settings.StartImmediately;
         }
 
@@ -173,11 +173,11 @@ private void Pipe_Created(object sender, PipeWatcherEventArgs e)
             if (opt.HasValue && opt.Value.BeingRemoved)
             {
                 pipesCache.RemoveKey(e.Pipe.Path);
-                pipesCache.AddOrUpdate(new PipeViewModel(e.Pipe));
+                pipesCache.AddOrUpdate(new PipeViewModel(e.Pipe, settings.PinnedNames.Contains(e.Pipe.Name)));
             }
             else if (!opt.HasValue)     // avoid re-creation of view models on startup
             {
-                pipesCache.AddOrUpdate(new PipeViewModel(e.Pipe));
+                pipesCache.AddOrUpdate(new PipeViewModel(e.Pipe, settings.PinnedNames.Contains(e.Pipe.Name)));
             }
         }
 

ViewModels/PipeViewModel.cs

@@ -49,8 +49,6 @@ class PipeViewModel : ReactiveObject, IComparable<PipeViewModel>, IComparable
         [Reactive] public bool RecentlyAdded { get; private set; }
         [Reactive] public bool Pinned { get; set; }
 
-        public ICommand SwitchPinnedStateCmd { get; }
-
         private DateTime lastUpdateTimestamp;
 
         public void MarkForRemoval()
@@ -60,13 +58,14 @@ public void MarkForRemoval()
             BeingRemoved = true;
         }
 
-        public PipeViewModel(PipeModel model, bool markAsRecentlyAdded = true)
+        public PipeViewModel(PipeModel model, bool pinned, bool markAsRecentlyAdded = true)
         {
             Host = model.Host;
             Name = model.Name;
             Path = model.Path;
 
-            SwitchPinnedStateCmd = ReactiveCommand.Create(() => Pinned = !Pinned);
+            Pinned = pinned;
+            this.WhenValueChanged(x => x.Pinned).Subscribe(UpdatePinnedNames);
 
             var nowStr = DateTime.Now.ToString(CultureInfo.CurrentUICulture);
             if (markAsRecentlyAdded)
@@ -85,6 +84,15 @@ public PipeViewModel(PipeModel model, bool markAsRecentlyAdded = true)
             Task.Delay(Locator.Current.GetService<ISettings>().HighlightDuration).ContinueWith(_ => RecentlyAdded = false);
         }
 
+        private void UpdatePinnedNames(bool isPinned)
+        {
+            var names = Locator.Current.GetService<ISettings>().PinnedNames;
+            if (isPinned)
+                names.Add(Name);
+            else
+                names.Remove(Name);
+        }
+
         private void UpdatePlainAcl(AclModel acl)
         {
             var sb = new StringBuilder();