url template tag gives no error if keyword arguments contain spaces

[scott@…]

I was stuck for a while using the url template tag with keyword arguments. The problem was I had a space after the comma delimiting the arguments, something like:

{% url test.views.check_location country=address.country, city=address.city %} # bad
{% url test.views.check_location country=address.country,city=address.city %} # good

The space caused MatchChecker to look for a keyword parameter named ' city' (i.e. with leading space). The pattern didn't match, so I got nothing. I think an exception might be more helpful, so here's a patch in case you agree. :)

[scott@…]

Patch to defaulttags and test

[Chris Beaven]

Thanks Scott.

Personally I'd prefer it if the arguments weren't comma separated but rather space separated (like most other tags we have), but that's a different issue altogether ;)

[Malcolm Tredinnick]

(In [5077]) Fixed #3543 -- Be a bit more tolerant of spaces following the comma
argument separator in the "url" tag.