Moved common check to function

Author: mrsaicharan1

app/api/tickets.py

@@ -21,10 +21,12 @@
 from app.api.helpers.db import get_count
 
 
-def validate_ticket_price(ticket_type, price):
-    if ticket_type != 'free' and int(price) <= 0:
+def validate_ticket_price(data):
+    if not data.get('price') and data.get('type'):
+        raise UnprocessableEntity({}, "Type/price of ticket is missing")
+    if data.get('type') != 'free' and int(data.get('price')) <= 0:
         raise UnprocessableEntity(
-            {'price': price}, "Price of a paid/donation ticket must be greater than zero")
+            {'price': data.get('price')}, "Price of a paid/donation ticket must be greater than zero")
 
 
 class TicketListPost(ResourceList):
@@ -66,7 +68,7 @@ def before_create_object(self, data, view_kwargs):
             if not event.is_payment_enabled():
                 raise UnprocessableEntity(
                     {'event_id': data['event']}, "Event having paid ticket must have a payment method")
-        validate_ticket_price(data.get('type'), data.get('price'))
+        validate_ticket_price(data)
 
     schema = TicketSchema
     methods = ['POST', ]
@@ -189,8 +191,7 @@ def before_update_object(self, ticket, data, view_kwargs):
             if not event.is_payment_enabled():
                 raise UnprocessableEntity(
                     {'event_id': ticket.event.id}, "Event having paid ticket must have a payment method")
-        if data.get('price') and data.get('type'):
-            validate_ticket_price(data.get('type'), data.get('price'))
+        validate_ticket_price(data)
 
     decorators = (api.has_permission('is_coorganizer', fetch='event_id',
                   fetch_as="event_id", model=Ticket, methods="PATCH,DELETE"),)