Skip to content

Commit e2e6c24

Browse files
committed
Remove the complicated tag processing from ballot_icon, instead just
decorate the function with @simple_tag(takes_context=True) - Legacy-Id: 6970
1 parent 35520c8 commit e2e6c24

1 file changed

Lines changed: 3 additions & 18 deletions

File tree

ietf/doc/templatetags/ballot_icon.py

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,10 @@ def showballoticon(doc):
6262

6363
return True
6464

65+
@register.simple_tag(takes_context=True)
66+
def ballot_icon(context, doc):
67+
user = context.get("user")
6568

66-
def render_ballot_icon(doc, user):
6769
if not doc:
6870
return ""
6971

@@ -118,23 +120,6 @@ def sort_key(t):
118120

119121
return "".join(res)
120122

121-
class BallotIconNode(template.Node):
122-
def __init__(self, doc_var):
123-
self.doc_var = doc_var
124-
def render(self, context):
125-
doc = template.resolve_variable(self.doc_var, context)
126-
return render_ballot_icon(doc, context.get("user"))
127-
128-
def do_ballot_icon(parser, token):
129-
try:
130-
tag_name, doc_name = token.split_contents()
131-
except ValueError:
132-
raise template.TemplateSyntaxError, "%r tag requires exactly two arguments" % token.contents.split()[0]
133-
return BallotIconNode(doc_name)
134-
135-
register.tag('ballot_icon', do_ballot_icon)
136-
137-
138123
@register.filter
139124
def ballotposition(doc, user):
140125
if not showballoticon(doc) or not has_role(user, "Area Director"):

0 commit comments

Comments
 (0)