Skip to content

Commit a28820b

Browse files
bclothierbclothier
committed
Merge branch 'next' of https://github.com/rubberduck-vba/Rubberduck into MiscBugFixes
# Conflicts: # Rubberduck.Main/Root/RubberduckIoCInstaller.cs # RubberduckTests/Refactoring/EncapsulateFieldTests.cs
2 parents 1911dfe + c776544 commit a28820b

File tree

143 files changed

+4363
-2203
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

143 files changed

+4363
-2203
lines changed

‎Rubberduck.CodeAnalysis/QuickFixes/EncapsulateFieldQuickFix.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,22 +7,23 @@
77
using Rubberduck.Refactorings.EncapsulateField;
88
using Rubberduck.SmartIndenter;
99
using Rubberduck.VBEditor.SafeComWrappers.Abstract;
10+
using Rubberduck.VBEditor.Utility;
1011

1112
namespace Rubberduck.Inspections.QuickFixes
1213
{
1314
public sealed class EncapsulateFieldQuickFix : QuickFixBase
1415
{
1516
private readonly RubberduckParserState _state;
16-
private readonly IVBE _vbe;
17+
private readonly ISelectionService _selectionService;
1718
private readonly IRewritingManager _rewritingManager;
1819
private readonly IIndenter _indenter;
1920
private readonly IRefactoringPresenterFactory _factory;
2021

21-
public EncapsulateFieldQuickFix(RubberduckParserState state, IVBE vbe, IIndenter indenter, IRefactoringPresenterFactory factory, IRewritingManager rewritingManager)
22+
public EncapsulateFieldQuickFix(RubberduckParserState state, IIndenter indenter, IRefactoringPresenterFactory factory, IRewritingManager rewritingManager, ISelectionService selectionService)
2223
: base(typeof(EncapsulatePublicFieldInspection))
2324
{
2425
_state = state;
25-
_vbe = vbe;
26+
_selectionService = selectionService;
2627
_rewritingManager = rewritingManager;
2728
_indenter = indenter;
2829
_factory = factory;
@@ -31,7 +32,7 @@ public EncapsulateFieldQuickFix(RubberduckParserState state, IVBE vbe, IIndenter
3132
//The rewriteSession is optional since it is not used in this particular quickfix because it is a refactoring quickfix.
3233
public override void Fix(IInspectionResult result, IRewriteSession rewriteSession = null)
3334
{
34-
var refactoring = new EncapsulateFieldRefactoring(_state, _vbe, _indenter, _factory, _rewritingManager);
35+
var refactoring = new EncapsulateFieldRefactoring(_state, _indenter, _factory, _rewritingManager, _selectionService);
3536
refactoring.Refactor(result.Target);
3637
}
3738

‎Rubberduck.CodeAnalysis/QuickFixes/MoveFieldCloserToUsageQuickFix.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,20 +7,21 @@
77
using Rubberduck.Refactorings.MoveCloserToUsage;
88
using Rubberduck.Resources.Inspections;
99
using Rubberduck.VBEditor.SafeComWrappers.Abstract;
10+
using Rubberduck.VBEditor.Utility;
1011

1112
namespace Rubberduck.Inspections.QuickFixes
1213
{
1314
public sealed class MoveFieldCloserToUsageQuickFix : QuickFixBase
1415
{
15-
private readonly IVBE _vbe;
16+
private readonly ISelectionService _selectionService;
1617
private readonly RubberduckParserState _state;
1718
private readonly IRewritingManager _rewritingManager;
1819
private readonly IMessageBox _messageBox;
1920

20-
public MoveFieldCloserToUsageQuickFix(IVBE vbe, RubberduckParserState state, IMessageBox messageBox, IRewritingManager rewritingManager)
21+
public MoveFieldCloserToUsageQuickFix(RubberduckParserState state, IMessageBox messageBox, IRewritingManager rewritingManager, ISelectionService selectionService)
2122
: base(typeof(MoveFieldCloserToUsageInspection))
2223
{
23-
_vbe = vbe;
24+
_selectionService = selectionService;
2425
_state = state;
2526
_rewritingManager = rewritingManager;
2627
_messageBox = messageBox;
@@ -29,7 +30,7 @@ public MoveFieldCloserToUsageQuickFix(IVBE vbe, RubberduckParserState state, IMe
2930
//The rewriteSession is optional since it is not used in this particular quickfix because it is a refactoring quickfix.
3031
public override void Fix(IInspectionResult result, IRewriteSession rewriteSession = null)
3132
{
32-
var refactoring = new MoveCloserToUsageRefactoring(_vbe, _state, _messageBox, _rewritingManager);
33+
var refactoring = new MoveCloserToUsageRefactoring(_state, _messageBox, _rewritingManager, _selectionService);
3334
refactoring.Refactor(result.Target);
3435
}
3536

‎Rubberduck.CodeAnalysis/QuickFixes/RemoveUnusedParameterQuickFix.cs

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,30 +6,31 @@
66
using Rubberduck.Refactorings;
77
using Rubberduck.Refactorings.RemoveParameters;
88
using Rubberduck.VBEditor.SafeComWrappers.Abstract;
9+
using Rubberduck.VBEditor.Utility;
910

1011
namespace Rubberduck.Inspections.QuickFixes
1112
{
1213
public sealed class RemoveUnusedParameterQuickFix : QuickFixBase
1314
{
14-
private readonly IVBE _vbe;
15-
private readonly RubberduckParserState _state;
15+
private readonly IDeclarationFinderProvider _declarationFinderProvider;
1616
private readonly IRefactoringPresenterFactory _factory;
1717
private readonly IRewritingManager _rewritingManager;
18+
private readonly ISelectionService _selectionService;
1819

19-
public RemoveUnusedParameterQuickFix(IVBE vbe, RubberduckParserState state, IRefactoringPresenterFactory factory, IRewritingManager rewritingManager)
20+
public RemoveUnusedParameterQuickFix(IDeclarationFinderProvider declarationFinderProvider, IRefactoringPresenterFactory factory, IRewritingManager rewritingManager, ISelectionService selectionService)
2021
: base(typeof(ParameterNotUsedInspection))
2122
{
22-
_vbe = vbe;
23-
_state = state;
23+
_declarationFinderProvider = declarationFinderProvider;
2424
_factory = factory;
2525
_rewritingManager = rewritingManager;
26+
_selectionService = selectionService;
2627
}
2728

2829
//The rewriteSession is optional since it is not used in this particular quickfix because it is a refactoring quickfix.
2930
public override void Fix(IInspectionResult result, IRewriteSession rewriteSession = null)
3031
{
31-
var refactoring = new RemoveParametersRefactoring(_state, _vbe, _factory, _rewritingManager);
32-
refactoring.QuickFix(_state, result.QualifiedSelection);
32+
var refactoring = new RemoveParametersRefactoring(_declarationFinderProvider, _factory, _rewritingManager, _selectionService);
33+
refactoring.QuickFix(result.QualifiedSelection);
3334
}
3435

3536
public override string Description(IInspectionResult result) => Resources.Inspections.QuickFixes.RemoveUnusedParameterQuickFix;

‎Rubberduck.CodeAnalysis/QuickFixes/RenameDeclarationQuickFix.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,25 +10,26 @@
1010
using Rubberduck.Refactorings.Rename;
1111
using Rubberduck.Resources;
1212
using Rubberduck.VBEditor.SafeComWrappers.Abstract;
13+
using Rubberduck.VBEditor.Utility;
1314

1415
namespace Rubberduck.Inspections.QuickFixes
1516
{
1617
public sealed class RenameDeclarationQuickFix : QuickFixBase
1718
{
18-
private readonly IVBE _vbe;
19+
private readonly ISelectionService _selectionService;
1920
private readonly RubberduckParserState _state;
2021
private readonly IRewritingManager _rewritingManager;
2122
private readonly IMessageBox _messageBox;
2223
private readonly IRefactoringPresenterFactory _factory;
2324

24-
public RenameDeclarationQuickFix(IVBE vbe, RubberduckParserState state, IMessageBox messageBox, IRefactoringPresenterFactory factory, IRewritingManager rewritingManager)
25+
public RenameDeclarationQuickFix(RubberduckParserState state, IMessageBox messageBox, IRefactoringPresenterFactory factory, IRewritingManager rewritingManager, ISelectionService selectionService)
2526
: base(typeof(HungarianNotationInspection),
2627
typeof(UseMeaningfulNameInspection),
2728
typeof(DefaultProjectNameInspection),
2829
typeof(UnderscoreInPublicClassModuleMemberInspection),
2930
typeof(ExcelUdfNameIsValidCellReferenceInspection))
3031
{
31-
_vbe = vbe;
32+
_selectionService = selectionService;
3233
_state = state;
3334
_rewritingManager = rewritingManager;
3435
_messageBox = messageBox;
@@ -38,7 +39,7 @@ public RenameDeclarationQuickFix(IVBE vbe, RubberduckParserState state, IMessage
3839
//The rewriteSession is optional since it is not used in this particular quickfix because it is a refactoring quickfix.
3940
public override void Fix(IInspectionResult result, IRewriteSession rewriteSession = null)
4041
{
41-
var refactoring = new RenameRefactoring(_vbe, _factory, _messageBox, _state, _state.ProjectsProvider, _rewritingManager);
42+
var refactoring = new RenameRefactoring(_factory, _messageBox, _state, _state.ProjectsProvider, _rewritingManager, _selectionService);
4243
refactoring.Refactor(result.Target);
4344
}
4445

‎Rubberduck.Core/AutoComplete/AutoCompleteHandlerBase.cs

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@
55

66
namespace Rubberduck.AutoComplete
77
{
8+
/// <summary>
9+
/// A base class/interface for AC services / "handlers".
10+
/// </summary>
811
public abstract class AutoCompleteHandlerBase
912
{
1013
protected AutoCompleteHandlerBase(ICodePaneHandler pane)
@@ -14,6 +17,13 @@ protected AutoCompleteHandlerBase(ICodePaneHandler pane)
1417

1518
protected ICodePaneHandler CodePaneHandler { get; }
1619

20+
/// <summary>
21+
/// A method that returns <c>false</c> if the input isn't handled, <c>true</c> if it is.
22+
/// </summary>
23+
/// <param name="e">The autocompletion event info</param>
24+
/// <param name="settings">The current AC settings</param>
25+
/// <param name="result">If handled, the resulting <c>CodeString</c></param>
26+
/// <returns></returns>
1727
public abstract bool Handle(AutoCompleteEventArgs e, AutoCompleteSettings settings, out CodeString result);
1828
}
1929
}

‎Rubberduck.Core/AutoComplete/Service/AutoCompleteService.cs renamed to ‎Rubberduck.Core/AutoComplete/AutoCompleteService.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,12 @@
55
using Rubberduck.Settings;
66
using Rubberduck.VBEditor.Events;
77

8-
namespace Rubberduck.AutoComplete.Service
8+
namespace Rubberduck.AutoComplete
99
{
10+
/// <summary>
11+
/// A service responsible for dispatching CodePane work to more specialized autocompletion services.
12+
/// Handles changes in configuration settings.
13+
/// </summary>
1014
public class AutoCompleteService : IDisposable
1115
{
1216
private static readonly ILogger Logger = LogManager.GetCurrentClassLogger();
@@ -151,6 +155,7 @@ private bool TryHandle(AutoCompleteEventArgs e, AutoCompleteHandlerBase handler)
151155
return false;
152156
}
153157

158+
Logger.Debug($"Keypress was handled by {handler.GetType().Name}.");
154159
e.Handled = true;
155160
return true;
156161

@@ -169,7 +174,8 @@ public void Dispose()
169174
}
170175

171176
private bool _isDisposed;
172-
protected virtual void Dispose(bool disposing)
177+
178+
private void Dispose(bool disposing)
173179
{
174180
if (_isDisposed || !disposing)
175181
{

‎Rubberduck.Core/AutoComplete/Service/SelfClosingPair.cs renamed to ‎Rubberduck.Core/AutoComplete/SelfClosingPairs/SelfClosingPair.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
using Rubberduck.VBEditor;
2-
using System;
1+
using System;
2+
using Rubberduck.VBEditor;
33

4-
namespace Rubberduck.AutoComplete.Service
4+
namespace Rubberduck.AutoComplete.SelfClosingPairs
55
{
66
public class SelfClosingPair : IEquatable<SelfClosingPair>
77
{

‎Rubberduck.Core/AutoComplete/Service/SelfClosingPairCompletionService.cs renamed to ‎Rubberduck.Core/AutoComplete/SelfClosingPairs/SelfClosingPairCompletionService.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
using Rubberduck.Parsing.VBA.Parsing;
99
using Rubberduck.VBEditor;
1010

11-
namespace Rubberduck.AutoComplete.Service
11+
namespace Rubberduck.AutoComplete.SelfClosingPairs
1212
{
1313
public class SelfClosingPairCompletionService
1414
{

‎Rubberduck.Core/AutoComplete/Service/SelfClosingPairHandler.cs renamed to ‎Rubberduck.Core/AutoComplete/SelfClosingPairs/SelfClosingPairHandler.cs

Lines changed: 34 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,16 @@
1-
using System.Collections.Generic;
2-
using System.Diagnostics;
1+
using System;
2+
using System.Collections.Generic;
33
using System.Linq;
44
using Rubberduck.Settings;
55
using Rubberduck.VBEditor;
66
using Rubberduck.VBEditor.Events;
77
using Rubberduck.VBEditor.SourceCodeHandling;
88

9-
namespace Rubberduck.AutoComplete.Service
9+
namespace Rubberduck.AutoComplete.SelfClosingPairs
1010
{
11+
/// <summary>
12+
/// An AC handler that automatically closes certain specific "pairs" of characters, e.g. double quotes, or parentheses.
13+
/// </summary>
1114
public class SelfClosingPairHandler : AutoCompleteHandlerBase
1215
{
1316
private const int MaximumLines = 25;
@@ -39,6 +42,7 @@ public override bool Handle(AutoCompleteEventArgs e, AutoCompleteSettings settin
3942
result = null;
4043
if (!_scpInputLookup.TryGetValue(e.Character, out var pair) && e.Character != '\b')
4144
{
45+
// not an interesting keypress.
4246
return false;
4347
}
4448

@@ -50,15 +54,25 @@ public override bool Handle(AutoCompleteEventArgs e, AutoCompleteSettings settin
5054
return false;
5155
}
5256

57+
if (!original.CaretPosition.IsSingleCharacter)
58+
{
59+
// here would be an opportunity to "wrap selection" with a SCP.
60+
// todo: WrapSelectionWith(pair)?
61+
result = null;
62+
return false;
63+
}
64+
5365
if (pair != null)
5466
{
67+
// found a SCP for the input key; see if we should handle it:
5568
if (!HandleInternal(e, original, pair, out result))
5669
{
5770
return false;
5871
}
5972
}
6073
else if (e.Character == '\b')
6174
{
75+
// backspace - see if SCP logic needs to intervene:
6276
foreach (var scp in _selfClosingPairs)
6377
{
6478
if (HandleInternal(e, original, scp, out result))
@@ -70,9 +84,11 @@ public override bool Handle(AutoCompleteEventArgs e, AutoCompleteSettings settin
7084

7185
if (result == null)
7286
{
87+
// no meaningful output; let the input be handled by another handler, maybe.
7388
return false;
7489
}
7590

91+
// 1-based selection span in the code pane starts at column 1 but really encompasses the entire line.
7692
var snippetPosition = new Selection(result.SnippetPosition.StartLine, 1, result.SnippetPosition.EndLine, 1);
7793
result = new CodeString(result.Code, result.CaretPosition, snippetPosition);
7894

@@ -82,13 +98,6 @@ public override bool Handle(AutoCompleteEventArgs e, AutoCompleteSettings settin
8298

8399
private bool HandleInternal(AutoCompleteEventArgs e, CodeString original, SelfClosingPair pair, out CodeString result)
84100
{
85-
if (!original.CaretPosition.IsSingleCharacter)
86-
{
87-
// todo: WrapSelectionWith(pair)?
88-
result = null;
89-
return false;
90-
}
91-
92101
// if executing the SCP against the original code yields no result, we need to bail out.
93102
if (!_scpService.Execute(pair, original, e.Character, out result))
94103
{
@@ -115,21 +124,34 @@ private bool HandleInternal(AutoCompleteEventArgs e, CodeString original, SelfCl
115124
);
116125
}
117126

127+
if (original.CaretLine.EndsWith(" ") &&
128+
string.Equals(original.CaretLine, prettified.CaretLine + " ", StringComparison.InvariantCultureIgnoreCase))
129+
{
130+
prettified = original;
131+
}
132+
118133
// if executing the SCP against the prettified code yields no result, we need to bail out.
119134
if (!_scpService.Execute(pair, prettified, e.Character, out result))
120135
{
121136
return false;
122137
}
123138

124139
var reprettified = CodePaneHandler.Prettify(e.Module, result);
125-
if (pair.OpeningChar == '(' && e.Character == pair.OpeningChar && !reprettified.Equals(result))
140+
if (pair.OpeningChar == '(' && e.Character == pair.OpeningChar)
126141
{
142+
if (string.Equals(reprettified.Code, result.Code, StringComparison.InvariantCultureIgnoreCase))
143+
{
144+
e.Handled = true;
145+
result = reprettified;
146+
return true;
147+
}
148+
127149
// VBE eats it. bail out but don't swallow the keypress.
128150
e.Handled = false;
129151
result = null;
130152
return false;
131153
}
132-
154+
133155
var currentLine = reprettified.Lines[reprettified.CaretPosition.StartLine];
134156
if (!string.IsNullOrWhiteSpace(currentLine) &&
135157
currentLine.EndsWith(" ") &&

‎Rubberduck.Core/AutoComplete/Service/ShowIntelliSenseCommand.cs renamed to ‎Rubberduck.Core/AutoComplete/ShowIntelliSenseCommand.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
using Rubberduck.UI.Command;
44
using Rubberduck.VBEditor.SafeComWrappers.Abstract;
55

6-
namespace Rubberduck.AutoComplete.Service
6+
namespace Rubberduck.AutoComplete
77
{
88
public interface IShowIntelliSenseCommand
99
{

0 commit comments

Comments
 (0)