Merge pull request #550 from bruecksen/549-imp-select-point-of-sales-per-production-day

Add point of sale to production day

Author: bruecksen

bakeup/pages/models.py

@@ -15,7 +15,6 @@
     CustomerOrder,
     CustomerOrderPosition,
     CustomerOrderTemplatePosition,
-    PointOfSale,
     ProductionDay,
     ProductionDayProduct,
 )
@@ -171,10 +170,10 @@ def get_context(self, request, *args, **kwargs):
                     )
                 )
             context["production_day_products"] = production_day_products
+            context["point_of_sales"] = self.production_day.point_of_sales.all()
         context["show_remaining_products"] = (
             request.tenant.clientsetting.show_remaining_products
         )
-        context["point_of_sales"] = PointOfSale.objects.all()
         context["all_production_days"] = list(
             ProductionDay.objects.published()
             .available_to_user(request.user)

bakeup/shop/forms.py

@@ -11,6 +11,7 @@
 from bakeup.shop.models import (
     Customer,
     CustomerOrderPosition,
+    PointOfSale,
     ProductionDay,
     ProductionDayProduct,
 )
@@ -120,7 +121,7 @@ def clean(self):
 class ProductionDayForm(forms.ModelForm):
     class Meta:
         model = ProductionDay
-        fields = ["day_of_sale", "description"]
+        fields = ["day_of_sale", "description", "point_of_sales"]
         widgets = {
             "day_of_sale": forms.DateInput(
                 format="%Y-%m-%d",
@@ -131,8 +132,18 @@ class Meta:
                 },
             ),
             "description": forms.Textarea(attrs={"rows": 3}),
+            "point_of_sales": forms.CheckboxSelectMultiple(
+                attrs={"class": "form-control"}
+            ),
         }
 
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        if not PointOfSale.objects.exists():
+            del self.fields["point_of_sales"]
+        elif not self.instance.pk:
+            self.fields["point_of_sales"].initial = PointOfSale.objects.all()
+
 
 class ProductionDayProductForm(forms.ModelForm):
     class Meta:

bakeup/shop/migrations/0005_productionday_point_of_sales.py

@@ -0,0 +1,24 @@
+# Generated by Django 4.2.23 on 2025-10-28 10:40
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("shop", "0004_alter_pointofsale_is_primary_alter_pointofsale_name_and_more"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="productionday",
+            name="point_of_sales",
+            field=models.ManyToManyField(
+                blank=True,
+                help_text="Select the point of sales where this production day is available.",
+                related_name="production_days",
+                to="shop.pointofsale",
+                verbose_name="Point of Sales",
+            ),
+        ),
+    ]

bakeup/shop/migrations/0006_add_pos.py

@@ -0,0 +1,28 @@
+# Generated by Django 4.2.23 on 2025-10-29 19:09
+
+from django.db import migrations
+
+
+def create_pos(apps, schema_editor):
+    ProductionDay = apps.get_model("shop", "ProductionDay")
+    PointOfSale = apps.get_model("shop", "PointOfSale")
+    point_of_sales = PointOfSale.objects.all()
+    for production_day in ProductionDay.objects.all():
+        if not production_day.point_of_sales.exists():
+            production_day.point_of_sales.set(point_of_sales)
+
+def remove_pos(apps, schema_editor):
+    ProductionDay = apps.get_model("shop", "ProductionDay")
+    for production_day in ProductionDay.objects.all():
+        production_day.point_of_sales.clear()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("shop", "0005_productionday_point_of_sales"),
+    ]
+
+    operations = [
+        migrations.RunPython(create_pos, remove_pos),
+    ]

bakeup/shop/models.py

@@ -83,6 +83,16 @@ class ProductionDay(CommonBaseClass):
         unique=True, verbose_name=_("Day of Sale"), db_index=True
     )
     description = models.TextField(blank=True, null=True, verbose_name=_("Description"))
+    point_of_sales = models.ManyToManyField(
+        "shop.PointOfSale",
+        blank=False,
+        related_name="production_days",
+        verbose_name=_("Point of Sales"),
+        help_text=_(
+            "Select the point of sales where this production day is available. At least"
+            " one must be selected."
+        ),
+    )
 
     objects = ProductionDayQuerySet.as_manager()
 
@@ -1215,41 +1225,50 @@ def create_abo_orders_for_production_days(
                         production_day.production_day_products.filter(product=product)
                     )
                     if not customer_order_position.exists() and production_day_product:
