From 1605ff02c60e0aa328e0d6b168fe53fd0bf351f1 Mon Sep 17 00:00:00 2001
From: Kanchan Chauhan <angel143.dr@gmail.com>
Date: Mon, 4 Jul 2016 14:28:09 +0530
Subject: [PATCH] [WIP] Address view fixed

---
 erpnext/hooks.py                             |  7 +++++++
 erpnext/templates/includes/address_row.html  | 14 ++++++++++----
 erpnext/utilities/doctype/address/address.py | 14 +++++++++++++-
 3 files changed, 30 insertions(+), 5 deletions(-)

diff --git a/erpnext/hooks.py b/erpnext/hooks.py
index d20e20b36f..15f01ca0e6 100644
--- a/erpnext/hooks.py
+++ b/erpnext/hooks.py
@@ -81,6 +81,13 @@ website_route_rules = [
 			"parents": [{"title": _("Request for Quotation"), "name": "rfq"}]
 		}
 	},
+	{"from_route": "/addresses", "to_route": "Address"},
+	{"from_route": "/addresses/<path:name>", "to_route": "addresses",
+		"defaults": {
+			"doctype": "Address",
+			"parents": [{"title": _("Addresses"), "name": "addresses"}]
+		}
+	},
 	{"from_route": "/jobs", "to_route": "Job Opening"},
 ]
 
diff --git a/erpnext/templates/includes/address_row.html b/erpnext/templates/includes/address_row.html
index 15dcb95196..2c5056a2ba 100644
--- a/erpnext/templates/includes/address_row.html
+++ b/erpnext/templates/includes/address_row.html
@@ -1,8 +1,14 @@
 <div class="web-list-item">
     <a href="/addresses?name={{ doc.name | urlencode }}" class="no-decoration">
-        <h4 class="strong">{{ doc.address_title }}</h4>
-        <p class="text-muted small">
-            {{ frappe.get_doc(doc).get_display() }}
-        </p>
+	    <div class="row">
+	        <div class="col-xs-3">
+	                 <span class="indicator {{ "red" if doc.address_type=="Office" else "green" if doc.address_type=="Billing" else "blue" if doc.address_type=="Shipping" else "darkgrey" }}">{{ doc.address_title }}</span>
+			</div>
+			<div class="col-xs-2"> {{ doc.address_type }} </div>
+			<div class="col-xs-2"> {{ doc.city }} </div>
+			<div class="col-xs-5 text-right small text-muted">
+	            {{ frappe.get_doc(doc).get_display() }}
+	        </div>
+	    </div>
     </a>
 </div>
diff --git a/erpnext/utilities/doctype/address/address.py b/erpnext/utilities/doctype/address/address.py
index f613faa67d..1d215cc0e7 100644
--- a/erpnext/utilities/doctype/address/address.py
+++ b/erpnext/utilities/doctype/address/address.py
@@ -9,6 +9,7 @@ from frappe.utils import cstr
 
 from frappe.model.document import Document
 from jinja2 import TemplateSyntaxError
+from frappe.utils.user import is_website_user
 
 class Address(Document):
 	def __setup__(self):
@@ -123,11 +124,22 @@ def get_list_context(context=None):
 	from erpnext.shopping_cart.cart import get_address_docs
 	return {
 		"title": _("Addresses"),
-		"get_list": get_address_docs,
+		"get_list": get_address_list,
 		"row_template": "templates/includes/address_row.html",
 		'no_breadcrumbs': True,
 	}
+	
+def get_address_list(doctype, txt, filters, limit_start, limit_page_length=20):
+	from frappe.www.list import get_list
+	user = frappe.session.user
+	ignore_permissions = False
+	if is_website_user():
+		if not filters: filters = []
+		filters.append(("Address", "owner", "=", user))
+		ignore_permissions = True
 
+	return get_list(doctype, txt, filters, limit_start, limit_page_length, ignore_permissions=ignore_permissions)
+	
 def has_website_permission(doc, ptype, user, verbose=False):
 	"""Returns true if customer or lead matches with user"""
 	customer = frappe.db.get_value("Contact", {"email_id": frappe.session.user}, "customer")