Skip to content

Commit 5e07cf7

Browse files
committed
Get RHS Dflt Parameter from Resource.Refactorings
1 parent 2311295 commit 5e07cf7

File tree

6 files changed

+14
-22
lines changed

6 files changed

+14
-22
lines changed

Rubberduck.Refactorings/Common/CodeBuilder.cs

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -68,16 +68,10 @@ bool TryBuildPropertySetCodeBlock(Declaration prototype,
6868
string accessibility = null,
6969
string content = null,
7070
string parameterIdentifier = null);
71-
/// <summary>
72-
/// Default RHS property parameter IdentifierName
73-
/// </summary>
74-
string DefaultLetSetRhsParameterIdentifierName { get; }
7571
}
7672

7773
public class CodeBuilder : ICodeBuilder
7874
{
79-
public string DefaultLetSetRhsParameterIdentifierName => "RHS";
80-
8175
public string BuildMemberBlockFromPrototype(ModuleBodyElementDeclaration declaration,
8276
string content = null,
8377
string accessibility = null,
@@ -117,7 +111,7 @@ private bool TryBuildPropertyBlockFromTarget<T>(T prototype, DeclarationType let
117111
return false;
118112
}
119113

120-
var propertyValueParam = parameterIdentifier ?? DefaultLetSetRhsParameterIdentifierName;
114+
var propertyValueParam = parameterIdentifier ?? Resources.Refactorings.Refactorings.CodeBuilder_DefaultPropertyRHSParam;
121115

122116
var asType = prototype.IsArray
123117
? $"{Tokens.Variant}"

Rubberduck.Refactorings/EncapsulateField/EncapsulateFieldElementsBuilder.cs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
using Rubberduck.Parsing.Grammar;
2-
using Rubberduck.Parsing.Symbols;
1+
using Rubberduck.Parsing.Symbols;
32
using Rubberduck.Parsing.VBA;
43
using Rubberduck.Refactorings.Common;
54
using Rubberduck.VBEditor;
@@ -111,7 +110,7 @@ private IEncapsulateFieldCandidate CreateCandidate(Declaration target, IValidate
111110
if (target.IsUserDefinedType())
112111
{
113112
var udtValidator = EncapsulateFieldValidationsProvider.NameOnlyValidator(NameValidators.UserDefinedType);
114-
var udtField = new UserDefinedTypeCandidate(target, udtValidator, _codeBuilder.DefaultLetSetRhsParameterIdentifierName) as IUserDefinedTypeCandidate;
113+
var udtField = new UserDefinedTypeCandidate(target, udtValidator) as IUserDefinedTypeCandidate;
115114

116115
(Declaration udtDeclaration, IEnumerable<Declaration> udtMembers) = GetUDTAndMembersForField(udtField);
117116

@@ -126,7 +125,7 @@ private IEncapsulateFieldCandidate CreateCandidate(Declaration target, IValidate
126125
{
127126
udtMemberValidator = EncapsulateFieldValidationsProvider.NameOnlyValidator(NameValidators.UserDefinedTypeMemberArray);
128127
}
129-
var candidateUDTMember = new UserDefinedTypeMemberCandidate(CreateCandidate(udtMemberDeclaration, udtMemberValidator), udtField, _codeBuilder.DefaultLetSetRhsParameterIdentifierName) as IUserDefinedTypeMemberCandidate;
128+
var candidateUDTMember = new UserDefinedTypeMemberCandidate(CreateCandidate(udtMemberDeclaration, udtMemberValidator), udtField) as IUserDefinedTypeMemberCandidate;
130129

131130
udtField.AddMember(candidateUDTMember);
132131
}
@@ -142,10 +141,10 @@ private IEncapsulateFieldCandidate CreateCandidate(Declaration target, IValidate
142141
}
143142
else if (target.IsArray)
144143
{
145-
return new ArrayCandidate(target, validator, _codeBuilder.DefaultLetSetRhsParameterIdentifierName);
144+
return new ArrayCandidate(target, validator);
146145
}
147146

148-
var candidate = new EncapsulateFieldCandidate(target, validator, _codeBuilder.DefaultLetSetRhsParameterIdentifierName);
147+
var candidate = new EncapsulateFieldCandidate(target, validator);
149148
return candidate;
150149
}
151150

Rubberduck.Refactorings/EncapsulateField/FieldCandidates/ArrayCandidate.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
using Rubberduck.Parsing.Grammar;
33
using Rubberduck.Parsing.Symbols;
44
using Rubberduck.Resources;
5-
using System;
65
using System.Linq;
76

87
namespace Rubberduck.Refactorings.EncapsulateField
@@ -16,8 +15,8 @@ public interface IArrayCandidate : IEncapsulateFieldCandidate
1615
public class ArrayCandidate : EncapsulateFieldCandidate, IArrayCandidate
1716
{
1817
private string _subscripts;
19-
public ArrayCandidate(Declaration declaration, IValidateVBAIdentifiers validator, string rhsParameterIdentifierName)
20-
:base(declaration, validator, rhsParameterIdentifierName)
18+
public ArrayCandidate(Declaration declaration, IValidateVBAIdentifiers validator)
19+
:base(declaration, validator)
2120
{
2221
ImplementLet = false;
2322
ImplementSet = false;

Rubberduck.Refactorings/EncapsulateField/FieldCandidates/EncapsulateFieldCandidate.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,11 +47,11 @@ public class EncapsulateFieldCandidate : IEncapsulateFieldCandidate
4747
protected EncapsulationIdentifiers _fieldAndProperty;
4848
private string _rhsParameterIdentifierName;
4949

50-
public EncapsulateFieldCandidate(Declaration declaration, IValidateVBAIdentifiers identifierValidator, string rhsParameterIdentifierName)
50+
public EncapsulateFieldCandidate(Declaration declaration, IValidateVBAIdentifiers identifierValidator)
5151
{
5252
_target = declaration;
5353
NameValidator = identifierValidator;
54-
_rhsParameterIdentifierName = rhsParameterIdentifierName;
54+
_rhsParameterIdentifierName = Resources.Refactorings.Refactorings.CodeBuilder_DefaultPropertyRHSParam;
5555

5656
_fieldAndProperty = new EncapsulationIdentifiers(declaration.IdentifierName, identifierValidator);
5757
IdentifierName = declaration.IdentifierName;

Rubberduck.Refactorings/EncapsulateField/FieldCandidates/UserDefinedTypeCandidate.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ public interface IUserDefinedTypeCandidate : IEncapsulateFieldCandidate
1717

1818
public class UserDefinedTypeCandidate : EncapsulateFieldCandidate, IUserDefinedTypeCandidate
1919
{
20-
public UserDefinedTypeCandidate(Declaration declaration, IValidateVBAIdentifiers identifierValidator, string rhsParameterIdentifierName)
21-
: base(declaration, identifierValidator, rhsParameterIdentifierName)
20+
public UserDefinedTypeCandidate(Declaration declaration, IValidateVBAIdentifiers identifierValidator)
21+
: base(declaration, identifierValidator)
2222
{
2323
}
2424

Rubberduck.Refactorings/EncapsulateField/FieldCandidates/UserDefinedTypeMemberCandidate.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,10 @@ public class UserDefinedTypeMemberCandidate : IUserDefinedTypeMemberCandidate
2121
private int _hashCode;
2222
private readonly string _uniqueID;
2323
private string _rhsParameterIdentifierName;
24-
public UserDefinedTypeMemberCandidate(IEncapsulateFieldCandidate candidate, IUserDefinedTypeCandidate udtField, string rhsParameterIdentifierName)
24+
public UserDefinedTypeMemberCandidate(IEncapsulateFieldCandidate candidate, IUserDefinedTypeCandidate udtField)
2525
{
2626
_wrappedCandidate = candidate;
27-
_rhsParameterIdentifierName = rhsParameterIdentifierName;
27+
_rhsParameterIdentifierName = Resources.Refactorings.Refactorings.CodeBuilder_DefaultPropertyRHSParam;
2828
UDTField = udtField;
2929
PropertyIdentifier = IdentifierName;
3030
BackingIdentifier = IdentifierName;

0 commit comments

Comments
 (0)