[v7]two bug of Sitemap file

iHello · August 23, 2016, 4:42am

Issue1:
A page have “no_sitemap = 1”,
but it is still listed in the sitemap.xml file.

frappe/frappe/blob/master/frappe/www/sitemap.py

# Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors
# MIT License. See license.txt

from __future__ import unicode_literals

from frappe.utils import get_request_site_address, get_datetime, nowdate
from frappe.website.router import get_pages, get_all_page_context_from_doctypes
from six import iteritems
from six.moves.urllib.parse import quote, urljoin

no_cache = 1
no_sitemap = 1
base_template_path = "templates/www/sitemap.xml"

def get_context(context):
	"""generate the sitemap XML"""
	host = get_request_site_address()
	links = []
	for route, page in iteritems(get_pages()):
		if not page.no_sitemap:

This file has been truncated. show original

issue2:
Because Variants have the same route as the template. so it should only include the template’s url in the sitemap.xml file.
But now it shows all the products of Variants as diffrent url.