Bug Fix

Saving a sequence in a folder without the necessary permissions caused a crash

Author: LuisMiSanVe

MouseDummy/Frm_MouseDummy.cs

@@ -337,36 +337,42 @@ private void cmbbx_savedSequences_SelectedIndexChanged(object sender, EventArgs
                                 break;
                         }
                     }
+                    try
+                    {
+                        // Save in a file
+                        if (!Directory.Exists("SavedSequences"))
+                            Directory.CreateDirectory("SavedSequences");
+                        File.WriteAllText("SavedSequences\\" + sequenceName + ".sqc", currentSequence);
 
-                    // Save in a file
-                    if (!Directory.Exists("SavedSequences"))
-                        Directory.CreateDirectory("SavedSequences");
-                    File.WriteAllText("SavedSequences\\" + sequenceName + ".sqc", currentSequence);
-
-                    // Adds the sequence to the ComboBox
-                    cmbbx_savedSequences.Items.Add(sequenceName);
+                        // Adds the sequence to the ComboBox
+                        cmbbx_savedSequences.Items.Add(sequenceName);
+                    }
+                    catch (Exception ex)
+                    {
+                        MessageBox.Show("The sequence couldn't be saved in your drive, make sure the installation path have the necessary permissions or start the program as Administrator.", "Permissions Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                    }
                 }
-            }
-            else if (cmbbx_savedSequences.Items[cmbbx_savedSequences.SelectedIndex].ToString().Contains("+ New"))
-            {
-                // Clear the current points
-                lstbx_points.Items.Clear();
-            }
-            else if (cmbbx_savedSequences.Items[cmbbx_savedSequences.SelectedIndex].ToString().Contains("------"))
-            {
-                // Does nothing as only serves as separator between options and actual saved sequences
-            }
-            else
-            {
-                // Clear the current points
-                lstbx_points.Items.Clear();
+                else if (cmbbx_savedSequences.Items[cmbbx_savedSequences.SelectedIndex].ToString().Contains("+ New"))
+                {
+                    // Clear the current points
+                    lstbx_points.Items.Clear();
+                }
+                else if (cmbbx_savedSequences.Items[cmbbx_savedSequences.SelectedIndex].ToString().Contains("------"))
+                {
+                    // Does nothing as only serves as separator between options and actual saved sequences
+                }
+                else
+                {
+                    // Clear the current points
+                    lstbx_points.Items.Clear();
 
-                // Load from the file
-                string[] savedSequence = File.ReadAllText("SavedSequences\\" + cmbbx_savedSequences.SelectedItem + ".sqc").Split("\n");
+                    // Load from the file
+                    string[] savedSequence = File.ReadAllText("SavedSequences\\" + cmbbx_savedSequences.SelectedItem + ".sqc").Split("\n");
 
-                lstbx_points.Items.AddRange(savedSequence);
+                    lstbx_points.Items.AddRange(savedSequence);
+                }
+                cmbbx_savedSequences.SelectedText = "";
             }
-            cmbbx_savedSequences.SelectedText = "";
         }
 
         private void nmrupdwn_actionDelay_ValueChanged(object sender, EventArgs e)

MouseDummy/MouseDummy.sln

@@ -1,7 +1,7 @@
 
 Microsoft Visual Studio Solution File, Format Version 12.00
 # Visual Studio Version 17
-VisualStudioVersion = 17.14.36109.1 d17.14
+VisualStudioVersion = 17.14.36109.1
 MinimumVisualStudioVersion = 10.0.40219.1
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MouseDummy", "MouseDummy.csproj", "{3C2FAB0B-4DAE-45E5-94BE-5D69ADBF7462}"
 EndProject