Merge pull request #3775 from MDoerner/FixForHasDefaultMember

Fix for AttributeNode

Author: retailcoder

Rubberduck.Parsing/VBA/Attributes.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Collections.Generic;
-using System.Diagnostics;
 using System.Linq;
 using Rubberduck.Parsing.Annotations;
 using Rubberduck.Parsing.Grammar;
@@ -21,33 +20,31 @@ public enum Instancing
 
     public class AttributeNode : IEquatable<AttributeNode>
     {
-        private readonly string _name;
         private readonly IList<string> _values;
 
         public AttributeNode(VBAParser.AttributeStmtContext context)
         {
             Context = context;
+            Name = Context?.attributeName().GetText() ?? String.Empty;
+            _values = Context?.attributeValue().Select(a => a.GetText()).ToList() ?? new List<string>();
         }
 
         public AttributeNode(string name, IEnumerable<string> values)
         {
-            _name = name;
+            Name = name;
             _values = values.ToList();
         }
 
         public VBAParser.AttributeStmtContext Context { get; }
 
-        public string Name => Context?.attributeName().GetText() ?? _name;
-
+        public string Name { get; }
+    
         public void AddValue(string value)
         {
             _values.Add(value);
         }
 
-        public IReadOnlyList<string> Values
-        {
-            get { return Context?.attributeValue().Select(a => a.GetText()).ToArray() ?? _values.ToArray(); }
-        }
+        public IReadOnlyCollection<string> Values => _values.AsReadOnly();
 
         public bool HasValue(string value)
         {