improve file rename logic

maxijabase · maxijabase · commit 1cd21d29a3d9 · 2021-09-23T18:00:18.000-03:00
diff --git a/UI/Windows/RenameWindow.xaml.cs b/UI/Windows/RenameWindow.xaml.cs
@@ -110,9 +110,7 @@ private bool IsNameTaken(string inputText)
 
         private void CheckExtension()
         {
-            try
-            {
-                if (!Program.MainWindow.FileIcons.ContainsKey(TxtName.Text.Substring(TxtName.Text.LastIndexOf('.'))))
+                if (!TxtName.Text.Contains(".") || !Program.MainWindow.FileIcons.ContainsKey(TxtName.Text.Substring(TxtName.Text.LastIndexOf('.'))))
                 {
                     lblError.Content = Program.Translations.GetLanguage("FileNotSupported");
                     lblError.ToolTip = Program.Translations.GetLanguage("FileWillBeExcluded");
@@ -122,11 +120,6 @@ private void CheckExtension()
                     lblError.Content = "";
                     lblError.ToolTip = "";
                 }
-            }
-            catch (ArgumentOutOfRangeException)
-            {
-                // this is ugly and newbish but I'm lazy and it works
-            }
         }
 
         public void Language_Translate()

Original file line number	Diff line number	Diff line change
`@@ -110,9 +110,7 @@ private bool IsNameTaken(string inputText)`
`110`	`110`
`111`	`111`	`private void CheckExtension()`
`112`	`112`	`{`
`113`		`- try`
`114`		`- {`
`115`		`- if (!Program.MainWindow.FileIcons.ContainsKey(TxtName.Text.Substring(TxtName.Text.LastIndexOf('.'))))`
	`113`	`+ if (!TxtName.Text.Contains(".") \|\| !Program.MainWindow.FileIcons.ContainsKey(TxtName.Text.Substring(TxtName.Text.LastIndexOf('.'))))`
`116`	`114`	`{`
`117`	`115`	`lblError.Content = Program.Translations.GetLanguage("FileNotSupported");`
`118`	`116`	`lblError.ToolTip = Program.Translations.GetLanguage("FileWillBeExcluded");`
`@@ -122,11 +120,6 @@ private void CheckExtension()`
`122`	`120`	`lblError.Content = "";`
`123`	`121`	`lblError.ToolTip = "";`
`124`	`122`	`}`
`125`		`- }`
`126`		`- catch (ArgumentOutOfRangeException)`
`127`		`- {`
`128`		`- // this is ugly and newbish but I'm lazy and it works`
`129`		`- }`
`130`	`123`	`}`
`131`	`124`
`132`	`125`	`public void Language_Translate()`