Close #1530

Author: Hosch250

RetailCoder.VBE/Inspections/ObjectVariableNotSetInspection.cs

@@ -47,7 +47,7 @@ public SetObjectVariableQuickFix(IdentifierReference reference)
         public override void Fix()
         {
             var codeModule = Selection.QualifiedName.Component.CodeModule;
-            var codeLine = codeModule.get_Lines(Selection.Selection.StartLine, 1);
+            var codeLine = codeModule.Lines[Selection.Selection.StartLine, 1];
 
             var letStatementLeftSide = Context.GetText();
             var setStatementLeftSide = Tokens.Set + ' ' + letStatementLeftSide;
@@ -86,19 +86,26 @@ public ObjectVariableNotSetInspection(RubberduckParserState state)
 
         public override IEnumerable<InspectionResultBase> GetInspectionResults()
         {
-            return State.AllUserDeclarations
-                .Where(item => !ValueTypes.Contains(item.AsTypeName)
-                    && !item.IsSelfAssigned
-                               && (item.DeclarationType == DeclarationType.Variable
-                                   || item.DeclarationType == DeclarationType.Parameter))
+            var interestingDeclarations =
+                State.AllUserDeclarations.Where(item =>
+                        !item.IsSelfAssigned &&
+                        !ValueTypes.Contains(item.AsTypeName) &&
+                        item.AsTypeDeclaration != null &&
+                        item.AsTypeDeclaration.DeclarationType != DeclarationType.Enumeration &&
+                        item.AsTypeDeclaration.DeclarationType != DeclarationType.UserDefinedType &&
+                        (item.DeclarationType == DeclarationType.Variable ||
+                         item.DeclarationType == DeclarationType.Parameter));
+
+            var interestingReferences = interestingDeclarations
                 .SelectMany(declaration =>
                     declaration.References.Where(reference =>
                     {
-                        var k = reference.Context.parent.GetType();
                         var setStmtContext = ParserRuleContextHelper.GetParent<VBAParser.LetStmtContext>(reference.Context);
-                        return setStmtContext != null && setStmtContext.LET() == null;
-                    }))
-                .Select(reference => new ObjectVariableNotSetInspectionResult(this, reference));
+                        return reference.IsAssignment && setStmtContext != null && setStmtContext.LET() == null;
+                    }));
+
+
+            return interestingReferences.Select(reference => new ObjectVariableNotSetInspectionResult(this, reference));
         }
     }
 }