fix counting different operations statuses in bulkStatus

Author: Vasiliev.A

app/code/Magento/AsynchronousOperations/Model/BulkStatus.php

@@ -153,24 +153,6 @@ public function getFailedOperationsByBulkId($bulkUuid, $failureType = null)
      */
     public function getOperationsCountByBulkIdAndStatus($bulkUuid, $status)
     {
-        if ($status === OperationInterface::STATUS_TYPE_OPEN) {
-            /**
-             * Total number of operations that has been scheduled within the given bulk
-             */
-            $allOperationsQty = $this->getOperationCount($bulkUuid);
-
-            /**
-             * Number of operations that has been processed (i.e. operations with any status but 'open')
-             */
-            $allProcessedOperationsQty = (int)$this->operationCollectionFactory->create()
-                                                                               ->addFieldToFilter(
-                                                                                   'bulk_uuid',
-                                                                                   $bulkUuid
-                                                                               )
-                                                                               ->getSize();
-
-            return $allOperationsQty - $allProcessedOperationsQty;
-        }
 
         /** @var \Magento\AsynchronousOperations\Model\ResourceModel\Operation\Collection $collection */
         $collection = $this->operationCollectionFactory->create();
@@ -295,12 +277,12 @@ public function getBulkDetails($bulkUuid)
         /** @var \Magento\AsynchronousOperations\Model\Operation\Details $operationDetails */
         $operationDetails = $this->operationDetailsFactory->create(['bulkUuid' => $bulkUuid]);
 
-        /** @var \Magento\AsynchronousOperations\Api\Data\BulkSummaryExtensionInterface $bulkExtensionAttribute */
-        $bulkExtensionAttribute = $this->bulkSummaryExtensionInterfaceFactory->create();
-        $bulkExtensionAttribute->setOperationsList($operationList);
+        /** @var \Magento\AsynchronousOperations\Api\Data\BulkSummaryExtensionInterface $bulkAttribute */
+        $bulkAttribute = $this->bulkSummaryExtensionInterfaceFactory->create();
+        $bulkAttribute->setOperationsList($operationList);
 
-        $bulkExtensionAttribute->setOperationsCount($operationDetails);
-        $bulk->setExtensionAttributes($bulkExtensionAttribute);
+        $bulkAttribute->setOperationsCount($operationDetails);
+        $bulk->setExtensionAttributes($bulkAttribute);
 
         return $bulk;
     }

app/code/Magento/AsynchronousOperations/Model/Operation/Details.php

@@ -78,12 +78,7 @@ public function getDetails($bulkUuid)
             );
         }
 
-        // total is sum of successful, rejected, retriable, not retriable and open operations
-        $details['operations_total'] =
-            array_sum($details) + $this->bulkStatus->getOperationsCountByBulkIdAndStatus(
-                $bulkUuid,
-                OperationInterface::STATUS_TYPE_OPEN
-            );
+        $details['operations_total'] = array_sum($details);
         $details['operations_failed'] = $details['failed_retriable'] + $details['failed_not_retriable'];
         $this->operationCache[$bulkUuid] = $details;