Skip to content

Commit

Permalink
Ignore deleted accounts when editing flair.
Browse files Browse the repository at this point in the history
  • Loading branch information
Logan Hanks committed Oct 28, 2011
1 parent 20afca6 commit c911b30
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions r2/r2/controllers/api.py
Expand Up @@ -1896,7 +1896,7 @@ def POST_editaward(self, form, jquery, award, colliding_award, codename,

@validatedForm(VFlairManager(),
VModhash(),
user = VExistingUname("name", allow_deleted=True),
user = VExistingUname("name"),
text = VFlairText("text"),
css_class = VFlairCss("css_class"))
def POST_flair(self, form, jquery, user, text, css_class):
Expand Down Expand Up @@ -1940,8 +1940,8 @@ def POST_flair(self, form, jquery, user, text, css_class):
form.set_html('.status', _('saved'))

@validatedForm(VFlairManager(),
VModhash(),
user = VExistingUname("name", allow_deleted=True))
VModhash(),
user = VExistingUname("name"))
def POST_deleteflair(self, form, jquery, user):
# Check validation.
if form.has_errors('name', errors.USER_DOESNT_EXIST, errors.NO_USER):
Expand Down Expand Up @@ -1978,7 +1978,7 @@ def POST_flaircsv(self, flair_csv):
line_result.error('row', 'improperly formatted row, ignoring')
continue

user = VExistingUname('name', allow_deleted=True).run(name)
user = VExistingUname('name').run(name)
if not user:
line_result.error('user',
"unable to resolve user `%s', ignoring"
Expand Down

0 comments on commit c911b30

Please sign in to comment.