Add configuration for reservations cron

Author: mstrzyzewski

Cron/CleanReservations.php

@@ -13,7 +13,7 @@
  */
 class CleanReservations
 {
-    const RESERVATIONS_CONFIG_KEY = 'allegro/order/reservations_enabled';
+    const RESERVATIONS_CRON_CONFIG_KEY = 'allegro/order/reservations_cron_enabled';
 
     /** @var Logger */
     private $logger;
@@ -41,7 +41,7 @@ public function __construct(
 
     public function execute()
     {
-        if ($this->scopeConfig->getValue(self::RESERVATIONS_CONFIG_KEY)) {
+        if ($this->scopeConfig->getValue(self::RESERVATIONS_CRON_CONFIG_KEY)) {
             $this->logger->addInfo("Cronjob clean reservations is executed.");
             $this->allegroReservation->cleanOldReservations();
         }

Model/Configuration.php

@@ -16,6 +16,7 @@ class Configuration
     const DESCRIPTION_ATTRIBUTE_CONFIG_PATH = 'allegro/offer_create/description_attribute';
     const STORE_ID_CONFIG_PATH = 'allegro/order/store';
     const RESERVATIONS_ENABLED_CONFIG_PATH = 'allegro/order/reservations_enabled';
+    const RESERVATIONS_CRON_ENABLED_CONFIG_PATH = 'allegro/order/reservations_cron_enabled';
     const LAST_EVENT_ID_FLAG_NAME = 'allegro_order_last_event_id';
     const LAST_USER_ID_FLAG_NAME = 'allegro_credentials_last_user_id';
     const INITIALIZATION_TIME_FLAG_NAME = 'allegro_initialization_time';
@@ -87,6 +88,18 @@ public function areReservationsEnabled(
         return $this->scopeConfig->isSetFlag(self::RESERVATIONS_ENABLED_CONFIG_PATH, $scopeType, $scopeCode);
     }
 
+    /**
+     * @param string $scopeType
+     * @param string|null $scopeCode
+     * @return bool
+     */
+    public function isReservationsCronEnabled(
+        string $scopeType = ScopeConfigInterface::SCOPE_TYPE_DEFAULT,
+        ?string $scopeCode = null
+    ): bool {
+        return $this->scopeConfig->isSetFlag(self::RESERVATIONS_CRON_ENABLED_CONFIG_PATH, $scopeType, $scopeCode);
+    }
+
     /**
      * @return string|null
      */

Model/OrderImporter/AllegroReservation.php

@@ -173,7 +173,7 @@ public function cleanOldReservations()
     {
         $tenDaysAgo = (new \DateTime())->modify('-10 day');
         $searchCriteria = $this->searchCriteriaBuilder
-            ->addFilter('created_at', $tenDaysAgo->format('Y-m-d 12:00:00'), 'lteq')
+            ->addFilter('created_at', $tenDaysAgo, 'lteq')
             ->create();
 
         $reservations = $this->reservationRepository->getList($searchCriteria);

etc/adminhtml/system.xml

@@ -72,6 +72,14 @@
                     <label>Reservations enabled</label>
                     <source_model>Magento\Config\Model\Config\Source\Yesno</source_model>
                 </field>
+                <field id="reservations_cron_enabled" translate="label" type="select" sortOrder="30" showInDefault="1" showInWebsite="1" showInStore="1">
+                    <depends>
+                        <field id="reservations_enabled">1</field>
+                    </depends>
+                    <label>Cron for cleaning old reservations enabled</label>
+                    <comment>Cron deletes reservations older than 10 days</comment>
+                    <source_model>Magento\Config\Model\Config\Source\Yesno</source_model>
+                </field>
                 <field id="store" translate="label" type="select" sortOrder="40" showInDefault="1" showInWebsite="1" showInStore="1">
                     <label>Store to create new orders</label>
                     <source_model>Magento\Store\Model\System\Store</source_model>

etc/config.xml

@@ -15,6 +15,7 @@
                 <stock_synchronization_enabled>1</stock_synchronization_enabled>
                 <tracking_number_sending_enabled>1</tracking_number_sending_enabled>
                 <reservations_enabled>1</reservations_enabled>
+                <reservations_cron_enabled>0</reservations_cron_enabled>
                 <overpayment_status>Pending:allegro_overpayment</overpayment_status>
                 <underpayment_status>Pending:allegro_underpayment</underpayment_status>
             </order>

etc/db_schema.xml

@@ -71,7 +71,7 @@
         <column xsi:type="int" name="reservation_id" padding="10" unsigned="true" nullable="false" identity="false" comment="Reservation ID"/>
         <column xsi:type="varchar" name="checkout_form_id" nullable="false" comment="Checkout form ID"/>
         <column xsi:type="varchar" name="sku" nullable="false" comment="Product SKU"/>
-        <column xsi:type="datetime" name="created_at" nullable="false" comment="Creation date"/>
+        <column xsi:type="datetime" name="created_at" on_update="false" nullable="false" default="CURRENT_TIMESTAMP" comment="Creation date"/>
         <constraint xsi:type="foreign" referenceId="FOREIGN" table="allegro_reservations" column="reservation_id" referenceTable="inventory_reservation" referenceColumn="reservation_id" onDelete="CASCADE"/>
         <constraint xsi:type="primary" referenceId="PRIMARY">
             <column name="entity_id"/>

i18n/pl_PL.csv

@@ -172,3 +172,5 @@
 "Reservation with ID: %1 has been successfully deleted","Rezerwacja o ID: %1 została pomyślnie usunięta"
 "Something went wrong while trying to delete reservation with ID: %1","Coś poszło nie tak podczas próby usunięcia rezerwacji o ID: %1"
 "Creation date","Data utworzenia"
+"Cron for cleaning old reservations enabled","Cron do czyszczenia starych rezerwacji jest włączony"
+"Cron deletes reservations older than 10 days","Cron usuwa rezerwacje starsze niż 10 dni"