Skip to content

Commit 6c17a83

Browse files
committed
FormDesignerRefactorRenameCommand:
- Reduced indentation levels
1 parent da5ec5f commit 6c17a83

File tree

1 file changed

+15
-17
lines changed

1 file changed

+15
-17
lines changed

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

Lines changed: 15 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ public FormDesignerRefactorRenameCommand(IVBE vbe, RubberduckParserState state,
2525

2626
protected override bool EvaluateCanExecute(object parameter)
2727
{
28-
return (_state?.Status ?? ParserState.None) == ParserState.Ready
28+
return _state?.Status == ParserState.Ready
2929
&& GetTarget() != null;
3030
}
3131

@@ -47,36 +47,34 @@ protected override void OnExecute(object parameter)
4747

4848
private Declaration GetTarget(QualifiedModuleName? qualifiedModuleName = null)
4949
{
50-
(var projectId, var component) =
51-
qualifiedModuleName.HasValue
52-
? (qualifiedModuleName.Value.ProjectId, qualifiedModuleName.Value.Component)
53-
: (Vbe.ActiveVBProject.ProjectId, Vbe.SelectedVBComponent);
50+
(var projectId, var component) = qualifiedModuleName.HasValue
51+
? (qualifiedModuleName.Value.ProjectId, qualifiedModuleName.Value.Component)
52+
: (Vbe.ActiveVBProject.ProjectId, Vbe.SelectedVBComponent);
5453

5554
if (component?.HasDesigner ?? false)
5655
{
5756
if (qualifiedModuleName.HasValue)
5857
{
5958
return _state.DeclarationFinder
60-
.MatchName(qualifiedModuleName.Value.Name)
61-
.SingleOrDefault(m => m.ProjectId == projectId
62-
&& m.DeclarationType.HasFlag(qualifiedModuleName.Value.ComponentType)
63-
&& m.ComponentName == component.Name);
59+
.MatchName(qualifiedModuleName.Value.Name)
60+
.SingleOrDefault(m => m.ProjectId == projectId
61+
&& m.DeclarationType.HasFlag(qualifiedModuleName.Value.ComponentType)
62+
&& m.ComponentName == component.Name);
6463
}
6564

6665
var selectedCount = component.SelectedControls.Count;
6766
if (selectedCount > 1) { return null; }
6867

6968
// Cannot use DeclarationType.UserForm, parser only assigns UserForms the ClassModule flag
70-
(var selectedType, var selectedName) =
71-
selectedCount == 0
72-
? (DeclarationType.ClassModule, component.Name)
73-
: (DeclarationType.Control, component.SelectedControls[0].Name);
69+
(var selectedType, var selectedName) = selectedCount == 0
70+
? (DeclarationType.ClassModule, component.Name)
71+
: (DeclarationType.Control, component.SelectedControls[0].Name);
7472

7573
return _state.DeclarationFinder
76-
.MatchName(selectedName)
77-
.SingleOrDefault(m => m.ProjectId == projectId
78-
&& m.DeclarationType.HasFlag(selectedType)
79-
&& m.ComponentName == component.Name);
74+
.MatchName(selectedName)
75+
.SingleOrDefault(m => m.ProjectId == projectId
76+
&& m.DeclarationType.HasFlag(selectedType)
77+
&& m.ComponentName == component.Name);
8078
}
8179
return null;
8280
}

0 commit comments

Comments
 (0)