Re: [Django] #4746: [patch] Allow whitespace before and after filter separator

[Django]

#4746: [patch] Allow whitespace before and after filter separator
-+-
 Reporter:  sciyoshi@…   |Owner:  aaugustin
 Type:   |   Status:  closed
  Cleanup/optimization   |  Version:  master
Component:  Template system  |   Resolution:  fixed
 Severity:  Normal   | Triage Stage:  Ready for
 Keywords:  template filter  |  checkin
  separator whitespace tplrf-fixed   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 Fixed in ddfc7c2530197112fb105b6dd237574bc2fd6b49.

-- 
Ticket URL: 
Django 
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.

Re: [Django] #4746: [patch] Allow whitespace before and after filter separator

[Django]

#4746: [patch] Allow whitespace before and after filter separator
-+-
 Reporter:  sciyoshi@…   |Owner:  aaugustin
 Type:   |   Status:  new
  Cleanup/optimization   |  Version:  master
Component:  Template system  |   Resolution:
 Severity:  Normal   | Triage Stage:  Ready for
 Keywords:  template filter  |  checkin
  separator whitespace tplrf-fixed   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * needs_tests:  1 => 0
 * stage:  Accepted => Ready for checkin


Comment:

 This is really a very simple change and it looks reasonable to me.

 The current behavior (raising !TemplateSyntaxError for spaces around the
 `|`) was tested. AFAICT this test exists mostly for completeness and it
 reasonable to change it if we make the filter syntax less strict.

 This may be a sensitive design change. I'll leave the patch in RFC for a
 few days so that other core devs can get a chance to review or veto it.

-- 
Ticket URL: 
Django 
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.

Re: [Django] #4746: [patch] Allow whitespace before and after filter separator

[Django]

#4746: [patch] Allow whitespace before and after filter separator
-+-
 Reporter:  sciyoshi@…   |Owner:  aaugustin
 Type:   |   Status:  new
  Cleanup/optimization   |  Version:  SVN
Component:  Template system  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  template filter  |  Needs documentation:  0
  separator whitespace tplrf-fixed   |  Patch needs improvement:  0
Has patch:  1|UI/UX:  0
  Needs tests:  1|
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * owner:  nobody => aaugustin
 * ui_ux:   => 0
 * easy:   => 0


-- 
Ticket URL: 
Django 
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.