From 0313abb010611075ff678231b8790fb853c2f862 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Mon, 29 Jul 2024 11:49:27 +0200
Subject: [PATCH 1/2] Allow other billing address for cembra

---
 .../pages/onboarding/vv/CheckoutAddress.vue   |  2 +-
 server/vbv_lernwelt/shop/invoice/abacus.py    | 12 ++---
 server/vbv_lernwelt/shop/models.py            | 12 +++++
 .../shop/tests/test_abacus_invoice.py         | 47 +++++++++++++++++++
 4 files changed, 66 insertions(+), 7 deletions(-)

diff --git a/client/src/pages/onboarding/vv/CheckoutAddress.vue b/client/src/pages/onboarding/vv/CheckoutAddress.vue
index ba309415..3bcfa0d3 100644
--- a/client/src/pages/onboarding/vv/CheckoutAddress.vue
+++ b/client/src/pages/onboarding/vv/CheckoutAddress.vue
@@ -293,7 +293,7 @@ const executePayment = async () => {
         {{ formErrors.personal.join(", ") }}
       </p>
 
-      <section v-if="address.payment_method !== 'cembra_byjuno'">
+      <section>
         <div class="mt-4">
           <button
             v-if="!withCompanyAddress"
diff --git a/server/vbv_lernwelt/shop/invoice/abacus.py b/server/vbv_lernwelt/shop/invoice/abacus.py
index 1af1a09a..2e7f9491 100644
--- a/server/vbv_lernwelt/shop/invoice/abacus.py
+++ b/server/vbv_lernwelt/shop/invoice/abacus.py
@@ -78,32 +78,32 @@ def create_customer_xml(checkout_information: CheckoutInformation):
         first_name=checkout_information.first_name,
         company_name=(
             checkout_information.organisation_detail_name
-            if checkout_information.invoice_address == "org"
+            if checkout_information.abacus_use_organisation_data()
             else ""
         ),
         street=(
             checkout_information.organisation_street
-            if checkout_information.invoice_address == "org"
+            if checkout_information.abacus_use_organisation_data()
             else checkout_information.street
         ),
         house_number=(
             checkout_information.organisation_street_number
-            if checkout_information.invoice_address == "org"
+            if checkout_information.abacus_use_organisation_data()
             else checkout_information.street_number
         ),
         zip_code=(
             checkout_information.organisation_postal_code
-            if checkout_information.invoice_address == "org"
+            if checkout_information.abacus_use_organisation_data()
             else checkout_information.postal_code
         ),
         city=(
             checkout_information.organisation_city
-            if checkout_information.invoice_address == "org"
+            if checkout_information.abacus_use_organisation_data()
             else checkout_information.city
         ),
         country=(
             checkout_information.organisation_country_id
-            if checkout_information.invoice_address == "org"
+            if checkout_information.abacus_use_organisation_data()
             else checkout_information.country_id
         ),
         language=customer.language,
diff --git a/server/vbv_lernwelt/shop/models.py b/server/vbv_lernwelt/shop/models.py
index 8373bc4d..04694fc6 100644
--- a/server/vbv_lernwelt/shop/models.py
+++ b/server/vbv_lernwelt/shop/models.py
@@ -128,3 +128,15 @@ class CheckoutInformation(models.Model):
         self.abacus_order_id = new_abacus_order_id
         self.save()
         return self
+
+    def abacus_address_type(self) -> str:
+        # always use priv for abacus and CembraPay
+        return (
+            self.INVOICE_ADDRESS_ORGANISATION
+            if self.invoice_address == self.INVOICE_ADDRESS_ORGANISATION
+            and not self.cembra_byjuno_invoice
+            else self.INVOICE_ADDRESS_PRIVATE
+        )
+
+    def abacus_use_organisation_data(self) -> bool:
+        return self.abacus_address_type() == self.INVOICE_ADDRESS_ORGANISATION
diff --git a/server/vbv_lernwelt/shop/tests/test_abacus_invoice.py b/server/vbv_lernwelt/shop/tests/test_abacus_invoice.py
index e7171ba0..025d0239 100644
--- a/server/vbv_lernwelt/shop/tests/test_abacus_invoice.py
+++ b/server/vbv_lernwelt/shop/tests/test_abacus_invoice.py
@@ -163,6 +163,53 @@ class AbacusInvoiceTestCase(TestCase):
         assert "<Street>Laupenstrasse</Street>" in customer_xml_content
         assert "<Country>CH</Country>" in customer_xml_content
 
+    def test_create_customer_xml_byjuno_cembra_with_company_address(self):
+        _pat = User.objects.get(username="patrizia.huggel@eiger-versicherungen.ch")
+        _pat.abacus_debitor_number = 60000011
+        _pat.save()
+        _ignore_checkout_information = CheckoutInformationFactory(
+            user=_pat, abacus_order_id=6_000_000_123
+        )
+
+        feuz = User.objects.get(username="andreas.feuz@eiger-versicherungen.ch")
+        feuz_checkout_info = CheckoutInformationFactory(
+            user=feuz,
+            transaction_id="24021508331287484",
+            first_name="Andreas",
+            last_name="Feuz",
+            street="Eggersmatt",
+            street_number="32",
+            postal_code="1719",
+            city="Zumholz",
+            country_id="CH",
+            invoice_address="org",
+            organisation_detail_name="VBV",
+            organisation_street="Laupenstrasse",
+            organisation_street_number="10",
+            organisation_postal_code="3000",
+            organisation_city="Bern",
+            organisation_country_id="CH",
+            cembra_byjuno_invoice=True,
+        )
+        feuz_checkout_info.created_at = datetime(2024, 2, 15, 8, 33, 12, 0)
+
+        customer_xml_filename, customer_xml_content = create_customer_xml(
+            checkout_information=feuz_checkout_info
+        )
+        print(customer_xml_content)
+        print(customer_xml_filename)
+
+        self.assertEqual("myVBV_debi_60000012.xml", customer_xml_filename)
+        assert "<CustomerNumber>60000012</CustomerNumber>" in customer_xml_content
+        assert (
+            "<Email>andreas.feuz@eiger-versicherungen.ch</Email>"
+            in customer_xml_content
+        )
+        assert "<AddressNumber>60000012</AddressNumber>" in customer_xml_content
+        assert "<Name>Feuz</Name>" in customer_xml_content
+        assert "<Text>VBV</Text>" not in customer_xml_content
+        assert "<Street>Eggersmatt</Street>" in customer_xml_content
+
     def test_render_customer_xml(self):
         customer_xml = render_customer_xml(
             abacus_debitor_number=60000012,

From 9603b37196e7aaa3a10180030dfb18e2c220acf3 Mon Sep 17 00:00:00 2001
From: Christian Cueni <christian.cueni@iterativ.ch>
Date: Tue, 30 Jul 2024 14:03:45 +0200
Subject: [PATCH 2/2] Add command for company stats

---
 .../management/commands/count_vv_members.py   | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 server/vbv_lernwelt/course_session/management/commands/count_vv_members.py

diff --git a/server/vbv_lernwelt/course_session/management/commands/count_vv_members.py b/server/vbv_lernwelt/course_session/management/commands/count_vv_members.py
new file mode 100644
index 00000000..35b62be1
--- /dev/null
+++ b/server/vbv_lernwelt/course_session/management/commands/count_vv_members.py
@@ -0,0 +1,30 @@
+import djclick as click
+import structlog
+from django.db.models import Count
+
+from vbv_lernwelt.course.models import CourseSessionUser
+
+logger = structlog.get_logger(__name__)
+
+
+@click.command()
+def command():
+    VV_COURSE_SESSIONS = [1, 2, 3]  # DE, FR, IT
+
+    # Aggregation of users per organisation for a specific course session
+    user_counts = (
+        CourseSessionUser.objects.filter(course_session__id__in=VV_COURSE_SESSIONS)
+        .values("user__organisation__organisation_id", "user__organisation__name_de")
+        .annotate(user_count=Count("id"))
+        .order_by("user__organisation__organisation_id")
+    )
+
+    for entry in user_counts:
+        print(
+            f"Organisation Name: {entry['user__organisation__name_de']}, "
+            f"User Count: {entry['user_count']}"
+        )
+
+    print(
+        f"Total number of users: {sum([entry['user_count'] for entry in user_counts])}"
+    )