Skip to content

Commit

Permalink
[#1178] Use the data returned by validate, instead of the one passed …
Browse files Browse the repository at this point in the history
…as param
  • Loading branch information
vitorbaptista committed Oct 11, 2013
1 parent 0bd163e commit d073ce7
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions ckan/logic/action/create.py
Expand Up @@ -865,23 +865,23 @@ def user_invite(context, data_dict):
'group_id': [validators.not_empty],
'role': [validators.not_empty],
}
_, errors = _validate(data_dict, user_invite_schema, context)
data, errors = _validate(data_dict, user_invite_schema, context)
if errors:
raise ValidationError(errors)

while True:
try:
name = _get_random_username_from_email(data_dict['email'])
name = _get_random_username_from_email(data['email'])
password = str(random.SystemRandom().random())
data_dict['name'] = name
data_dict['password'] = password
data_dict['state'] = ckan.model.State.PENDING
user_dict = _get_action('user_create')(context, data_dict)
data['name'] = name
data['password'] = password
data['state'] = ckan.model.State.PENDING
user_dict = _get_action('user_create')(context, data)
user = ckan.model.User.get(user_dict['id'])
member_dict = {
'username': user.id,
'id': data_dict['group_id'],
'role': data_dict['role']
'id': data['group_id'],
'role': data['role']
}
_get_action('group_member_create')(context, member_dict)
mailer.send_invite(user)
Expand Down

0 comments on commit d073ce7

Please sign in to comment.