diff --git a/src/Umbraco.Web/Routing/DefaultLastChanceLookup.cs b/src/Umbraco.Web/Routing/DefaultLastChanceLookup.cs index 0b8d967530..b20da16226 100644 --- a/src/Umbraco.Web/Routing/DefaultLastChanceLookup.cs +++ b/src/Umbraco.Web/Routing/DefaultLastChanceLookup.cs @@ -48,12 +48,6 @@ namespace Umbraco.Web.Routing docRequest.RoutingContext.UmbracoContext, handler.redirectID); - //SD: We are setting the Is404 to true here because these are 404 handlers. - // if people in the future add their own last chance lookup resolver, they might not want things to be 404s - // and instead do their own thing so we should leave it up to the last chance resolver to set the 404, not the - // module. - docRequest.Is404 = true; - // FIXME - could it be null? LogHelper.Debug("Handler '{0}' found node with id={1}.", () => handler.GetType().FullName, () => handler.redirectID); @@ -76,6 +70,12 @@ namespace Umbraco.Web.Routing } } + //SD: We are setting the Is404 to true here because these are 404 handlers. + // if people in the future add their own last chance lookup resolver, they might not want things to be 404s + // and instead do their own thing so we should leave it up to the last chance resolver to set the 404, not the + // module. + docRequest.Is404 = true; + return currentPage; }