Marta Miranda / boletin_api

Added 1 commit:

a467e570 - correcion de return en modelo edicto

Acá puedes calcular sólo una vez today

today = timezone.now().date()

y luego utilizarlo en la query de precios_activos y después en la condición obj.bigencia_desde <= today

Donde haz definido UsuarioNoAutorizado? esa clase existe? Por favor utiliza en primera instancia las excepciones ya creadas en el framework: https://www.django-rest-framework.org/api-guide/exceptions/

Por favor utiliza: PermissionDenied(detail="Aquí el detalle que quieras")

Idem al comentario anterior. Qué IDE utilizas? Por que el editor debería indicarte que esos nombres (EdictoOPrecioNoEncontrado, UsuarioNoAutorizado) no existen. Por favor si no utilizas PyCharm, te recomiendo que lo hagas ... Creo que la mayoría de los chicos en la oficina lo utiliza para los proyecto Python.

Por favor utiliza:

NotFound(detail="Aquí el mensaje que quieras relacionado con que no se encontró el edicto")

Por qué utilizas esto? A donde se ejecuta una transacción dentro de éste método?

https://docs.djangoproject.com/en/4.2/topics/db/transactions/

Added 1 commit:

b985804b - correcion de errores

Added 1 commit:

d550baa9 - eliminacion de comentarios

Por favor utiliza:

NotFound(detail="Aquí el mensaje que quieras relacionado con que no se encontró el precio")

Te recomiendo que utilices las excepciones que ya existen en el framework, siempre y cuando no sea algo particular del negocio para luego ser controladas.

Imagina que luego tengas otro modelo y necesites emitir un error 404, vas a tener que crear otra excepción, cuando ya existe la excepción NotFound que representa esto que estás haciendo aquí.

Esto es un principio en la programación: "Don't repeat yourself"

Status changed to closed

  from django.contrib import admin
  from django.core.exceptions import ValidationError
  from django.utils import timezone
  from .models import Edicto, Precio
  # Register your models here.
  admin.site.register(Precio)
  class PrecioAdmin(admin.ModelAdmin):
      def save_model(self, request, obj, form, change):
          precios_activos = Precio.objects.filter(vigencia_hasta__gte=timezone.now().date())

      ordering_fields = ('usuario', )
      ordering = 'usuario'
      queryset = Precio.objects.all()
  class PagoViewSets(viewsets.ModelViewSet):
      serializer_class = PagoSerializer
      permission_classes = [IsAuthenticated, IsAdminOrAuthorized]
      queryset = ComprobantePago.objects.all()
      def perform_create(self, serializer):
          try:
              edicto = Edicto.objects.get(estado='pendiente_de_pago')
              if edicto.usuario != self.request.user:
                  raise UsuarioNoAutorizado

      permission_classes = [IsAuthenticated, IsAdminOrAuthorized]
      queryset = ComprobantePago.objects.all()
      def perform_create(self, serializer):
          try:
              edicto = Edicto.objects.get(estado='pendiente_de_pago')
              if edicto.usuario != self.request.user:
                  raise UsuarioNoAutorizado
              precio = Precio.objects.latest('id')
              monto = calculadora(edicto, precio)
              serializer.validated_data.update({'monto': monto, 'edicto': edicto})
              serializer.save()
          except (Edicto.DoesNotExist, Precio.DoesNotExist):
              raise EdictoOPrecioNoEncontrado

+     result_ejemplar = (copias * precio_ejemplar) + (publicar * precio_ejemplar)
+     if organismo:
+         resultado = result_ejemplar + result_palabra / 2
+     else:
+         resultado = result_ejemplar + result_palabra
+     precio_resultado = ComprobantePago(edicto=edicto, resultado=resultado)
+     precio_resultado.save()
+     return precio_resultado
+ from datetime import datetime, time
+ def calculadora(edicto, precio):
+     try:
+         with transaction.atomic():

      def perform_create(self, serializer):
          try:
              edicto = Edicto.objects.get(estado='pendiente_de_pago')
              if edicto.usuario != self.request.user:
                  raise UsuarioNoAuthorizedException
              precio = Precio.objects.latest('id')
              monto = calculadora(edicto, precio)
              serializer.validated_data.update({'monto': monto, 'edicto': edicto})
              serializer.save()
          except Edicto.DoesNotExist:
              raise EdictoNotFoundException
          except Precio.DoesNotExist:
              raise PrecioNotFoundException

Marta Miranda / boletin_api

Feature/#48 crear dos enpoints para comprobantepago

Closed by Enzo Yair 2023-09-12 13:44:02 UTC

Feature/#48 crear dos enpoints para comprobantepago

Check out, review, and merge locally

Closed by Enzo Yair 2023-09-12 13:44:02 UTC //<![CDATA[ $('.js-timeago-pending').removeClass('js-timeago-pending').timeago() //]]>

Closed by Enzo Yair 2023-09-12 13:44:02 UTC