From 4c6306831a3e3d2a78e78e68389f6b588d272306 Mon Sep 17 00:00:00 2001 From: Akshar Raaj Date: Tue, 23 Sep 2014 13:42:22 +0530 Subject: [PATCH] Cleanup * Try/except should be used in minimal code block so it doesn't accidentally catch something we don't want it to catch. * Removed print statement --- tavern/templatetags/tavern_unjoined_filters.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tavern/templatetags/tavern_unjoined_filters.py b/tavern/templatetags/tavern_unjoined_filters.py index 57ee287..0afbec0 100644 --- a/tavern/templatetags/tavern_unjoined_filters.py +++ b/tavern/templatetags/tavern_unjoined_filters.py @@ -24,10 +24,7 @@ def get_user_tavern_unjoined_groups(parser, token): """gets all unjoined groups for a user""" try: tag_name, for_keyword, user, as_keyword, var_name = token.split_contents() - user = token.split_contents()[2] - print user - if not (user in ('request.user', 'user')): - raise template.TemplateSyntaxError("%r variable is not the user" % token.contents.split()[2]) except ValueError: raise template.TemplateSyntaxError("%r tag is invalid. use 'get_user_tavern_unjoined_groups for request.user/user in user_tavern_unjoined_groups'" % token.contents.split()[0]) + user = token.split_contents()[2] return UserUnjoinedTavernGroup(user, var_name)