Abandoned cart report generate duplicate records

Author: glo05363

app/code/Magento/Backend/Block/Widget/Grid/Extended.php

@@ -87,8 +87,6 @@ class Extended extends \Magento\Backend\Block\Widget\Grid implements \Magento\Ba
     protected $_massactionIdFilter;
 
     /**
-     * Massaction block name
-     *
      * @var string
      */
     protected $_massactionBlockName = \Magento\Backend\Block\Widget\Grid\Massaction\Extended::class;
@@ -122,8 +120,6 @@ class Extended extends \Magento\Backend\Block\Widget\Grid implements \Magento\Ba
     protected $_headersVisibility = true;
 
     /**
-     * Filter visibility
-     *
      * @var boolean
      */
     protected $_filterVisibility = true;
@@ -148,15 +144,11 @@ class Extended extends \Magento\Backend\Block\Widget\Grid implements \Magento\Ba
     protected $_isCollapsed;
 
     /**
-     * Count subtotals
-     *
      * @var boolean
      */
     protected $_countSubTotals = false;
 
     /**
-     * SubTotals
-     *
      * @var \Magento\Framework\DataObject[]
      */
     protected $_subtotals = [];
@@ -964,6 +956,7 @@ public function _exportIterateCollection($callback, array $args)
 
         while ($break !== true) {
             $collection = clone $originalCollection;
+            $collection->clear();
             $collection->setPageSize($this->_exportPageSize);
             $collection->setCurPage($page);
             $collection->load();
@@ -975,7 +968,6 @@ public function _exportIterateCollection($callback, array $args)
                 $break = true;
             }
             $page++;
-
             foreach ($collection as $item) {
                 //phpcs:ignore Magento2.Functions.DiscouragedFunction
                 call_user_func_array(

app/code/Magento/Reports/Model/ResourceModel/Quote/Collection.php

@@ -200,9 +200,22 @@ public function resolveCustomerNames()
         foreach ($this->getItems() as $item) {
             foreach ($customersData as $customerItemData) {
                 if ($item['customer_id'] == $customerItemData['entity_id']) {
-                    $item->setData(array_merge($item->getData(), $customerItemData));
+                    $item['customer_name'] = $customerItemData['customer_name'];
+                    $item['email'] = $customerItemData['email'];
+                    $item->setData($item->getData());
                 }
             }
         }
     }
+
+    /**
+     * Append customer data after collection load
+     *
+     * @return Collection|void
+     */
+    protected function _afterLoad()
+    {
+        parent::_afterLoad();
+        $this->resolveCustomerNames();
+    }
 }

app/code/Magento/Reports/Test/Unit/Model/ResourceModel/Quote/CollectionTest.php

@@ -131,41 +131,41 @@ public function testResolveCustomerNames()
         $this->selectMock->expects($this->any())
             ->method('getConnection')
             ->willReturn($this->connectionMock);
-        $this->selectMock->expects($this->once())
+        $this->selectMock->expects($this->atLeastOnce())
             ->method('from')
             ->with(['customer' => $customerTableName], ['entity_id', 'email'])
             ->willReturnSelf();
-        $this->selectMock->expects($this->once())
+        $this->selectMock->expects($this->atLeastOnce())
             ->method('columns')
             ->with(['customer_name' => $customerName])
             ->willReturnSelf();
-        $this->selectMock->expects($this->once())
+        $this->selectMock->expects($this->atLeastOnce())
             ->method('where')
             ->with('customer.entity_id IN (?)')
             ->willReturnSelf();
 
-        $this->connectionMock->expects($this->once())
+        $this->connectionMock->expects($this->atLeastOnce())
             ->method('getConcatSql')
             ->with(['firstname', 'lastname'], ' ')
             ->willReturn($customerName);
 
         $this->customerResourceMock->expects($this->any())
             ->method('getConnection')
             ->willReturn($this->connectionMock);
-        $this->customerResourceMock->expects($this->once())
+        $this->customerResourceMock->expects($this->atLeastOnce())
             ->method('getTable')
             ->with('customer_entity')
             ->willReturn($customerTableName);
 
         $this->connectionMock->expects($this->any())
             ->method('select')
             ->willReturn($this->selectMock);
-        $this->connectionMock->expects($this->once())
+        $this->connectionMock->expects($this->atLeastOnce())
             ->method('fetchAll')
             ->with($this->selectMock)
             ->willReturn($customersData);
 
-        $this->fetchStrategyMock->expects($this->once())
+        $this->fetchStrategyMock->expects($this->atLeastOnce())
             ->method('fetchAll')
             ->withAnyParameters()
             ->willReturn($customerId);