Get rid of explicit call by name arguments.

rubberduck-vba · Jan 17, 2019 · 8ab45d8 · 8ab45d8
1 parent a354a0b
commit 8ab45d8
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/Rubberduck.Core/Navigation/CodeExplorer/CodeExplorerViewModel.cs b/Rubberduck.Core/Navigation/CodeExplorer/CodeExplorerViewModel.cs
@@ -19,17 +19,15 @@
 using Rubberduck.UI.UnitTesting.Commands;
 using Rubberduck.VBEditor.SafeComWrappers.Abstract;
 
-// ReSharper disable ExplicitCallerInfoArgument
-
 namespace Rubberduck.Navigation.CodeExplorer
 {
     [Flags]
     public enum CodeExplorerSortOrder
     {
         Undefined = 0,
         Name = 1,
-        CodeLine = 1 << 2,
-        DeclarationType = 1 << 3,
+        CodeLine = 1 << 1,
+        DeclarationType = 1 << 2,
         DeclarationTypeThenName = DeclarationType | Name,
         DeclarationTypeThenCodeLine = DeclarationType | CodeLine
     }
@@ -76,11 +74,11 @@ public sealed class CodeExplorerViewModel : ViewModelBase
                 RemoveCommand = new DelegateCommand(LogManager.GetCurrentClassLogger(), ExecuteRemoveCommand, _externalRemoveCommand.CanExecute);
             }
 
-            OnPropertyChanged("Projects");
+            OnPropertyChanged(nameof(Projects));
 
             SyncCodePaneCommand = syncProvider.GetSyncCommand(this);
             // Force a call to EvaluateCanExecute
-            OnPropertyChanged("SyncCodePaneCommand");
+            OnPropertyChanged(nameof(SyncCodePaneCommand));
         }
 
         public ObservableCollection<ICodeExplorerNode> Projects { get; } = new ObservableCollection<ICodeExplorerNode>();
@@ -109,8 +107,8 @@ public ICodeExplorerNode SelectedItem
 
                 OnPropertyChanged();
 
-                OnPropertyChanged("ExportVisibility");
-                OnPropertyChanged("ExportAllVisibility");
+                OnPropertyChanged(nameof(ExportVisibility));
+                OnPropertyChanged(nameof(ExportAllVisibility));
             }
         }