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

Unified 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.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | includes/jobs/generic-footer.tmpl » ('j') | includes/jobs/generic-footer.tmpl » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: filters/find.py
===================================================================
new file mode 100644
--- /dev/null
+++ b/filters/find.py
@@ -0,0 +1,8 @@
+from jinja2 import environmentfilter, Environment
Wladimir Palant 2015/03/22 22:06:42 Nit: Environment isn't used. Not that it really ma
+
+@environmentfilter
+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.
+ for item in sequence:
+ if (item if attr is None else environment.getitem(item, attr)) == value:
+ return item
+ return None
« 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