Merge pull request #3424 from rubberduck-vba/SaferComWrappers

Prevents iterating VBComponents collection on a locked project.

Author: retailcoder

Rubberduck.VBEEditor/SafeComWrappers/VBA/VBComponents.cs

@@ -99,6 +99,8 @@ public override int GetHashCode()
 
         public void ImportSourceFile(string path)
         {
+            if (IsWrappingNullReference) { return; }
+
             var ext = Path.GetExtension(path);
             var name = Path.GetFileNameWithoutExtension(path);
             if (!File.Exists(path))
@@ -152,6 +154,8 @@ public void ImportSourceFile(string path)
 
         public void RemoveSafely(IVBComponent component)
         {
+            if (component.IsWrappingNullReference) { return; }
+
             switch (component.Type)
             {
                 case ComponentType.ClassModule:
@@ -174,7 +178,7 @@ private static void AttachEvents(VB.VBComponents components)
         {
             lock (Locker)
             {
-                if (_components == null)
+                if (_components == null && components != null)
                 {
                     _components = components;
                     _componentAdded = OnComponentAdded;

Rubberduck.VBEEditor/SafeComWrappers/VBA/VBProject.cs

@@ -60,7 +60,7 @@ public IReferences References
 
         public IVBComponents VBComponents
         {
-            get { return new VBComponents(IsWrappingNullReference ? null : Target.VBComponents); }
+            get { return new VBComponents(IsWrappingNullReference || Protection == ProjectProtection.Locked ? null : Target.VBComponents); }
         }
 
         public ProjectProtection Protection
@@ -232,4 +232,4 @@ public string ProjectDisplayName
             }
         }
     }
-}
+}