Closed CrispinF closed 2 years ago
This is also true for the link from site logo image.
Partial views: CmsSiteLogoPartial CmsSiteTitlePartial both use var rootUrl = Url.RouteUrl(pageRoutes.PageRouteName, new { siteFolder = Model.SiteFolderName, slug = "" });
...which takes you well into the weeds of the simplecontent routing definitions
routes.MapControllerRoute(
name: ProjectConstants.CultureFolderPageIndexRouteName,
pattern: "{sitefolder}/{culture}/{slug=none}"
, defaults: new { controller = "Page", action = "Index" }
, constraints: new { sitefolder = siteFolderConstraint, culture = cultureConstraint }
);
Maybe not a quick fix (needs to ensure correct behaviour in various multi-tenant scenarios, so you can't just link to "/")
Actually this might be a relatively simple template issue:
if _simpleContentOption = "b";
then the _Layout views don't want to be doing this:
@*#if (SimpleContentConfig == 'a' || SimpleContentConfig == 'b')*@
<partial name="CmsSiteLogoPartial" model="@Tenant" />
@*#else*@
<partial name="SiteLogoPartial" model="@Tenant" />
@*#endif*@
Since only the 'cms' version links back to the 'pages' route.
But there may be a larger bug in the template: I don't think the custom manually input route segment and title to link to the pages are working at all... it always goes to /p
Sorted in v6.3 template
With "Pages and Blog with Home Controller as default route" the site title hyperlink goes to the "Pages" root instead of the site default page. It should link to the site default page. The link is made by this:
<partial name="CmsSiteTitlePartial" model="@Tenant" />
I'm not sure where the mistake is, so logging this against the VS template for now.