Merge pull request #59 from nventive/dev/jpl/fix-set-active-section

jeanplevesque · web-flow · commit 77822d31b1a4 · 2024-02-22T16:02:32.000-05:00
fix: Fix old regression in SetActiveSection.
diff --git a/src/SectionsNavigation.Abstractions/ISectionsNavigator.Extensions.cs b/src/SectionsNavigation.Abstractions/ISectionsNavigator.Extensions.cs
@@ -76,7 +76,7 @@ public static async Task<ISectionStackNavigator> SetActiveSection(this ISections
 				else
 				{
 					// If the section root page isn't in the stack, clear everything and navigate to it.
-					await sectionNavigator.Navigate(ct, StackNavigatorRequest.GetNavigateRequest(viewModelProvider, suppressTransitions: true, clearBackStack: true));
+					await sectionNavigator.Navigate(ct, StackNavigatorRequest.GetNavigateRequest(viewModelType, viewModelProvider, suppressTransitions: true, clearBackStack: true));
 				}
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ public static async Task<ISectionStackNavigator> SetActiveSection(this ISections`
`76`	`76`	`else`
`77`	`77`	`{`
`78`	`78`	`// If the section root page isn't in the stack, clear everything and navigate to it.`
`79`		`- await sectionNavigator.Navigate(ct, StackNavigatorRequest.GetNavigateRequest(viewModelProvider, suppressTransitions: true, clearBackStack: true));`
	`79`	`+ await sectionNavigator.Navigate(ct, StackNavigatorRequest.GetNavigateRequest(viewModelType, viewModelProvider, suppressTransitions: true, clearBackStack: true));`
`80`	`80`	`}`
`81`	`81`	`}`
`82`	`82`