Issue 2559 - [cms] Detect conflicting pages in the development server

cms/bin/test_server.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2015 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
@@ skipping 32 matching lines @@
       <p>An error occurred while processing the request for {{uri}}:</p>
       <pre>{{error}}</pre>
     {% endif %}
   </body>
 </html>"""
 
 # Create our own instance, the default one will introduce "random" host-specific
 # behavior by parsing local config files.
 mime_types = mimetypes.MimeTypes()
 
-def get_data(path):
-  if source.has_static(path):
-    return source.read_static(path)
-
+def get_page(path):
   path = path.strip("/")
   if path == "":
     path = source.read_config().get("general", "defaultlocale")
   if "/" in path:
     locale, page = path.split("/", 1)
   else:
     locale, page = path, ""
 
   default_page = source.read_config().get("general", "defaultpage")
   alternative_page = "/".join([page, default_page]).lstrip("/")
+
   for format in converters.iterkeys():
     for p in (page, alternative_page):
       if source.has_page(p, format):
-        return process_page(source, locale, p, format, "http://127.0.0.1:5000")
+        return (p, process_page(source, locale, p, format, "http://127.0.0.1:5000"))
   if source.has_localizable_file(locale, page):
-    return source.read_localizable_file(locale, page)
+    return (page, source.read_localizable_file(locale, page))
 
-  return None
+  return (None, None)
+
+def has_conflicting_pages(page):
+  pages = [p for p, _ in source.list_pages()]
+  pages.extend(source.list_localizable_files())
+
+  if pages.count(page) > 1:
+    return True
+  if any(p.startswith(page + "/") or page.startswith(p + "/") for p in pages):
+    return True
+  return False
+
+def get_data(path):
+  if source.has_static(path):
+    return source.read_static(path)
+
+  page, data = get_page(path)
+  if page and has_conflicting_pages(page):
+    raise Exception("The requested page conflicts with another page")
+  return data
 
 def show_error(start_response, status, **kwargs):
   env = jinja2.Environment(autoescape=True)
   template = env.from_string(ERROR_TEMPLATE)
   mime = "text/html; encoding=%s" % UNICODE_ENCODING
   start_response(status, [("Content-Type", mime)])
   for fragment in template.stream(status=status, **kwargs):
     yield fragment.encode(UNICODE_ENCODING)
 
 def handler(environ, start_response):
   path = environ.get("PATH_INFO")
 
   data = get_data(path)
-  if data == None:
+  if data is None:
     return show_error(start_response, "404 Not Found", uri=path)
 
   mime = mime_types.guess_type(path)[0] or "text/html"
 
   if isinstance(data, unicode):
     data = data.encode(UNICODE_ENCODING)
     mime = "%s; charset=%s" % (mime, UNICODE_ENCODING)
 
   start_response("200 OK", [("Content-Type", mime)])
   return [data]
@@ skipping 26 matching lines @@
           return app(environ, start_response)
         except Exception, e:
           return show_error(start_response, "500 Internal Server Error",
               uri=environ.get("PATH_INFO"), error=e)
 
       server = make_server(host, port, wrapper)
       print " * Running on http://%s:%i/" % server.server_address
       server.serve_forever()
 
   run("localhost", 5000, handler, use_reloader=True, use_debugger=True)