Fix a few CanExecute implementations (#1609)

Author: Hosch250

RetailCoder.VBE/UI/Command/FindAllImplementationsCommand.cs

@@ -38,12 +38,11 @@ public FindAllImplementationsCommand(INavigateCommand navigateCommand, IMessageB
 
         public override bool CanExecute(object parameter)
         {
-            if (_vbe.ActiveCodePane == null && _state.Status != ParserState.Ready)
+            if (_vbe.ActiveCodePane == null || _state.Status != ParserState.Ready)
             {
                 return false;
             }
 
-            // todo: make this work for Code/Project Explorer context menus too (may require a new command implementation)
             var target = FindTarget(parameter);
             var canExecute = target != null;
 

RetailCoder.VBE/UI/Command/FindAllReferencesCommand.cs

@@ -35,12 +35,11 @@ public FindAllReferencesCommand(INavigateCommand navigateCommand, IMessageBox me
 
         public override bool CanExecute(object parameter)
         {
-            if (_vbe.ActiveCodePane == null && _state.Status != ParserState.Ready)
+            if (_vbe.ActiveCodePane == null || _state.Status != ParserState.Ready)
             {
                 return false;
             }
-
-            // todo: make this work for Code/Project Explorer context menus too (may require a new command implementation)
+            
             var target = FindTarget(parameter);
             var canExecute = target != null;
 

RetailCoder.VBE/UI/Command/Refactorings/RefactorRemoveParametersCommand.cs

@@ -45,7 +45,7 @@ public override bool CanExecute(object parameter)
                 return false;
             }
 
-            var parameters = _state.AllUserDeclarations.Where(item => member.Equals(item.ParentScopeDeclaration)).ToList();
+            var parameters = _state.AllUserDeclarations.Where(item => item.DeclarationType == DeclarationType.Parameter && member.Equals(item.ParentScopeDeclaration)).ToList();
             var canExecute = (member.DeclarationType == DeclarationType.PropertyLet || member.DeclarationType == DeclarationType.PropertySet)
                     ? parameters.Count > 1
                     : parameters.Any();

RetailCoder.VBE/UI/Command/Refactorings/RefactorReorderParametersCommand.cs

@@ -49,7 +49,7 @@ public override bool CanExecute(object parameter)
                 return false;
             }
 
-            var parameters = _state.AllUserDeclarations.Where(item => member.Equals(item.ParentScopeDeclaration)).ToList();
+            var parameters = _state.AllUserDeclarations.Where(item => item.DeclarationType == DeclarationType.Parameter && member.Equals(item.ParentScopeDeclaration)).ToList();
             var canExecute = (member.DeclarationType == DeclarationType.PropertyLet || member.DeclarationType == DeclarationType.PropertySet)
                     ? parameters.Count > 2
                     : parameters.Count > 1;