From 95abc4753d3a1773b019d121aa132f2f1a5e8b3e Mon Sep 17 00:00:00 2001
From: Anand Doshi <anand@erpnext.com>
Date: Fri, 24 Apr 2015 13:15:47 -0400
Subject: [PATCH] =?UTF-8?q?removed=20scrub=5Frelative=5Furls=E2=80=94now?=
 =?UTF-8?q?=20we=20can=20have=20relative=20urls!?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 frappe/templates/includes/breadcrumbs.html | 7 +++++--
 frappe/website/render.py                   | 2 +-
 frappe/website/template.py                 | 6 ++----
 frappe/website/utils.py                    | 6 ------
 4 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/frappe/templates/includes/breadcrumbs.html b/frappe/templates/includes/breadcrumbs.html
index 62f18dd8e8..5b7b5af365 100644
--- a/frappe/templates/includes/breadcrumbs.html
+++ b/frappe/templates/includes/breadcrumbs.html
@@ -1,7 +1,10 @@
 {% if parents and parents|length > 0 %}
 <ul class="breadcrumb">
-	<li><a href="{{ parents[-1].name }}"><span class="icon icon-angle-left"></span>
-		{{ parents[-1].page_title or parents[-1].title or "" }}</a></li>
+	<li>
+		<span class="icon icon-angle-left"></span>
+		<a href="{{ parents[-1].name | with_leading_slash }}">
+		{{ parents[-1].page_title or parents[-1].title or "" }}</a>
+	</li>
 	{#
 	<!-- {% for parent in parents %}
 		<li><a href="{{ parent.name }}">{{ parent.page_title or parent.title or "" }}</a></li>
diff --git a/frappe/website/render.py b/frappe/website/render.py
index afbd3f4c5e..580ad5508d 100644
--- a/frappe/website/render.py
+++ b/frappe/website/render.py
@@ -10,7 +10,7 @@ from werkzeug.wrappers import Response
 from werkzeug.routing import Map, Rule, NotFound
 
 from frappe.website.context import get_context
-from frappe.website.utils import scrub_relative_urls, get_home_page, can_cache, delete_page_cache
+from frappe.website.utils import get_home_page, can_cache, delete_page_cache
 from frappe.website.router import clear_sitemap
 
 class PageNotFoundError(Exception): pass
diff --git a/frappe/website/template.py b/frappe/website/template.py
index 8a0187b722..fffa226665 100644
--- a/frappe/website/template.py
+++ b/frappe/website/template.py
@@ -6,7 +6,6 @@ import frappe
 
 from frappe.utils import strip_html
 from frappe import _
-from frappe.website.utils import scrub_relative_urls
 from jinja2.utils import concat
 from jinja2 import meta
 import re
@@ -49,7 +48,7 @@ def render_blocks(template_path, out, context):
 	template = frappe.get_template(template_path)
 	for block, render in template.blocks.items():
 		new_context = template.new_context(context)
-		out[block] = scrub_relative_urls(concat(render(new_context)))
+		out[block] = concat(render(new_context))
 
 def separate_style_and_script(out, context):
 	"""Extract `style` and `script` tags into separate blocks"""
@@ -62,8 +61,7 @@ def set_breadcrumbs(out, context):
 
 	# breadcrumbs
 	if not out["no_breadcrumbs"] and "breadcrumbs" not in out:
-		out["breadcrumbs"] = scrub_relative_urls(
-			frappe.get_template("templates/includes/breadcrumbs.html").render(context))
+		out["breadcrumbs"] = frappe.get_template("templates/includes/breadcrumbs.html").render(context)
 
 def set_title_and_header(out, context):
 	"""Extract and set title and header from content or context."""
diff --git a/frappe/website/utils.py b/frappe/website/utils.py
index e9325174c5..fe20657f68 100644
--- a/frappe/website/utils.py
+++ b/frappe/website/utils.py
@@ -12,12 +12,6 @@ def delete_page_cache(path):
 	cache.delete_keys("page_context:" + path)
 	cache.delete_keys("sitemap_options:" + path)
 
-def scrub_relative_urls(html):
-	"""prepend a slash before a relative url"""
-	html = re.sub("""(src|href)[^\w'"]*['"](?!http|ftp|mailto|/|#|%|{)([^'" >]+)['"]""", '\g<1> = "/\g<2>"', html)
-	html = re.sub("""url\((?!(['"]?)(http|ftp|/|#|%|{))(['"]?)([^\(\)]+)\)""", 'url(\g<3>/\g<4>)', html)
-	return html
-
 def find_first_image(html):
 	m = re.finditer("""<img[^>]*src\s?=\s?['"]([^'"]*)['"]""", html)
 	try: