Separate GetModuleState and GetOrCreateModuleState

Author: Hosch250

RetailCoder.VBE/UI/CodeExplorer/Commands/CodeExplorer_RefreshComponentCommand.cs

@@ -18,7 +18,7 @@ public override bool CanExecute(object parameter)
             var node = parameter as CodeExplorerComponentViewModel;
 
             return node != null && node.QualifiedSelection.HasValue &&
-                   _state.GetModuleState(node.QualifiedSelection.Value.QualifiedName.Component) == ParserState.Pending;
+                   _state.GetOrCreateModuleState(node.QualifiedSelection.Value.QualifiedName.Component) == ParserState.Pending;
         }
 
         public override void Execute(object parameter)

Rubberduck.Parsing/VBA/RubberduckParserState.cs

@@ -267,28 +267,30 @@ private ParserState EvaluateParserState()
             return result;
         }
 
-        public ParserState GetModuleState(VBComponent component)
+        public ParserState GetOrCreateModuleState(VBComponent component)
         {
             var key = new QualifiedModuleName(component);
-
-            var isDirty = IsNewOrModified(key);
             var state = _moduleStates.GetOrAdd(key, ParserState.Pending);
-            
-            if (!isDirty)
-            {
-                return state;
-            }
 
             if (state == ParserState.Pending)
             {
                 return state;   // we are slated for a reparse already
             }
 
-            _moduleStates.TryUpdate(key, ParserState.Pending, state);
+            if (!IsNewOrModified(key))
+            {
+                return state;
+            }
 
+            _moduleStates.TryUpdate(key, ParserState.Pending, state);
             return ParserState.Pending;
         }
 
+        public ParserState GetModuleState(VBComponent component)
+        {
+            return _moduleStates.GetOrAdd(new QualifiedModuleName(component), ParserState.Pending);
+        }
+
         private ParserState _status;
         public ParserState Status
         {