Missed some cleanup

Author: BZngr

Rubberduck.Core/UI/Refactorings/AssignedByValParameterQuickFixDialog.cs

@@ -1,22 +1,18 @@
 using System;
-using System.Collections.Generic;
 using System.Windows.Forms;
-using System.Linq;
 using Rubberduck.Common;
 using Rubberduck.Resources;
 
 namespace Rubberduck.UI.Refactorings
 {
     public partial class AssignedByValParameterQuickFixDialog : Form, IAssignedByValParameterQuickFixDialog
     {
-        //private readonly IEnumerable<string> _forbiddenNames;
         private readonly Func<string, bool> _isConflictingName;
 
         public AssignedByValParameterQuickFixDialog(string identifier, string identifierType, Func<string, bool> nameCollisionChecker)//, IEnumerable<string> forbiddenNames)
         {
             InitializeComponent();
             InitializeCaptions(identifier, identifierType);
-            //_forbiddenNames = forbiddenNames;
             _isConflictingName = nameCollisionChecker;
         }
 
@@ -56,10 +52,6 @@ private string GetVariableNameFeedback()
             {
                 return string.Empty;
             }
-            //if (_forbiddenNames.Any(name => name.Equals(NewName, StringComparison.OrdinalIgnoreCase)))
-            //{
-            //    return string.Format(RubberduckUI.AssignedByValDialog_NewNameAlreadyUsedFormat, NewName);
-            //}
             if (_isConflictingName(NewName))
             {
                 return string.Format(RubberduckUI.AssignedByValDialog_NewNameAlreadyUsedFormat, NewName);
@@ -85,8 +77,7 @@ private string GetVariableNameFeedback()
 
         private void SetControlsProperties()
         {
-            //var isValid = VariableNameValidator.IsValidName(NewName) && !_forbiddenNames.Any(name => name.Equals(NewName, StringComparison.OrdinalIgnoreCase));
-            var isValid = VariableNameValidator.IsValidName(NewName); // && !_forbiddenNames.Any(name => name.Equals(NewName, StringComparison.OrdinalIgnoreCase));
+            var isValid = VariableNameValidator.IsValidName(NewName);
             OkButton.Visible = isValid;
             OkButton.Enabled = isValid;
             InvalidNameValidationIcon.Visible = !isValid;

Rubberduck.Core/UI/Refactorings/IAssignedByValParameterQuickFixDialogFactory.cs

@@ -1,12 +1,10 @@
-using Rubberduck.Parsing.Symbols;
-using System;
-using System.Collections.Generic;
+using System;
 
 namespace Rubberduck.UI.Refactorings
 {
     public interface IAssignedByValParameterQuickFixDialogFactory
     {
-        IAssignedByValParameterQuickFixDialog Create(string identifier, string identifierType, Func<string, bool> nameCollisionChecker); //, IEnumerable<string> forbiddenNames);
+        IAssignedByValParameterQuickFixDialog Create(string identifier, string identifierType, Func<string, bool> nameCollisionChecker);
         void Release(IAssignedByValParameterQuickFixDialog dialog);
     }
 }

RubberduckTests/QuickFixes/AssignedByValParameterMakeLocalCopyQuickFixTests.cs

@@ -1,5 +1,4 @@
 using NUnit.Framework;
-using System.Collections.Generic;
 using System.Linq;
 using System.Threading;
 using Rubberduck.Inspections.QuickFixes;
@@ -393,7 +392,7 @@ private Mock<IAssignedByValParameterQuickFixDialogFactory> BuildMockDialogFactor
             mockDialog.SetupGet(m => m.DialogResult).Returns(() => DialogResult.OK);
 
             var mockDialogFactory = new Mock<IAssignedByValParameterQuickFixDialogFactory>();
-            mockDialogFactory.Setup(f => f.Create(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<Func<string,bool>>()/*, It.IsAny<IEnumerable<string>>()*/)).Returns(mockDialog.Object);
+            mockDialogFactory.Setup(f => f.Create(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<Func<string,bool>>())).Returns(mockDialog.Object);
 
             return mockDialogFactory;
         }