From 835173db6cda78a5e809d144931beed92387fafa Mon Sep 17 00:00:00 2001
From: Andrey Rusakov <andrey.rusakov@camptocamp.com>
Date: Thu, 5 Dec 2024 16:33:41 +0100
Subject: [PATCH] Use ID to select format for the order address

---
 .../account/new-order/new-order.component.ts    | 17 ++++++-----------
 src/locale/messages.de.xlf                      |  4 ++--
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/src/app/account/new-order/new-order.component.ts b/src/app/account/new-order/new-order.component.ts
index c071107a..95c25510 100644
--- a/src/app/account/new-order/new-order.component.ts
+++ b/src/app/account/new-order/new-order.component.ts
@@ -73,7 +73,7 @@ export class NewOrderComponent implements OnInit, OnDestroy {
   }
 
   get IsOrderTypePrivate() {
-    return this.orderFormGroup?.get('orderType')?.value?.name.startsWith('Priv');
+    return this.orderFormGroup?.get('orderType')?.value?.id === 1;
   }
 
   get orderTypeCtrl() {
@@ -201,9 +201,8 @@ export class NewOrderComponent implements OnInit, OnDestroy {
     return returnValue;
   }
 
-  private getOrderType(id: string | number) {
-    return this.orderTypes.find(x => typeof id === 'number' ?
-      id === x.id : id === x.name) || {
+  private getOrderType(id: number) {
+    return this.orderTypes.find(x => id === x.id) || {
       id: 1,
       name: ConstantsService.ORDERTYPE_PRIVATE
     };
@@ -221,11 +220,7 @@ export class NewOrderComponent implements OnInit, OnDestroy {
     });
     this.orderTypeCtrl?.valueChanges.subscribe(
       (choice) => {
-        if (choice.name === ConstantsService.ORDER_NAME.PRIVATE) {
-          this.addressChoiceCtrl?.setValue('1');
-        } else {
-          this.addressChoiceCtrl?.setValue('2');
-        }
+        this.addressChoiceCtrl?.setValue(choice.id);
       }
     );
     this.emailDeliverChoiceCtrl?.valueChanges.subscribe(
@@ -270,7 +265,7 @@ export class NewOrderComponent implements OnInit, OnDestroy {
     this.isCustomerSelected = order.HasInvoiceContact;
 
     this.orderFormGroup?.setValue({
-      orderType: this.getOrderType(order.order_type),
+      orderType: this.getOrderType(parseInt(order.order_type)),
       title: order.title,
       invoice_reference: order.invoice_reference,
       emailDeliver: order.email_deliver,
@@ -428,7 +423,7 @@ export class NewOrderComponent implements OnInit, OnDestroy {
   resetForms() {
     this.isCustomerSelected = this.currentOrder.HasInvoiceContact;
     this.orderFormGroup.reset({
-      orderType: this.getOrderType(this.currentOrder.order_type),
+      orderType: this.getOrderType(parseInt(this.currentOrder.order_type)),
       title: this.currentOrder.title,
       invoice_reference: this.currentOrder.invoice_reference,
       emailDeliver: this.currentOrder.email_deliver,
diff --git a/src/locale/messages.de.xlf b/src/locale/messages.de.xlf
index 5314ae4f..d3dd1bb4 100644
--- a/src/locale/messages.de.xlf
+++ b/src/locale/messages.de.xlf
@@ -1175,7 +1175,7 @@
       </trans-unit>
       <trans-unit id="order.public" datatype="html">
         <source>Public</source>
-        <target>Öffentlich</target>
+        <target>Für einen Mandanten</target>
         <context-group purpose="location">
           <context context-type="sourcefile">src/app/constants.service.ts</context>
           <context context-type="linenumber">37</context>
@@ -1183,7 +1183,7 @@
       </trans-unit>
       <trans-unit id="order.private" datatype="html">
         <source>Privé</source>
-        <target>Privat</target>
+        <target>Für mich selbst</target>
         <context-group purpose="location">
           <context context-type="sourcefile">src/app/constants.service.ts</context>
           <context context-type="linenumber">38</context>