dotnet/aspnetcore

NavigationManager.Refresh does not make use of optional parameter 'forceReload'

Open

#59,854 建立於 2025年1月13日

在 GitHub 查看
 (13 留言) (0 反應) (0 負責人)C# (37,933 star) (10,653 fork)batch import
area-blazorbughelp wanted

描述

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

Hey!

I wanted to use NavigationManager.Refresh(forceReload: false) on our .NET8 Blazor Server app and noticed that forceReload is not being passed through to NavigateTo, instead the forceReload of NavigateTo defaults to true. You can easily see what I mean in this file in line 178: https://github.com/dotnet/aspnetcore/blob/8d6d208542cbfa4553ae59436cc90d84aac08c6a/src/Components/Components/src/NavigationManager.cs#L178

Expected Behavior

I assume this is not intentional as I cannot come up with a reason why it would be. If the parameter exists one would expect it to be used, right?

Steps To Reproduce

The code is obvious, but NavigationManager.Refresh(forceReload: false) reproduces the issue.

Exceptions (if any)

None

.NET Version

8.0.x

Anything else?

I think the issue speaks for itsself.

貢獻者指南

技術棧
csharp
領域
backend
議題類型
bug
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
under 1 hour
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
active
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
無
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
95
研究方向
The bug is in the NavigationManager.Refresh method in the file src/Components/Components/src/NavigationManager.cs at line 178. The method currently calls NavigateTo with a fixed second parameter 'true' instead of passing the 'forceReload' parameter. A first time contributor can fix this by replacing the hardcoded 'true' with the 'forceReload' variable. No additional changes are required.
NavigationManager.Refresh does not make use of optional parameter 'forceReload' · dotnet/aspnetcore#59854 | Good First Issue