Add support for 2.3.x

Author: srenon

Controller/Adminhtml/Indexer/ReindexOnTheFly.php

@@ -6,22 +6,26 @@
  */
 namespace MagePal\Reindex\Controller\Adminhtml\Indexer;
 
+use Exception;
 use Magento\Backend\App\Action\Context;
+use Magento\Framework\Indexer\IndexerInterface;
+use Magento\Indexer\Model\IndexerFactory;
+use MagePal\Reindex\Controller\Adminhtml\Indexer;
 
-class ReindexOnTheFly extends \MagePal\Reindex\Controller\Adminhtml\Indexer
+class ReindexOnTheFly extends Indexer
 {
 
-    /** @var \Magento\Framework\Indexer\IndexerInterface  */
+    /** @var IndexerInterface  */
     protected $indexerFactory;
 
     /**
      * Index constructor.
      * @param Context $context
-     * @param \Magento\Indexer\Model\IndexerFactory $indexerFactory
+     * @param IndexerFactory $indexerFactory
      */
     public function __construct(
         Context $context,
-        \Magento\Indexer\Model\IndexerFactory $indexerFactory
+        IndexerFactory $indexerFactory
     ) {
         $this->indexerFactory = $indexerFactory;
         parent::__construct($context);
@@ -34,21 +38,19 @@ public function execute()
     {
         $indexerIds = $this->getRequest()->getParam('indexer_ids');
         if (!is_array($indexerIds)) {
-            $this->messageManager->addError(__('Please select indexers.'));
+            $this->messageManager->addErrorMessage(__('Please select indexers.'));
         } else {
             try {
                 foreach ($indexerIds as $indexerId) {
                     $indexer = $this->indexerFactory->create();
                     $indexer->load($indexerId)->reindexAll();
                 }
 
-                $this->messageManager->addSuccess(
+                $this->messageManager->addSuccessMessage(
                     __('Reindex %1 indexer(s).', count($indexerIds))
                 );
-            } catch (\Magento\Framework\Exception\LocalizedException $e) {
-                $this->messageManager->addError($e->getMessage());
-            } catch (\Exception $e) {
-                $this->messageManager->addException(
+            } catch (Exception $e) {
+                $this->messageManager->addExceptionMessage(
                     $e,
                     __("We couldn't reindex because of an error.")
                 );

composer.json

@@ -10,7 +10,7 @@
     ],
     "require": {
         "php": "~5.6.0|7.0.2|7.0.4|~7.0.6|~7.1.0|~7.1.3|~7.2.0",
-        "magento/module-backend": "100.0.*|100.1.*|100.2.*|101.0.0",
+        "magento/module-backend": "100.0.*|100.1.*|100.2.*|101.0.*",
         "magento/framework": "100.0.*|100.1.*|101.0.*|102.0.*"
     },
     "type": "magento2-module",

etc/adminhtml/system.xml

@@ -18,7 +18,7 @@
 			<group id="about" showInDefault="1" showInStore="1" showInWebsite="1" sortOrder="0" translate="label">
 				<attribute type="expanded">1</attribute>
 				<label>About MagePal Reindex</label>
-				<comment><![CDATA[ <strong>Copyright © 2018 <a href="http://bit.ly/MPEXRI" target="_blank">www.magepal.com</a> / <a href="mailto:support@magepal.com">support@magepal.com</a></strong><br />
+				<comment><![CDATA[ <strong>Copyright © 2019 <a href="http://bit.ly/MPEXRI" target="_blank">www.magepal.com</a> / <a href="mailto:support@magepal.com">support@magepal.com</a></strong><br />
                     Discover other must have extensions at <a href="http://bit.ly/MPEXRI" target="_blank">www.magepal.com</a>. Thanks for choosing MagePal Extensions.<br/><br/>
                     <hr style="border-top: 1px solid #e3e3e3" /> ]]></comment>
 				<field id="module_version" translate="label" type="label" sortOrder="0" showInDefault="1" showInWebsite="0" showInStore="0">