internal fixes
This commit is contained in:
@@ -271,17 +271,48 @@ class PlanChangeSerializer(serializers.Serializer):
|
||||
class LeadSerializer(serializers.ModelSerializer):
|
||||
route = serializers.PrimaryKeyRelatedField(queryset=Route.objects.all())
|
||||
moving_user = serializers.PrimaryKeyRelatedField(queryset=User.objects.all())
|
||||
moving_price = serializers.DecimalField(max_digits=10, decimal_places=2)
|
||||
|
||||
class Meta:
|
||||
model = Leads
|
||||
fields = ['route', 'moving_user', 'moving_price', 'moving_date', 'comment']
|
||||
|
||||
def validate_moving_date(self, value):
|
||||
if value < timezone.now().date():
|
||||
raise serializers.ValidationError("Дата перевозки не может быть в прошлом")
|
||||
return value
|
||||
try:
|
||||
if value < timezone.now().date():
|
||||
raise serializers.ValidationError("Дата перевозки не может быть в прошлом")
|
||||
return value
|
||||
except Exception as e:
|
||||
raise
|
||||
|
||||
def validate_moving_price(self, value):
|
||||
if value <= 0:
|
||||
raise serializers.ValidationError("Цена должна быть больше нуля")
|
||||
return value
|
||||
try:
|
||||
if isinstance(value, str):
|
||||
value = value.replace(',', '.')
|
||||
price = float(value)
|
||||
if price <= 0:
|
||||
raise serializers.ValidationError("Цена должна быть больше нуля")
|
||||
return price
|
||||
except (TypeError, ValueError) as e:
|
||||
raise serializers.ValidationError("Некорректный формат цены")
|
||||
|
||||
def validate(self, data):
|
||||
try:
|
||||
# проверяем что все обязательные поля присутствуют
|
||||
required_fields = ['route', 'moving_user', 'moving_price', 'moving_date']
|
||||
missing_fields = [field for field in required_fields if field not in data]
|
||||
if missing_fields:
|
||||
raise serializers.ValidationError(f"Отсутствуют обязательные поля: {', '.join(missing_fields)}")
|
||||
|
||||
# проверяем что пользователь не пытается откликнуться на свою заявку
|
||||
if data['route'].owner.id == data['moving_user'].id:
|
||||
raise serializers.ValidationError("Вы не можете откликнуться на собственную заявку")
|
||||
return data
|
||||
except Exception as e:
|
||||
raise
|
||||
|
||||
def to_internal_value(self, data):
|
||||
try:
|
||||
return super().to_internal_value(data)
|
||||
except Exception as e:
|
||||
raise
|
||||
@@ -198,10 +198,11 @@ class LeadViewSet(ViewSet):
|
||||
|
||||
@action(detail=False, methods=['post'])
|
||||
@handle_exceptions
|
||||
def send_lead(self, request, id):
|
||||
def send_lead(self, request):
|
||||
"""
|
||||
Создание новой заявки на перевозку
|
||||
"""
|
||||
|
||||
# добавляем текущего пользователя в данные
|
||||
data = request.data.copy()
|
||||
data['moving_user'] = request.user.id
|
||||
@@ -209,40 +210,54 @@ class LeadViewSet(ViewSet):
|
||||
# проверяем существование и доступность маршрута
|
||||
try:
|
||||
route = Route.objects.get(id=data.get('route'))
|
||||
if route.owner == request.user:
|
||||
if route.owner.id == request.user.id:
|
||||
print("Error: User trying to respond to their own route")
|
||||
return Response(
|
||||
{"error": "Вы не можете откликнуться на собственную заявку"},
|
||||
status=status.HTTP_400_BAD_REQUEST
|
||||
)
|
||||
except Route.DoesNotExist:
|
||||
print("Route not found for ID:", data.get('route'))
|
||||
return Response(
|
||||
{"error": "Указанный маршрут не найден"},
|
||||
status=status.HTTP_404_NOT_FOUND
|
||||
)
|
||||
|
||||
serializer = LeadSerializer(data=data)
|
||||
if serializer.is_valid():
|
||||
lead = serializer.save()
|
||||
|
||||
# собираем ответ с данными о заявке для фронта
|
||||
response_data = {
|
||||
"status": "success",
|
||||
"message": "Заявка успешно создана",
|
||||
"data": {
|
||||
"id": lead.id,
|
||||
"route_id": lead.route.id,
|
||||
"moving_price": lead.moving_price,
|
||||
"moving_date": lead.moving_date,
|
||||
is_valid = serializer.is_valid()
|
||||
|
||||
if is_valid:
|
||||
try:
|
||||
lead = serializer.save()
|
||||
|
||||
# собираем ответ с данными о заявке для фронта
|
||||
response_data = {
|
||||
"status": "success",
|
||||
"message": "Заявка успешно создана",
|
||||
"data": {
|
||||
"id": lead.id,
|
||||
"route_id": lead.route.id,
|
||||
"moving_price": lead.moving_price,
|
||||
"moving_date": lead.moving_date,
|
||||
}
|
||||
}
|
||||
}
|
||||
return Response(response_data, status=status.HTTP_201_CREATED)
|
||||
|
||||
return Response(
|
||||
{
|
||||
"status": "error",
|
||||
"message": "Ошибка валидации данных",
|
||||
"errors": serializer.errors
|
||||
},
|
||||
status=status.HTTP_400_BAD_REQUEST
|
||||
)
|
||||
return Response(response_data, status=status.HTTP_201_CREATED)
|
||||
except Exception as e:
|
||||
return Response(
|
||||
{
|
||||
"status": "error",
|
||||
"message": "Ошибка при сохранении заявки",
|
||||
"error": str(e)
|
||||
},
|
||||
status=status.HTTP_500_INTERNAL_SERVER_ERROR
|
||||
)
|
||||
else:
|
||||
return Response(
|
||||
{
|
||||
"status": "error",
|
||||
"message": "Ошибка валидации данных",
|
||||
"errors": serializer.errors
|
||||
},
|
||||
status=status.HTTP_400_BAD_REQUEST
|
||||
)
|
||||
|
||||
Reference in New Issue
Block a user