refactor: Rename validate function to can_render

frappe/website/page_controllers/document_page.py

@@ -7,7 +7,7 @@ from frappe.website.router import (get_doctypes_with_web_view,
 
 
 class DocumentPage(BaseTemplatePage):
-	def validate(self):
+	def can_render(self):
 		'''
 		Find a document with matching `route` from all doctypes with `has_web_view`=1
 		'''

frappe/website/page_controllers/error_page.py

@@ -6,5 +6,5 @@ class ErrorPage(TemplatePage):
 		super().__init__(path=path, http_status_code=http_status_code)
 		self.http_status_code = getattr(exception, 'http_status_code', None) or http_status_code or 500
 
-	def validate(self):
+	def can_render(self):
 		return True

frappe/website/page_controllers/list_page.py

@@ -2,7 +2,7 @@ import frappe
 from frappe.website.page_controllers.template_page import TemplatePage
 
 class ListPage(TemplatePage):
-	def validate(self):
+	def can_render(self):
 		return frappe.db.exists('DocType', self.path, True)
 
 	def render(self):

frappe/website/page_controllers/not_found_page.py

@@ -15,7 +15,7 @@ class NotFoundPage(TemplatePage):
 		http_status_code = 404
 		super().__init__(path=path, http_status_code=http_status_code)
 
-	def validate(self):
+	def can_render(self):
 		return True
 
 	def render(self):

frappe/website/page_controllers/not_permitted_page.py

@@ -9,7 +9,7 @@ class NotPermittedPage(TemplatePage):
 		super().__init__(path=path, http_status_code=http_status_code)
 		self.http_status_code = 403
 
-	def validate(self):
+	def can_render(self):
 		return True
 
 	def render(self):

frappe/website/page_controllers/print_page.py

@@ -6,7 +6,7 @@ class PrintPage(TemplatePage):
 	default path returns a printable object (based on permission)
 	/Quotation/Q-0001
 	'''
-	def validate(self):
+	def can_render(self):
 		parts = self.path.split('/', 1)
 		if len(parts)==2:
 			if (frappe.db.exists('DocType', parts[0], True)

frappe/website/page_controllers/redirect_page.py

@@ -6,7 +6,7 @@ class RedirectPage(object):
 		self.path = path
 		self.http_status_code = http_status_code
 
-	def validate(self):
+	def can_render(self):
 		return True
 
 	def render(self):

frappe/website/page_controllers/static_page.py

@@ -23,7 +23,7 @@ class StaticPage(WebPage):
 			if os.path.isfile(file_path):
 				self.file_path = file_path
 
-	def validate(self):
+	def can_render(self):
 		return self.is_valid_file_path() and self.file_path
 
 	def is_valid_file_path(self):

frappe/website/page_controllers/template_page.py

@@ -36,7 +36,7 @@ class TemplatePage(BaseTemplatePage):
 						self.basepath = os.path.dirname(file_path)
 						return
 
-	def validate(self):
+	def can_render(self):
 		return hasattr(self, 'template_path') and bool(self.template_path)
 
 	def get_index_path_options(self, search_path):

frappe/website/page_controllers/web_form.py

@@ -2,5 +2,5 @@ from frappe.website.page_controllers.web_page import WebPage
 import frappe
 
 class WebFormPage(WebPage):
-	def validate(self):
+	def can_render(self):
 		return bool(frappe.get_all("Web Form", filters={'route': self.path}))

frappe/website/page_controllers/web_page.py

@@ -13,7 +13,7 @@ class WebPage(object):
 		if self.validate():
 			return self.render()
 
-	def validate(self):
+	def can_render(self):
 		pass
 
 	def render(self):

frappe/website/path_resolver.py

@@ -32,15 +32,14 @@ class PathResolver():
 		try:
 			resolve_redirect(self.path, request.query_string)
 		except frappe.Redirect:
-			return self.path, RedirectPage(self.path)
+			return frappe.flags.redirect_location, RedirectPage(self.path)
 
 		endpoint = resolve_path(self.path)
 		renderers = (StaticPage, WebFormPage, TemplatePage, ListPage, DocumentPage, PrintPage, NotFoundPage)
 
 		for renderer in renderers:
 			renderer_instance = renderer(endpoint, 200)
-			can_render = renderer_instance.validate()
-			if can_render:
+			if renderer_instance.can_render():
 				return endpoint, renderer_instance
 
 		return endpoint, NotFoundPage(endpoint)