Code Quality: Moved static methods from MainPageViewModel to NavigationHelpers (#14027)

Author: 0x5bfa

src/Files.App/Actions/Navigation/DuplicateCurrentTabAction.cs

@@ -7,8 +7,6 @@ internal class DuplicateCurrentTabAction : IAction
 	{
 		private readonly IMultitaskingContext context;
 
-		private readonly MainPageViewModel mainPageViewModel;
-
 		public string Label
 			=> "DuplicateTab".GetLocalizedResource();
 
@@ -18,16 +16,23 @@ public string Description
 		public DuplicateCurrentTabAction()
 		{
 			context = Ioc.Default.GetRequiredService<IMultitaskingContext>();
-			mainPageViewModel = Ioc.Default.GetRequiredService<MainPageViewModel>();
 		}
 
 		public async Task ExecuteAsync()
 		{
 			var arguments = context.CurrentTabItem.NavigationParameter;
+
 			if (arguments is null)
-				await mainPageViewModel.AddNewTabByPathAsync(typeof(PaneHolderPage), "Home");
+			{
+				await NavigationHelpers.AddNewTabByPathAsync(typeof(PaneHolderPage), "Home");
+			}
 			else
-				await mainPageViewModel.AddNewTabByParamAsync(arguments.InitialPageType, arguments.NavigationParameter, context.CurrentTabIndex + 1);
+			{
+				await NavigationHelpers.AddNewTabByParamAsync(
+					arguments.InitialPageType,
+					arguments.NavigationParameter,
+					context.CurrentTabIndex + 1);
+			}
 		}
 	}
 }

src/Files.App/Actions/Navigation/DuplicateSelectedTabAction.cs

@@ -7,8 +7,6 @@ internal class DuplicateSelectedTabAction : IAction
 	{
 		private readonly IMultitaskingContext context;
 
-		private readonly MainPageViewModel mainPageViewModel;
-
 		public string Label
 			=> "DuplicateTab".GetLocalizedResource();
 
@@ -21,16 +19,20 @@ public HotKey HotKey
 		public DuplicateSelectedTabAction()
 		{
 			context = Ioc.Default.GetRequiredService<IMultitaskingContext>();
-			mainPageViewModel = Ioc.Default.GetRequiredService<MainPageViewModel>();
 		}
 
 		public async Task ExecuteAsync()
 		{
 			var arguments = context.SelectedTabItem.NavigationParameter;
+
 			if (arguments is null)
-				await mainPageViewModel.AddNewTabByPathAsync(typeof(PaneHolderPage), "Home");
+			{
+				await NavigationHelpers.AddNewTabByPathAsync(typeof(PaneHolderPage), "Home");
+			}
 			else
-				await mainPageViewModel.AddNewTabByParamAsync(arguments.InitialPageType, arguments.NavigationParameter, context.SelectedTabIndex + 1);
+			{
+				await NavigationHelpers.AddNewTabByParamAsync(arguments.InitialPageType, arguments.NavigationParameter, context.SelectedTabIndex + 1);
+			}
 		}
 	}
 }

src/Files.App/Actions/Navigation/NewTabAction.cs

@@ -5,8 +5,6 @@ namespace Files.App.Actions
 {
 	internal class NewTabAction : IAction
 	{
-		private readonly MainPageViewModel mainPageViewModel;
-
 		public string Label
 			=> "NewTab".GetLocalizedResource();
 
@@ -18,12 +16,11 @@ public HotKey HotKey
 
 		public NewTabAction()
 		{
-			mainPageViewModel = Ioc.Default.GetRequiredService<MainPageViewModel>();
 		}
 
 		public Task ExecuteAsync()
 		{
-			return mainPageViewModel.AddNewTabAsync();
+			return NavigationHelpers.AddNewTabAsync();
 		}
 	}
 }

src/Files.App/Actions/Navigation/OpenDirectoryInNewTabAction.cs

@@ -9,8 +9,6 @@ internal class OpenDirectoryInNewTabAction : ObservableObject, IAction
 
 		private readonly IUserSettingsService userSettingsService;
 
-		private readonly MainPageViewModel _mainPageViewModel;
-
 		public string Label
 			=> "OpenInNewTab".GetLocalizedResource();
 
@@ -31,7 +29,6 @@ public OpenDirectoryInNewTabAction()
 		{
 			context = Ioc.Default.GetRequiredService<IContentPageContext>();
 			userSettingsService = Ioc.Default.GetRequiredService<IUserSettingsService>();
-			_mainPageViewModel = Ioc.Default.GetRequiredService<MainPageViewModel>();
 
 			context.PropertyChanged += Context_PropertyChanged;
 		}
@@ -45,7 +42,7 @@ public async Task ExecuteAsync()
 			{
 				await MainWindow.Instance.DispatcherQueue.EnqueueOrInvokeAsync(async () =>
 				{
-					await _mainPageViewModel.AddNewTabByPathAsync(
+					await NavigationHelpers.AddNewTabByPathAsync(
 						typeof(PaneHolderPage),
 						(listedItem as ShortcutItem)?.TargetPath ?? listedItem.ItemPath);
 				},