Fix clean performance leak

Author: thomas-kl1

Model/Clean.php

@@ -10,8 +10,6 @@
 
 use DateTime;
 use Exception;
-use GhostUnicorns\WebapiLogs\Model\Log\Logger;
-use GhostUnicorns\WebapiLogs\Model\ResourceModel\Entity\LogCollectionFactory;
 use GhostUnicorns\WebapiLogs\Model\ResourceModel\LogResourceModel;
 
 class Clean
@@ -21,71 +19,36 @@ class Clean
      */
     private $config;
 
-    /**
-     * @var Logger
-     */
-    private $logger;
-
-    /**
-     * @var LogCollectionFactory
-     */
-    private $logCollectionFactory;
-
     /**
      * @var ResourceModel\LogResourceModel
      */
     private $logResourceModel;
 
-    /**
-     * @param Config $config
-     * @param Logger $logger
-     * @param LogCollectionFactory $logCollectionFactory
-     * @param LogResourceModel $logResourceModel
-     */
     public function __construct(
         Config $config,
-        Logger $logger,
-        LogCollectionFactory $logCollectionFactory,
         LogResourceModel $logResourceModel
     ) {
         $this->config = $config;
-        $this->logger = $logger;
-        $this->logCollectionFactory = $logCollectionFactory;
         $this->logResourceModel = $logResourceModel;
     }
 
+    public function cleanAll()
+    {
+        $this->logResourceModel->getConnection()->truncateTable($this->logResourceModel->getMainTable());
+    }
+
     /**
      * @throws Exception
      */
     public function execute()
     {
-        if (!$this->config->isEnabled()) {
-            return;
-        }
-
-        $this->logger->info(__('Start webapi logs clean'));
-        $hours = $this->config->getCleanOlderThanHours();
-        $datetime = new DateTime('-' . $hours . ' hour');
-        $page = 1;
-
-        $collection = $this->logCollectionFactory->create();
-        $collection = $collection->addFieldToSelect(LogResourceModel::LOG_ID)
-                                 ->addFieldToFilter(LogResourceModel::CREATED_AT, ['lt' => $datetime])
-                                 ->setPageSize(2);
-
-        $pageCount = $collection->getLastPageNumber();
-        $currentPage = 1;
-        $tot = 0;
-        while ($currentPage <= $pageCount) {
-            $collection->setCurPage($currentPage);
-            foreach ($collection as $row) {
-                $this->logResourceModel->delete($row);
-                $tot++;
-            }
-            $collection->clear();
-            $currentPage++;
-        }
-
-        $this->logger->info(__('End webapi logs clean. Deleted %1 elements.', $tot));
+        $this->logResourceModel->getConnection()->delete(
+            $this->logResourceModel->getMainTable(),
+            sprintf(
+                '%s < NOW() - INTERVAL %s HOUR',
+                LogResourceModel::CREATED_AT,
+                (int)$this->config->getCleanOlderThanHours()
+            )
+        );
     }
 }