Produits Entr'ouvert » Chrono

Le code d'origine ne permettait que de modifier le champ "user_was_present", je suppose que c'est ce qui a donnée ces contraintes¹. Quand on a ajouté l'édition des autres champs² les contraintes n'ont pas été revues, je suppose que c'est l'occasion de le faire.

Sur un GET je ne vois pas trop pourquoi appliquer des contraintes. PS: j'ai gardé celle sur les bookings secondaires, celle-ci reste nécessaire.

Sur un DELETE on peut conserver les contraintes actuelles.

Pour le PATCH je dirai bien de maintenir les conditions actuelles si user_was_present, user_absence_reason ou user_presence_reason sont modifiés, pour les autres cas (user_first_name, user_last_name, user_email, user_phone_number, use_color_for, label et pour get_extra_data) on peut retirer le cas in_waiting_list.

¹

class BookingAPI(APIView):
...
    def check_booking(self, check_waiting_list=False):
        if self.booking.cancellation_datetime:
            raise APIError(N_('booking is cancelled'))

        if self.booking.primary_booking is not None:
            raise APIError(N_('secondary booking'), err=2)

        if check_waiting_list and self.booking.in_waiting_list:
            raise APIError(N_('booking is in waiting list'), err=3)
...
   def patch(self, request, *args, **kwargs):
        self.check_booking(check_waiting_list=True)

²

class BookingSerializer(serializers.ModelSerializer):
    user_was_present = serializers.BooleanField(required=False, allow_null=True)
    user_absence_reason = serializers.CharField(required=False, allow_blank=True, allow_null=True)
    user_presence_reason = serializers.CharField(required=False, allow_blank=True, allow_null=True)
    use_color_for = serializers.CharField(required=False, allow_blank=True, allow_null=True, source='color')

    class Meta:
        model = Booking
        fields = [
            'id',
            'in_waiting_list',
            'user_first_name',
            'user_last_name',
            'user_email',
            'user_phone_number',
            'user_was_present',
            'user_absence_reason',
            'user_presence_reason',
            'use_color_for',
            'extra_data',
            'creation_datetime',
            'cancellation_datetime',
            'label',
        ]
        read_only_fields = [
            'id',
            'in_waiting_list',
            'extra_data',
            'creation_datetime',
            'cancellation_datetime',

Automatic expiration