From 9bf971b040bee3266bc80b6145c3c7a2d2499a04 Mon Sep 17 00:00:00 2001
From: Vincent BRIDIER <vincent.bridier@ldc.fr>
Date: Tue, 23 Feb 2021 17:49:51 +0100
Subject: [PATCH] - Update CRM Client V1

---
 .../Controllers/Api/v1/CustomerController.php | 20 ++++----
 clients/LDCCRM_V1.php                         | 47 ++++++++++++++-----
 2 files changed, 44 insertions(+), 23 deletions(-)

diff --git a/app/Http/Controllers/Api/v1/CustomerController.php b/app/Http/Controllers/Api/v1/CustomerController.php
index b93a1d8..ef10b5b 100644
--- a/app/Http/Controllers/Api/v1/CustomerController.php
+++ b/app/Http/Controllers/Api/v1/CustomerController.php
@@ -158,34 +158,34 @@ class CustomerController extends Controller
         if (!$customerInfos)
             return new StatusResource(['status' => 'KO', 'code' => 105, 'message' => 'User not found.']);
 
-        if ($request->firstname !== null)
+        if (isset($request->firstname))
             $customerInfos->firstname = filter_var($request->firstname, FILTER_SANITIZE_STRING);
 
-        if ($request->lastname !== null)
+        if (isset($request->lastname))
             $customerInfos->lastname = filter_var($request->lastname, FILTER_SANITIZE_STRING);
 
-        if ($request->birth_date !== null)
+        if (isset($request->birth_date))
             $customerInfos->birth_date = filter_var($request->birth_date, FILTER_SANITIZE_STRING);
 
-        if ($request->address !== null)
+        if (isset($request->address))
             $customerInfos->address = filter_var($request->address, FILTER_SANITIZE_STRING);
 
-        if ($request->address2 !== null)
+        if (isset($request->address2))
             $customerInfos->address2 = filter_var($request->address2, FILTER_SANITIZE_STRING);
 
-        if ($request->phone !== null)
+        if (isset($request->phone))
             $customerInfos->phone = filter_var($request->phone, FILTER_SANITIZE_STRING);
 
-        if ($request->country !== null)
+        if (isset($request->country))
             $customerInfos->country = filter_var($request->country, FILTER_SANITIZE_STRING);
 
-        if ($request->city !== null)
+        if (isset($request->city))
             $customerInfos->city = filter_var($request->city, FILTER_SANITIZE_STRING);
 
-        if ($request->postcode !== null)
+        if (isset($request->postcode))
             $customerInfos->postcode = filter_var($request->postcode, FILTER_SANITIZE_STRING);
 
-        if ($request->opt_in !== null) {
+        if (isset($request->opt_in)) {
             $optIn = filter_var($request->opt_in, FILTER_SANITIZE_STRING);
 
             if ($customerInfos->opt_in != $optIn) {
diff --git a/clients/LDCCRM_V1.php b/clients/LDCCRM_V1.php
index 4d6c6da..7499bf9 100644
--- a/clients/LDCCRM_V1.php
+++ b/clients/LDCCRM_V1.php
@@ -39,19 +39,40 @@ class ApiV1
         return $this->callAPI('POST', '/customer/create', $user);
     }
 
-    public function updateCustomer($crmId, $firstname, $lastname, $optin, $birthdate = null, $phone = null, $address = null, $address2 = null, $city = null, $postcode = null, $country = null){
-        $user = [
-            'firstname' => $firstname,
-            'lastname' => $lastname,
-            'opt_in' => $optin,
-            'birth_date' => $birthdate,
-            'phone' => $phone,
-            'address' => $address,
-            'address2' => $address2,
-            'city' => $city,
-            'postcode' => $postcode,
-            'country' => $country
-        ];
+    public function updateCustomer($crmId, $firstname = null, $lastname = null, $optin = null, $birthdate = null, $phone = null, $address = null, $address2 = null, $city = null, $postcode = null, $country = null){
+        $user = [];
+
+        if ($firstname !== null)
+            $user['firstname'] = $firstname;
+
+        if ($lastname !== null)
+            $user['lastname'] = $lastname;
+
+        if ($optin !== null)
+            $user['opt_in'] = $optin;
+
+        if ($birthdate !== null)
+            $user['birth_date'] = $birthdate;
+
+        if ($phone !== null)
+            $user['phone'] = $phone;
+
+        if ($address !== null)
+            $user['address'] = $address;
+
+        if ($address2 !== null)
+            $user['address2'] = $address2;
+
+        if ($city !== null)
+            $user['city'] = $city;
+
+        if ($postcode !== null)
+            $user['postcode'] = $postcode;
+
+        if ($country !== null)
+            $user['country'] = $country;
+
+
         return $this->callAPI('PUT', "/customer/$crmId", $user);
     }