Merge pull request #453 from ghost/fix/mods-list

fixed lag & breaking changes for bulk mods edit

Author: Bishbash777

Torch.Server/Views/ModListControl.xaml.cs

@@ -269,7 +269,10 @@ private void BulkButton_OnClick(object sender, RoutedEventArgs e)
             modList.RemoveAll(m =>
             {
                 var mod = m.ToString();
-                return idList.Any(mod.Equals);
+                return idList.Any(id =>
+                    id.IndexOf('-') > 0
+                        ? mod.Equals(id, StringComparison.Ordinal)
+                        : m.PublishedFileId.Equals(ulong.Parse(id)));
             });
             modList.AddRange(idList.Select(id =>
             {
@@ -282,13 +285,8 @@ private void BulkButton_OnClick(object sender, RoutedEventArgs e)
                 _instanceManager.DedicatedConfig.Mods.Add(mod);
 
             if (tasks.Any())
-                Task.WaitAll(tasks.ToArray());
-            
-            Dispatcher.Invoke(() =>
-                              {
-                                  _instanceManager.DedicatedConfig.Save();
-                              });
-
+                Task.WhenAll(tasks.ToArray())
+                    .ContinueWith(_ => Dispatcher.Invoke(() => _instanceManager.DedicatedConfig.Save()));
         }
 
         private void UgcServiceTypeBox_OnSelectionChanged(object sender, SelectionChangedEventArgs e)

Torch/Utils/ModItemUtils.cs

@@ -16,7 +16,8 @@ public static MyObjectBuilder_Checkpoint.ModItem Create(ulong modId, string serv
         public static MyObjectBuilder_Checkpoint.ModItem Create(string str)
         {
             var arr = str.Split('-');
-            return new MyObjectBuilder_Checkpoint.ModItem(ulong.Parse(arr[0]), arr[1]);
+            // backward compat
+            return new MyObjectBuilder_Checkpoint.ModItem(ulong.Parse(arr[0]), arr.Length > 1 ? arr[1] : GetDefaultServiceName());
         }
 
         //because KEEEN!