-                        production_day_product = production_day_product.get()
-                        max_quantity = production_day_product.calculate_max_quantity(
-                            customer
-                        )
-                        quantity = min(
-                            customer_order_template_position.quantity, max_quantity
-                        )
-                        if not production_day_product.is_locked and quantity > 0:
-                            print(
-                                "Create Abo order: ", production_day, product, quantity
+                        if (
+                            not production_day.point_of_sales.exists()
+                            or customer.point_of_sale
+                            in production_day.point_of_sales.all()
+                        ):
+                            # check if pos is available for this production day
+                            production_day_product = production_day_product.get()
+                            max_quantity = (
+                                production_day_product.calculate_max_quantity(customer)
                             )
-                            customer_order, created = (
-                                CustomerOrder.objects.get_or_create(
-                                    production_day=production_day,
-                                    customer=customer,
-                                    defaults={
-                                        "point_of_sale": customer.point_of_sale,
-                                    },
-                                )
-                            )
-                            price = None
-                            price_total = None
-                            if product.sale_price:
-                                price = product.sale_price.price.amount
-                                price_total = price * quantity
-                            position = CustomerOrderPosition.objects.create(
-                                order=customer_order,
-                                product=product,
-                                quantity=quantity,
-                                price=price,
-                                price_total=price_total,
+                            quantity = min(
+                                customer_order_template_position.quantity, max_quantity
                             )
-                            customer_order_template_position.orders.add(position)
-                            customer_order_template_position.order_template.set_locked()
-                            is_order_created = True
+                            if not production_day_product.is_locked and quantity > 0:
+                                print(
+                                    "Create Abo order: ",
+                                    production_day,
+                                    product,
+                                    quantity,
+                                )
+                                customer_order, created = (
+                                    CustomerOrder.objects.get_or_create(
+                                        production_day=production_day,
+                                        customer=customer,
+                                        defaults={
+                                            "point_of_sale": customer.point_of_sale,
+                                        },
+                                    )
+                                )
+                                price = None
+                                price_total = None
+                                if product.sale_price:
+                                    price = product.sale_price.price.amount
+                                    price_total = price * quantity
+                                position = CustomerOrderPosition.objects.create(
+                                    order=customer_order,
+                                    product=product,
+                                    quantity=quantity,
+                                    price=price,
+                                    price_total=price_total,
+                                )
+                                customer_order_template_position.orders.add(position)
+                                customer_order_template_position.order_template.set_locked()
+                                is_order_created = True
                 from bakeup.pages.models import EmailSettings
 
                 if (

bakeup/shop/views.py

@@ -377,10 +377,10 @@ def get_context_data(self, **kwargs):
                     )
                 )
             context["production_day_products"] = production_day_products
+            context["point_of_sales"] = self.production_day.point_of_sales.all()
         context["show_remaining_products"] = (
             self.request.tenant.clientsetting.show_remaining_products
         )
-        context["point_of_sales"] = PointOfSale.objects.all()
         context["production_days"] = ProductionDay.objects.upcoming().exclude(
             id=self.production_day.pk
         )

bakeup/templates/workshop/productionday_detail.html

@@ -48,7 +48,14 @@
 </ul>
 <div class="tab-content">
   <div class="tab-pane fade show active" id="summary" role="tabpanel" aria-labelledby="summary-tab">
-    <p class="mt-4"><strong>{% trans "Description" %}: </strong> {{ object.description }}</p>
+    <p class="mt-4 mb-0"><strong>{% trans "Description" %}: </strong> {{ object.description }}</p>
+    {% if object.point_of_sales.exists %}
+    <p class="mb-0"><strong>{% trans "Point of sales" %}: </strong>
+      {% for pos in object.point_of_sales.all %}
+      {{ pos }}{% if not forloop.last %}, {% endif %}
+      {% endfor %}
+    </p>
+    {% endif %}
 
     <h2 class="mt-4">{% trans "Summary" %}
 

bakeup/templates/workshop/productionday_form.html

@@ -17,6 +17,9 @@
       {{ formset.management_form }}
       <div class="card p-3 mb-3">
         {% bootstrap_field form.day_of_sale %}
+        {% if form.point_of_sales %}
+        {% bootstrap_field form.point_of_sales %}
+        {% endif %}
         {% bootstrap_field form.description %}
       </div>
   </div>