Merge pull request #1837 from Hosch250/Issue1814

Don't fire ResolvedDeclarations unless the project parsed and resolve…

Author: Hosch250

Rubberduck.Parsing/VBA/RubberduckParser.cs

@@ -84,11 +84,10 @@ private void ReparseRequested(object sender, ParseRequestEventArgs e)
                     _state.SetModuleState(e.Component, ParserState.Resolving);
                     ResolveDeclarations(qualifiedName.Component,
                         _state.ParseTrees.Find(s => s.Key == qualifiedName).Value);
-
-                    _state.SetStatusAndFireStateChanged(ParserState.ResolvedDeclarations);
-
+                    
                     if (_state.Status < ParserState.Error)
                     {
+                        _state.SetStatusAndFireStateChanged(ParserState.ResolvedDeclarations);
                         ResolveReferencesAsync();
                     }
                 });
@@ -170,10 +169,10 @@ public void Parse()
             }
 
             Task.WaitAll(parseTasks);
-            _state.SetStatusAndFireStateChanged(ParserState.ResolvedDeclarations);
 
             if (_state.Status < ParserState.Error)
             {
+                _state.SetStatusAndFireStateChanged(ParserState.ResolvedDeclarations);
                 Task.WaitAll(ResolveReferencesAsync());
             }
         }
@@ -277,10 +276,10 @@ private void ParseAll()
             }
 
             Task.WaitAll(parseTasks);
-            _state.SetStatusAndFireStateChanged(ParserState.ResolvedDeclarations);
 
             if (_state.Status < ParserState.Error)
             {
+                _state.SetStatusAndFireStateChanged(ParserState.ResolvedDeclarations);
                 ResolveReferencesAsync();
             }
         }