#16677: templates tests fail when the path to django contains a space
--------------------------------------+---------------------------------
               Reporter:  aaugustin   |          Owner:  nobody
                   Type:  Bug         |         Status:  new
              Milestone:              |      Component:  Template system
                Version:  1.3         |       Severity:  Normal
             Resolution:              |       Keywords:
           Triage Stage:  Unreviewed  |      Has patch:  0
    Needs documentation:  0           |    Needs tests:  0
Patch needs improvement:  0           |  Easy pickings:  0
                  UI/UX:  0           |
--------------------------------------+---------------------------------

Comment (by aaugustin):

 The `ssi` tag is defined twice, in `django/template/defaulttags.py` and in
 `django/templatetags/future.py`. I recommend:
 - not changing the behavior of the "old" version, and just disable its
 tests when the path contains a space; all this code will be removed in
 Django 1.5 anyway.
 - fixing the "new" version by using `token.split_contents()` (quotes-aware
 spitting) instead of `token.contents.split()` (naïve splitting) to parse
 the content of the tag.

 I think the second point is a real bug and I've added a test case for it —
 a template whose file name contains spaces.

 Attached patch implements this.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/16677#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to