Fix validation of template for service.

When the delivery app creates the sms on behalf of the service,
the validation was failing.
This commit is contained in:
Rebecca Law
2016-02-15 15:02:19 +00:00
parent e7ef5945e5
commit fcaed04918

View File

@@ -122,7 +122,9 @@ class SmsTemplateNotificationSchema(SmsNotificationSchema):
raise ValidationError('Invalid phone number for restricted service', 'restricted')
# Assert the template is valid for the service which made the request.
service = api_user['client']
if (service != current_app.config.get('ADMIN_CLIENT_USER_NAME') and
admin_users = [current_app.config.get('ADMIN_CLIENT_USER_NAME'),
current_app.config.get('DELIVERY_CLIENT_USER_NAME')]
if (service not in admin_users and
template.service != models.Service.query.filter_by(id=service).first()):
raise ValidationError('Invalid template', 'restricted')