fully debug remove old solution feature

Author: fredatgithub

GitAutoUpdateGUI/FormMain.cs

@@ -20,6 +20,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 
 using GitAutoUpdateGUI.Properties;
 using System;
+using System.Collections;
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.Drawing;
@@ -684,7 +685,7 @@ private void buttonUpdateVSProjects_Click(object sender, EventArgs e)
         return;
       }
 
-      var gitBinaryPath = "\\Git\\bin"; // valid for x86 and x64 pc because of (x86) directory
+      const string gitBinaryPath = "\\Git\\bin"; // valid for x86 and x64 pc because of (x86) directory
       string pathVariable = Environment.GetEnvironmentVariable("Path");
       if (pathVariable != null && !ContainsIgnoreCase(pathVariable, gitBinaryPath))
       {
@@ -877,6 +878,7 @@ private void buttonLoadVSProjects_Click(object sender, EventArgs e)
       }
 
       Logger.Add(textBoxLog, Translate("Searching for Visual Studio projects"));
+
       if (checkBoxUnlistVSSolution.Checked && textBoxUnlistOldSolution.Text == string.Empty)
       {
         checkBoxUnlistVSSolution.Checked = false;
@@ -920,7 +922,11 @@ private void buttonLoadVSProjects_Click(object sender, EventArgs e)
           if (subfilteredDirs.Count != 0)
           {
             //removing old or bad solution
-            if (NotHavingWords(tmpSolNameOnly, textBoxUnlistOldSolution.Text.Split(',')))
+            if (!checkBoxUnlistVSSolution.Checked ||
+              textBoxUnlistOldSolution.Text == string.Empty || 
+              (checkBoxUnlistVSSolution.Checked && 
+              NotHavingWords(tmpSolNameOnly, textBoxUnlistOldSolution.Text.Split(','), 
+              checkBoxCaseSensitive.Checked)))
             {
               ListViewItem item1 = new ListViewItem(tmpSolNameOnly) { Checked = false };
               item1.SubItems.Add(tmpSolNameOnly);
@@ -938,9 +944,37 @@ private void buttonLoadVSProjects_Click(object sender, EventArgs e)
       buttonUpdateVSProjects.Enabled = true;
     }
 
-    private static bool NotHavingWords(string source, IEnumerable<string> badWords)
+    private static bool NotHavingWords(string source, IEnumerable<string> badWords, bool caseSensitive = false)
     {
-      return badWords.All(badWord => !source.Contains(badWord));
+      bool result = true;
+      if (caseSensitive)
+      {
+        // Linq version
+        // if (badWords.Any(badWord => string.Compare(badWord.Trim(), source.Trim(), StringComparison.CurrentCulture) == 0))
+        foreach (string badWord in badWords)
+        {
+          if (string.Compare(badWord.Trim(), source.Trim(), StringComparison.CurrentCulture) == 0)
+          {
+            result = false;
+            break;
+          }
+        }
+
+        return result;
+      }
+
+      // Linq version
+      // if (badWords.Any(badWord => string.Compare(badWord.Trim(), source.Trim(), StringComparison.CurrentCultureIgnoreCase) == 0))
+      foreach (string badWord in badWords)
+      {
+        if (string.Compare(badWord.Trim(), source.Trim(), StringComparison.CurrentCultureIgnoreCase) == 0)
+        {
+          result = false;
+          break;
+        }
+      }
+
+      return result;
     }
 
     private static string FrenchPlural(int number, string currentLanguage = "english")