Manejo de Pedidos en CoffeShop

Realicé este complemento con funciones o methodos dentro de los modelo para calcular total, igv, subtotal: Pero no son campos dentro de los modelo, solo se calculan por los campos quantity y precio. Mi pregunta esta es una buena practica? o debo crear los campos en los modelos para estos casos que son necesarios los totales, subtotales, igv (IVA) ? Y como lo realizaría? 

Esto estaba en la documentación: #### The `LoginRequiredMixin` mixin[¶](https://docs.djangoproject.com/en/5.1/topics/auth/default/#the-loginrequiredmixin-mixin "Permalink to this headline") When using [class-based views](https://docs.djangoproject.com/en/5.1/topics/class-based-views/), you can achieve the same behavior as with `login_required` by using the `LoginRequiredMixin`. This mixin should be at the leftmost position in the inheritance list. *class* `LoginRequiredMixin`[\[source\]](https://github.com/django/django/blob/stable/5.1.x/django/contrib/auth/mixins.py#L67)[¶](https://docs.djangoproject.com/en/5.1/topics/auth/default/#django.contrib.auth.mixins.LoginRequiredMixin "Permalink to this definition") If a view is using this mixin, all requests by non-authenticated users will be redirected to the login page or shown an HTTP 403 Forbidden error, depending on the [`raise_exception`](https://docs.djangoproject.com/en/5.1/topics/auth/default/#django.contrib.auth.mixins.AccessMixin.raise_exception "django.contrib.auth.mixins.AccessMixin.raise_exception") parameter. You can set any of the parameters of [`AccessMixin`](https://docs.djangoproject.com/en/5.1/topics/auth/default/#django.contrib.auth.mixins.AccessMixin "django.contrib.auth.mixins.AccessMixin") to customize the handling of unauthorized users: from django.contrib.auth.mixins import LoginRequiredMixin class MyView(LoginRequiredMixin, View): login\_url = "/login/" redirect\_field\_name = "redirect\_to" url: <https://docs.djangoproject.com/en/5.1/topics/auth/default/> Pero si vas al source code te muestra como usarlo: class LoginRequiredMixin(AccessMixin): """Verify that the current user is authenticated.""" def dispatch(self, request, \*args, \*\*kwargs): if not request.user.is\_authenticated: return self.handle\_no\_permission() return super().dispatch(request, \*args, \*\*kwargs) Lo unico que le agregue al codigo del profe fue: ```python from django.shortcuts import render from django.views.generic import DetailView from .models import Order from django.contrib.auth.mixins import LoginRequiredMixin #la importación del mixin class OrderDetailView(LoginRequiredMixin, DetailView): #se le pasa a la clase model = Order template_name = "orders/order_detail.html" context_object_name = 'order' def get_object(self, queryset=None): return Order.objects.filter(user=self.request.user, is_active=True).first() #y en el filtro se pone otra condición para que el is:active pertenezca al usuario logeado ```from django.shortcuts import renderfrom django.views.generic import DetailViewfrom .models import Orderfrom django.contrib.auth.mixins import LoginRequiredMixin class OrderDetailView(LoginRequiredMixin, DetailView): model = Order template\_name = "orders/order\_detail.html" context\_object\_name = 'order' def get\_object(self, queryset=None): return Order.objects.filter(user=self.request.user, is\_active=True).first()