V9: Visiting a page without a template returns "Page Not Found" (#11852)

* Returns "Page Not Found" when template doesn't exist for a docType

* Fix comment

* Use IsNullOrWhiteSpace instead of IsNullOrEmpty

Co-authored-by: Mole <[email protected]>

Author: elit0451

src/Umbraco.Web.Common/Controllers/RenderController.cs

@@ -53,6 +53,26 @@ protected IUmbracoContext UmbracoContext
         /// </summary>
         public virtual IActionResult Index() => CurrentTemplate(new ContentModel(CurrentPage));
 
+        /// <summary>
+        /// Gets an action result based on the template name found in the route values and a model.
+        /// </summary>
+        /// <typeparam name="T">The type of the model.</typeparam>
+        /// <param name="model">The model.</param>
+        /// <returns>The action result.</returns>
+        /// <remarks>
+        /// If the template found in the route values doesn't physically exist, Umbraco not found result is returned.
+        /// </remarks>
+        protected override IActionResult CurrentTemplate<T>(T model)
+        {
+            if (EnsurePhsyicalViewExists(UmbracoRouteValues.TemplateName) == false)
+            {
+                // no physical template file was found
+                return new PublishedContentNotFoundResult(UmbracoContext);
+            }
+
+            return View(UmbracoRouteValues.TemplateName, model);
+        }
+
         /// <summary>
         /// Before the controller executes we will handle redirects and not founds
         /// </summary>
@@ -123,6 +143,6 @@ private PublishedContentNotFoundResult GetNoTemplateResult(IPublishedRequest pcr
             {
                 return new PublishedContentNotFoundResult(UmbracoContext);
             }
-        }        
+        }
     }
 }

src/Umbraco.Web.Common/Controllers/UmbracoPageController.cs

@@ -74,7 +74,7 @@ protected virtual IPublishedContent CurrentPage
         /// <param name="model">The model.</param>
         /// <returns>The action result.</returns>
         /// <exception cref="InvalidOperationException">If the template found in the route values doesn't physically exist and exception is thrown</exception>
-        protected IActionResult CurrentTemplate<T>(T model)
+        protected virtual IActionResult CurrentTemplate<T>(T model)
         {
             if (EnsurePhsyicalViewExists(UmbracoRouteValues.TemplateName) == false)
             {
@@ -90,6 +90,13 @@ protected IActionResult CurrentTemplate<T>(T model)
         /// <param name="template">The view name.</param>
         protected bool EnsurePhsyicalViewExists(string template)
         {
+            if (string.IsNullOrWhiteSpace(template))
+            {
+                string docTypeAlias = UmbracoRouteValues.PublishedRequest.PublishedContent.ContentType.Alias;
+                _logger.LogWarning("No physical template file was found for document type with alias {Alias}", docTypeAlias);
+                return false;
+            }
+
             ViewEngineResult result = _compositeViewEngine.FindView(ControllerContext, template, false);
             if (result.View != null)
             {
@@ -99,6 +106,5 @@ protected bool EnsurePhsyicalViewExists(string template)
             _logger.LogWarning("No physical template file was found for template {Template}", template);
             return false;
         }
-
     }
 }