Set minimium PHP version to 8.1

The parent module only supports PHP 8.1+ too, so there is no point in supporting PHP 7.4 any more.

Author: avstudnitz

Block/Form/AsyncEvent/GenericButton.php

@@ -4,26 +4,27 @@
 namespace MageOS\AsyncEventsAdminUi\Block\Form\AsyncEvent;
 
 use Magento\Backend\Block\Widget\Context;
+use Magento\Framework\App\RequestInterface;
 use Magento\Framework\UrlInterface;
 
 /**
  * Generic (form) button for Asynchronous Event Subscriber entity.
  */
 class GenericButton
 {
-    private Context $context;
     private UrlInterface $urlBuilder;
+    private RequestInterface $request;
 
     public function __construct(
         Context $context
     ) {
-        $this->context = $context;
         $this->urlBuilder = $context->getUrlBuilder();
+        $this->request = $context->getRequest();
     }
 
     public function getSubscriptionId(): int
     {
-        return (int)$this->context->getRequest()->getParam('subscription_id');
+        return (int)$this->request->getParam('subscription_id');
     }
 
     /**

Command/AsyncEvent/SaveCommand.php

@@ -15,18 +15,11 @@
  */
 class SaveCommand
 {
-    private LoggerInterface $logger;
-    private AsyncEventResource $resource;
-    private EncryptorInterface $encryptor;
-
     public function __construct(
-        LoggerInterface        $logger,
-        AsyncEventResource     $resource,
-        EncryptorInterface     $encryptor
+        private readonly LoggerInterface    $logger,
+        private readonly AsyncEventResource $resource,
+        private readonly EncryptorInterface $encryptor
     ) {
-        $this->logger = $logger;
-        $this->resource = $resource;
-        $this->encryptor = $encryptor;
     }
 
     /**

Model/Config.php

@@ -7,12 +7,9 @@
 
 class Config
 {
-    private DataInterface $dataStorage;
-
     public function __construct(
-        DataInterface $dataStorage
+        private readonly DataInterface $dataStorage
     ) {
-        $this->dataStorage = $dataStorage;
     }
 
     /**

Query/AsyncEvent/GetListQuery.php

@@ -16,21 +16,12 @@
  */
 class GetListQuery
 {
-    private CollectionProcessorInterface $collectionProcessor;
-    private AsyncEventCollectionFactory $entityCollectionFactory;
-    private SearchCriteriaBuilder $searchCriteriaBuilder;
-    private SearchResultsInterfaceFactory $searchResultFactory;
-
     public function __construct(
-        CollectionProcessorInterface  $collectionProcessor,
-        AsyncEventCollectionFactory   $entityCollectionFactory,
-        SearchCriteriaBuilder         $searchCriteriaBuilder,
-        SearchResultsInterfaceFactory $searchResultFactory
+        private readonly CollectionProcessorInterface  $collectionProcessor,
+        private readonly AsyncEventCollectionFactory   $entityCollectionFactory,
+        private readonly SearchCriteriaBuilder         $searchCriteriaBuilder,
+        private readonly SearchResultsInterfaceFactory $searchResultFactory
     ) {
-        $this->collectionProcessor = $collectionProcessor;
-        $this->entityCollectionFactory = $entityCollectionFactory;
-        $this->searchCriteriaBuilder = $searchCriteriaBuilder;
-        $this->searchResultFactory = $searchResultFactory;
     }
 
     /**

Ui/Component/Listing/Column/AsyncEventBlockActions.php

@@ -23,12 +23,10 @@ class AsyncEventBlockActions extends Column
      */
     private const EDIT_URL_PATH = 'async_events/events/edit';
 
-    private UrlInterface $urlBuilder;
-
     public function __construct(
         ContextInterface   $context,
         UiComponentFactory $uiComponentFactory,
-        UrlInterface       $urlBuilder,
+        private readonly UrlInterface $urlBuilder,
         array              $components = [],
         array              $data = []
     ) {
@@ -38,7 +36,6 @@ public function __construct(
             $components,
             $data
         );
-        $this->urlBuilder = $urlBuilder;
     }
 
     public function prepareDataSource(array $dataSource): array

Ui/DataProvider/AsyncEventDataProvider.php

@@ -17,8 +17,6 @@
  */
 class AsyncEventDataProvider extends DataProvider
 {
-    private GetListQuery $getListQuery;
-    private SearchResultFactory $searchResultFactory;
     private array $loadedData = [];
 
     public function __construct(
@@ -29,8 +27,8 @@ public function __construct(
         SearchCriteriaBuilder $searchCriteriaBuilder,
         RequestInterface $request,
         FilterBuilder $filterBuilder,
-        GetListQuery $getListQuery,
-        SearchResultFactory $searchResultFactory,
+        private readonly GetListQuery $getListQuery,
+        private readonly SearchResultFactory $searchResultFactory,
         array $meta = [],
         array $data = []
     ) {
@@ -45,8 +43,6 @@ public function __construct(
             $meta,
             $data
         );
-        $this->getListQuery = $getListQuery;
-        $this->searchResultFactory = $searchResultFactory;
     }
 
     /**

Ui/Source/EventNames.php

@@ -7,12 +7,9 @@
 
 class EventNames implements \Magento\Framework\Option\ArrayInterface
 {
-    private Config $config;
-
     public function __construct(
-        Config $config
+        private readonly Config $config
     ) {
-        $this->config = $config;
     }
 
     public function toOptionArray()

Ui/Source/Stores.php

@@ -8,12 +8,9 @@
 
 class Stores implements \Magento\Framework\Option\ArrayInterface
 {
-    private StoreManagerInterface $storeManager;
-
     public function __construct(
-        StoreManagerInterface $storeManager
+        private readonly StoreManagerInterface $storeManager
     ) {
-        $this->storeManager = $storeManager;
     }
 
     public function toOptionArray()

composer.json

@@ -3,7 +3,7 @@
   "description": "Add event subscriptions for the Mage-OS Async Events module via an admin form instead of using REST requests.",
   "type": "magento2-module",
   "require": {
-    "php":  ">=7.4",
+    "php":  ">=8.1",
     "magento/framework": "*",
     "mage-os/mageos-async-events": "*"
   },