fix: preserve navigated.current value

flow-server/src/main/resources/com/vaadin/flow/server/frontend/Flow.tsx

@@ -165,7 +165,7 @@ function Flow() {
     const ref = useRef<HTMLOutputElement>(null);
     const navigate = useNavigate();
     const blocker = useBlocker(({ currentLocation, nextLocation }) => {
-        navigated.current = nextLocation.pathname === currentLocation.pathname && nextLocation.search === currentLocation.search && nextLocation.hash === currentLocation.hash;
+        navigated.current = navigated.current || (nextLocation.pathname === currentLocation.pathname && nextLocation.search === currentLocation.search && nextLocation.hash === currentLocation.hash);
         return true;
     });
     const {pathname, search, hash} = useLocation();
@@ -200,7 +200,7 @@ function Flow() {
 
     const vaadinNavigateEventHandler = useCallback((event: CustomEvent<{state: unknown, url: string, replace?: boolean}>) => {
         const path = '/' + event.detail.url;
-        navigated.current = true;
+        navigated.current = !event.detail.replace;
         navigate(path, { state: event.detail.state, replace: event.detail.replace});
     }, [navigate]);
 

flow-tests/test-react-router/src/main/java/com/vaadin/flow/NavigationView.java

@@ -36,6 +36,7 @@ public class NavigationView extends Div {
 
     public static final String REACT_ANCHOR_ID = "anchor-react-navigation";
     public static final String REACT_ID = "react-navigation";
+    public static final String SET_PARAMETER_COUNTER_ID = "set-parameter-counter";
 
     public NavigationView() {
         Anchor anchorNavigation = new Anchor("com.vaadin.flow.AnchorView",

flow-tests/test-react-router/src/main/java/com/vaadin/flow/ServerView.java

@@ -19,10 +19,16 @@
 import com.vaadin.flow.component.html.Div;
 import com.vaadin.flow.component.html.NativeButton;
 import com.vaadin.flow.component.html.Span;
+import com.vaadin.flow.router.BeforeEvent;
+import com.vaadin.flow.router.HasUrlParameter;
 import com.vaadin.flow.router.Route;
+import com.vaadin.flow.router.WildcardParameter;
 
 @Route("com.vaadin.flow.ServerView")
-public class ServerView extends Div {
+public class ServerView extends Div implements HasUrlParameter<String> {
+
+    private final Span setParameterSpan = new Span();
+    private int setParameterCount = 0;
 
     public ServerView() {
         NativeButton serverNavigation = new NativeButton(
@@ -31,7 +37,15 @@ public ServerView() {
                             .navigate(NavigationView.class);
                 });
         serverNavigation.setId(NavigationView.SERVER_ID);
+        setParameterSpan.setId(NavigationView.SET_PARAMETER_COUNTER_ID);
+
+        add(new Span("ServerView"), new Div(), serverNavigation, new Div(),
+                setParameterSpan);
+    }
 
-        add(new Span("ServerView"), new Div(), serverNavigation);
+    @Override
+    public void setParameter(BeforeEvent event,
+            @WildcardParameter String parameter) {
+        setParameterSpan.setText("" + ++setParameterCount);
     }
 }

flow-tests/test-react-router/src/test/java/com/vaadin/flow/NavigationIT.java

@@ -425,4 +425,15 @@ public void testNavigatedForPostponeView() {
         checkNavigatedEvent(
                 "navigated to /view/com.vaadin.flow.NavigationView");
     }
+
+    @Test
+    public void testNavigation_HasUrlParameter_setParameterCalledOnce() {
+        open();
+
+        $(NativeButtonElement.class).id(NavigationView.SERVER_ID).click();
+        Assert.assertEquals("ServerView",
+                $(SpanElement.class).first().getText());
+        Assert.assertEquals("1", $(SpanElement.class)
+                .id(NavigationView.SET_PARAMETER_COUNTER_ID).getText());
+    }
 }

flow-tests/test-root-context/src/test/java/com/vaadin/flow/uitest/ui/HistoryIT.java

@@ -58,7 +58,7 @@ public void testHistory() throws URISyntaxException {
         Assert.assertEquals(baseUrl, getCurrentUrl());
         // idx value in history state is added by react-router
         Assert.assertEquals(
-                Arrays.asList("New location: asdf","New location: com.vaadin.flow.uitest.ui.HistoryView"),
+                Arrays.asList("New location: com.vaadin.flow.uitest.ui.HistoryView"),
                 getStatusMessages());
         clearButton.click();