ACP2E-203: Issues with admin actions report logging

Author: Jon Doe

app/code/Magento/Customer/Model/ResourceModel/CustomerRepository.php

@@ -219,11 +219,7 @@ public function save(CustomerInterface $customer, $passwordHash = null)
         $customer->setAddresses($origAddresses);
         /** @var CustomerModel $customerModel */
         $customerModel = $this->customerFactory->create(['data' => $customerData]);
-        if (!empty($prevCustomerDataArr)) {
-            foreach ($prevCustomerDataArr as $field => $value) {
-                $customerModel->setOrigData($field, $value);
-            }
-        }
+        $this->populateWithOrigData($customerModel, $prevCustomerDataArr);
 
         //Model's actual ID field maybe different than "id" so "id" field from $customerData may be ignored.
         $customerModel->setId($customer->getId());
@@ -301,6 +297,19 @@ public function save(CustomerInterface $customer, $passwordHash = null)
         return $savedCustomer;
     }
 
+    /**
+     * @param CustomerModel $customerModel
+     * @param array $prevCustomerDataArr
+     */
+    private function populateWithOrigData(CustomerModel $customerModel, array $prevCustomerDataArr)
+    {
+        if (!empty($prevCustomerDataArr)) {
+            foreach ($prevCustomerDataArr as $field => $value) {
+                $customerModel->setOrigData($field, $value);
+            }
+        }
+    }
+
     /**
      * Delete addresses by ids
      *

app/code/Magento/Customer/Test/Unit/Model/ResourceModel/CustomerRepositoryTest.php

@@ -694,7 +694,7 @@ public function testGetList(): void
     }
 
     /**
-    * @return void
+     * @return void
     */
     public function testDeleteById(): void
     {
@@ -715,8 +715,8 @@ public function testDeleteById(): void
     }
 
     /**
-    * @return void
-    */
+     * @return void
+     */
     public function testDelete(): void
     {
         $customerId = 14;