Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: filters/find.py

Issue 5994668214976512: Issue 662 - Job links shouldn't use anchors (Closed)
Patch Set: Created March 21, 2015, 5:21 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | includes/jobs/generic-footer.tmpl » ('j') | includes/jobs/generic-footer.tmpl » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 from jinja2 import environmentfilter, Environment
Wladimir Palant 2015/03/22 22:06:42 Nit: Environment isn't used. Not that it really ma
2
3 @environmentfilter
4 def find(environment, sequence, value, attr=None):
Wladimir Palant 2015/03/22 22:06:42 Please keep attribute as the parameter name here,
saroyanm 2015/03/22 23:40:51 Done.
5 for item in sequence:
6 if (item if attr is None else environment.getitem(item, attr)) == value:
7 return item
8 return None
OLDNEW
« no previous file with comments | « no previous file | includes/jobs/generic-footer.tmpl » ('j') | includes/jobs/generic-footer.tmpl » ('J')

Powered by Google App Engine
This is Rietveld