Skip to content

Commit d28bf6a

Browse files
committed
Revert userform declaration type back to DeclarationType.ClassModule.
1 parent 688d922 commit d28bf6a

File tree

3 files changed

+8
-15
lines changed

3 files changed

+8
-15
lines changed

Rubberduck.Core/UI/Converters/CodeExplorerNodeToIconConverter.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,9 @@ public override object Convert(object value, Type targetType, object parameter,
9898
return InterfaceIcon;
9999
}
100100

101+
var isUserForm = component.Declaration.QualifiedModuleName.ComponentType == VBEditor.SafeComWrappers.ComponentType.UserForm;
101102
return DeclarationIcons.ContainsKey(component.Declaration.DeclarationType)
102-
? DeclarationIcons[component.Declaration.DeclarationType]
103+
? DeclarationIcons[isUserForm ? DeclarationType.UserForm : component.Declaration.DeclarationType]
103104
: ExceptionIcon;
104105
default:
105106
return value is ICodeExplorerNode node &&

Rubberduck.Parsing/Symbols/ClassModuleDeclaration.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,13 +29,12 @@ public ClassModuleDeclaration(
2929
bool isWithEvents = false,
3030
bool hasDefaultInstanceVariable = false,
3131
bool isControl = false,
32-
bool isDocument = false,
33-
bool isUserForm = false)
32+
bool isDocument = false)
3433
: base(
3534
qualifiedName,
3635
projectDeclaration,
3736
name,
38-
isDocument ? DeclarationType.Document : isUserForm ? DeclarationType.UserForm : DeclarationType.ClassModule,
37+
isDocument ? DeclarationType.Document : DeclarationType.ClassModule,
3938
isUserDefined,
4039
annotations,
4140
attributes,

Rubberduck.Parsing/VBA/DeclarationResolving/DeclarationResolveRunnerBase.cs

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,7 @@ private ModuleDeclaration NewModuleDeclaration(
191191
true,
192192
moduleAnnotations,
193193
moduleAttributes);
194+
194195
case ComponentType.ClassModule:
195196
return new ClassModuleDeclaration(
196197
qualifiedModuleName.QualifyMemberName(qualifiedModuleName.ComponentName),
@@ -199,24 +200,16 @@ private ModuleDeclaration NewModuleDeclaration(
199200
true,
200201
moduleAnnotations,
201202
moduleAttributes);
202-
case ComponentType.UserForm:
203-
return new ClassModuleDeclaration(
204-
qualifiedModuleName.QualifyMemberName(qualifiedModuleName.ComponentName),
205-
projectDeclaration,
206-
qualifiedModuleName.ComponentName,
207-
true,
208-
moduleAnnotations,
209-
moduleAttributes,
210-
hasDefaultInstanceVariable:true,
211-
isUserForm:true);
203+
212204
case ComponentType.Document:
213205
return new DocumentModuleDeclaration(
214206
qualifiedModuleName.QualifyMemberName(qualifiedModuleName.ComponentName),
215207
projectDeclaration,
216208
qualifiedModuleName.ComponentName,
217209
moduleAnnotations,
218210
moduleAttributes);
219-
default:
211+
212+
default: /*ComponentType.UserForm*/
220213
return new ClassModuleDeclaration(
221214
qualifiedModuleName.QualifyMemberName(qualifiedModuleName.ComponentName),
222215
projectDeclaration,

0 commit comments

Comments
 (0)