Merge pull request #4940 from bclothier/HotFixTodoRefreshPrematurely

Improves the execution condition for the ToDo Explorer refresh

Author: retailcoder

Rubberduck.Core/UI/ToDoItems/ToDoExplorerViewModel.cs

@@ -53,9 +53,31 @@ public ToDoExplorerViewModel(
             RefreshCommand = new DelegateCommand(LogManager.GetCurrentClassLogger(),
                 _ =>
                 {
-                    _state.OnParseRequested(this);
+                    switch(_state.Status)
+                    {
+                        case ParserState.Ready:
+                        case ParserState.Error:
+                        case ParserState.ResolverError:
+                        case ParserState.UnexpectedError:
+                        case ParserState.Pending:
+                            _state.OnParseRequested(this);
+                            break;
+                    }
                 },
-                _ => _state.IsDirty());
+                _ =>
+                {
+                    switch (_state.Status)
+                    {
+                        case ParserState.Ready:
+                        case ParserState.Error:
+                        case ParserState.ResolverError:
+                        case ParserState.UnexpectedError:
+                        case ParserState.Pending:
+                            return true;
+                        default:
+                            return false;
+                    }
+                });
             NavigateCommand = new NavigateCommand(selectionService);
             RemoveCommand = new DelegateCommand(LogManager.GetCurrentClassLogger(), ExecuteRemoveCommand, CanExecuteRemoveCommand);
             CollapseAllCommand = new DelegateCommand(LogManager.GetCurrentClassLogger(), ExecuteCollapseAll);