Skip to content

Commit bc705e7

Browse files
committed
Fixed nitpick comment
1 parent e1eb469 commit bc705e7

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

src/Framework/Framework/Routing/DotvvmRouteTable.cs

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -312,18 +312,19 @@ IEnumerator IEnumerable.GetEnumerator()
312312
[return: NotNullIfNotNull(nameof(appendedPath))]
313313
private string? CombinePath(string? prefix, string? appendedPath)
314314
{
315-
if (string.IsNullOrEmpty(prefix))
315+
if (appendedPath == null)
316316
{
317-
return appendedPath;
317+
// NB: if you are adding presenter route inside a group, we ignore the group's virtual path because presenter routes have VirtualPath = null
318+
return null;
318319
}
319320

320-
if (appendedPath == null)
321+
if (string.IsNullOrEmpty(prefix))
321322
{
322-
return null;
323+
return appendedPath;
323324
}
324-
else if (appendedPath == string.Empty)
325+
else if (string.IsNullOrEmpty(appendedPath))
325326
{
326-
return prefix ?? string.Empty;
327+
return prefix;
327328
}
328329

329330
return $"{prefix}/{appendedPath}";

0 commit comments

Comments
 (0)