Mercurial > piecrust2

changeset 231:137bcd498ef9

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
serve: Fix a bug where empty route metadata is not the same as invalid route. Some routes can capture no metadata at all, like for example when `foo/%path:slug%` is capturing URL `foo`. The route is still valid, so it returns an empty dictionary. It's not the same as returning `None`, which means "no match".
author Ludovic Chabant <ludovic@chabant.com>
date Wed, 11 Feb 2015 08:31:13 -0800
parents 016d42c23ba9
children e534d2bc657c
files piecrust/serving.py
diffstat 1 files changed, 1 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/piecrust/serving.py	Wed Feb 11 08:29:09 2015 -0800
+++ b/piecrust/serving.py	Wed Feb 11 08:31:13 2015 -0800
@@ -414,7 +414,7 @@
     res = []
     for route in routes:
         metadata = route.matchUri(uri)
-        if metadata:
+        if metadata is not None:
             res.append((route, metadata))
     return res