From bac46aa01cee4c993dfc34e6db4c20ede0520c2d Mon Sep 17 00:00:00 2001
From: Peter Jaap Blaakmeer <peterjaap@blaakmeer.com>
Date: Thu, 28 Dec 2023 10:55:22 +0100
Subject: [PATCH] Improve UI to enable/disable cronjobs (#202)

---
 .../Adminhtml/Config/Job/MassDisableJobs.php  | 62 +++++++++++++++++++
 Ui/Component/Listing/Column/Frequency.php     | 29 +++++++++
 .../cronjobmanager_config_grid.xml            | 15 ++++-
 3 files changed, 105 insertions(+), 1 deletion(-)
 create mode 100644 Controller/Adminhtml/Config/Job/MassDisableJobs.php
 create mode 100644 Ui/Component/Listing/Column/Frequency.php

diff --git a/Controller/Adminhtml/Config/Job/MassDisableJobs.php b/Controller/Adminhtml/Config/Job/MassDisableJobs.php
new file mode 100644
index 00000000..d8ca3f20
--- /dev/null
+++ b/Controller/Adminhtml/Config/Job/MassDisableJobs.php
@@ -0,0 +1,62 @@
+<?php
+
+namespace EthanYehuda\CronjobManager\Controller\Adminhtml\Config\Job;
+
+use EthanYehuda\CronjobManager\Helper\JobConfig;
+use Exception;
+use Magento\Backend\App\Action\Context;
+use Magento\Backend\App\Action;
+use Magento\Framework\App\CacheInterface;
+use Magento\Framework\App\Config as ConfigCache;
+
+class MassDisableJobs extends Action
+{
+    public const ADMIN_RESOURCE = "EthanYehuda_CronjobManager::cronjobmanager";
+
+    /**
+     * @param Context        $context
+     * @param JobConfig      $helper
+     * @param CacheInterface $cache
+     */
+    public function __construct(
+        Context $context,
+        private readonly JobConfig $helper,
+        private readonly CacheInterface $cache,
+    ) {
+        parent::__construct($context);
+    }
+
+    /**
+     * Save cronjob
+     *
+     * @return void
+     */
+    public function execute()
+    {
+        $params = $this->getRequest()->getParam('selected');
+        if (!isset($params)) {
+            $this->getMessageManager()->addErrorMessage("Something went wrong when receiving the request");
+            $this->_redirect('*/config/index');
+            return;
+        }
+
+        try {
+            foreach ($params as $jobCode) {
+                $path = $this->helper->constructFrequencyPath($jobCode);
+                // Empty frequency disables the job
+                $this->helper->saveJobFrequencyConfig($path, '');
+            }
+
+            // Clear the config cache
+            $this->cache->clean([ConfigCache::CACHE_TAG]);
+        } catch (Exception $e) {
+            $this->getMessageManager()->addErrorMessage($e->getMessage());
+            $this->_redirect('*/config/index/');
+
+            return;
+        }
+
+        $this->getMessageManager()->addSuccessMessage("Successfully disabled " . count($params) . " jobs");
+        $this->_redirect("*/config/index/");
+    }
+}
diff --git a/Ui/Component/Listing/Column/Frequency.php b/Ui/Component/Listing/Column/Frequency.php
new file mode 100644
index 00000000..e4fc0e2d
--- /dev/null
+++ b/Ui/Component/Listing/Column/Frequency.php
@@ -0,0 +1,29 @@
+<?php
+
+namespace EthanYehuda\CronjobManager\Ui\Component\Listing\Column;
+
+use Magento\Ui\Component\Listing\Columns\Column;
+use EthanYehuda\CronjobManager\Api\Data\ScheduleInterface;
+
+class Frequency extends Column
+{
+    /**
+     * Prepare Data Source
+     *
+     * @param array $dataSource
+     *
+     * @return array
+     */
+    public function prepareDataSource(array $dataSource)
+    {
+        if (isset($dataSource['data']['items'])) {
+            foreach ($dataSource['data']['items'] as & $item) {
+                if (empty($item['frequency'])) {
+                    $item['frequency'] = __('Disabled');
+                }
+            }
+        }
+
+        return $dataSource;
+    }
+}
diff --git a/view/adminhtml/ui_component/cronjobmanager_config_grid.xml b/view/adminhtml/ui_component/cronjobmanager_config_grid.xml
index 3d0043a7..55c2b2c8 100644
--- a/view/adminhtml/ui_component/cronjobmanager_config_grid.xml
+++ b/view/adminhtml/ui_component/cronjobmanager_config_grid.xml
@@ -89,6 +89,19 @@
 					</item>
 				</argument>
 			</action>
+			<action name="disable_jobs">
+				<argument name="data" xsi:type="array">
+					<item name="config" xsi:type="array">
+						<item name="confirm" xsi:type="array">
+							<item name="title" xsi:type="string" translate="true">Disable Jobs</item>
+							<item name="message" xsi:type="string" translate="true">Are you sure you want to disable these cron jobs (use Restore to re-enable them)?</item>
+						</item>
+						<item name="type" xsi:type="string">disable_jobs</item>
+						<item name="label" xsi:type="string" translate="true">Disable Jobs</item>
+						<item name="url" xsi:type="url" path="cronjobmanager/config_job/massDisableJobs" />
+					</item>
+				</argument>
+			</action>
 			<action name="restore">
 				<argument name="data" xsi:type="array">
 					<item name="config" xsi:type="array">
@@ -138,7 +151,7 @@
 				</item>
 			</argument>
 		</column>
-		<column name="frequency">
+		<column name="frequency" class="EthanYehuda\CronjobManager\Ui\Component\Listing\Column\Frequency">
 			<argument name="data" xsi:type="array">
 				<item name="config" xsi:type="array">
 					<item name="filter" xsi:type="string">text</item>