Codding styles improvements

ihorvansach · ihorvansach · commit 88772d5b6bad · 2025-07-02T19:53:39.000+03:00
diff --git a/Api/BreezeThemeDetectionInterface.php b/Api/BreezeThemeDetectionInterface.php
@@ -16,5 +16,4 @@ interface BreezeThemeDetectionInterface
      * @return bool
      */
     public function execute($storeId = null): bool;
-
 }
diff --git a/Api/GetParentProductIdsInterface.php b/Api/GetParentProductIdsInterface.php
@@ -1,23 +1,23 @@
-<?php
-/**
- * Copyright © Magefan (support@magefan.com). All rights reserved.
- * Please visit Magefan.com for license details (https://magefan.com/end-user-license-agreement).
- */
-
-namespace Magefan\Community\Api;
-
-/**
- * Return parent ids by child ids
- *
- * @api
- * @since 2.1.19
- */
-interface GetParentProductIdsInterface
-{
-    /**
-     * @api
-     * @param array $productIds
-     * @return array
-     */
-    public function execute(array $productIds) : array;
-}
+<?php
+/**
+ * Copyright © Magefan (support@magefan.com). All rights reserved.
+ * Please visit Magefan.com for license details (https://magefan.com/end-user-license-agreement).
+ */
+
+namespace Magefan\Community\Api;
+
+/**
+ * Return parent ids by child ids
+ *
+ * @api
+ * @since 2.1.19
+ */
+interface GetParentProductIdsInterface
+{
+    /**
+     * @api
+     * @param array $productIds
+     * @return array
+     */
+    public function execute(array $productIds) : array;
+}
diff --git a/Api/GetWebsitesMapInterface.php b/Api/GetWebsitesMapInterface.php
@@ -1,22 +1,22 @@
-<?php
-/**
- * Copyright © Magefan (support@magefan.com). All rights reserved.
- * Please visit Magefan.com for license details (https://magefan.com/end-user-license-agreement).
- */
-
-namespace Magefan\Community\Api;
-
-/**
- * Return Websites Map
- *
- * @api
- * @since 2.1.19
- */
-interface GetWebsitesMapInterface
-{
-    /**
-     * @api
-     * @return array
-     */
-    public function execute() : array;
-}
+<?php
+/**
+ * Copyright © Magefan (support@magefan.com). All rights reserved.
+ * Please visit Magefan.com for license details (https://magefan.com/end-user-license-agreement).
+ */
+
+namespace Magefan\Community\Api;
+
+/**
+ * Return Websites Map
+ *
+ * @api
+ * @since 2.1.19
+ */
+interface GetWebsitesMapInterface
+{
+    /**
+     * @api
+     * @return array
+     */
+    public function execute() : array;
+}
diff --git a/Api/HyvaThemeDetectionInterface.php b/Api/HyvaThemeDetectionInterface.php
@@ -16,5 +16,4 @@ interface HyvaThemeDetectionInterface
      * @return bool
      */
     public function execute($storeId = null): bool;
-
 }
diff --git a/Api/SecureHtmlRendererInterface.php b/Api/SecureHtmlRendererInterface.php
@@ -25,5 +25,4 @@ public function renderTag(
         ?string $content = null,
         bool $textContent = true
     );
-
 }
diff --git a/Block/Adminhtml/Edit/BackButton.php b/Block/Adminhtml/Edit/BackButton.php
@@ -8,9 +8,6 @@
 
 use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;
 
-/**
- * Class BackButton
- */
 class BackButton extends GenericButton implements ButtonProviderInterface
 {
     /**
diff --git a/Block/Adminhtml/Edit/DeleteButton.php b/Block/Adminhtml/Edit/DeleteButton.php
@@ -8,9 +8,6 @@
 
 use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;
 
-/**
- * Class DeleteButton
- */
 class DeleteButton extends GenericButton implements ButtonProviderInterface
 {
 
diff --git a/Block/Adminhtml/Edit/DuplicateButton.php b/Block/Adminhtml/Edit/DuplicateButton.php
@@ -8,9 +8,6 @@
 
 use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;
 
-/**
- * Class DuplicateButton
- */
 class DuplicateButton extends GenericButton implements ButtonProviderInterface
 {
     /**
diff --git a/Block/Adminhtml/Edit/GenericButton.php b/Block/Adminhtml/Edit/GenericButton.php
@@ -10,9 +10,6 @@
 use Magento\Framework\Exception\NoSuchEntityException;
 use Magento\Framework\AuthorizationInterface;
 
-/**
- * Class GenericButton
- */
 class GenericButton
 {
     /**
diff --git a/Block/Adminhtml/Edit/PreviewButton.php b/Block/Adminhtml/Edit/PreviewButton.php
@@ -8,9 +8,6 @@
 
 use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;
 
-/**
- * Class PreviewButton
- */
 class PreviewButton extends GenericButton implements ButtonProviderInterface
 {
     /**
diff --git a/Block/Adminhtml/Edit/ResetButton.php b/Block/Adminhtml/Edit/ResetButton.php
@@ -8,9 +8,6 @@
 
 use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;
 
-/**
- * Class ResetButton
- */
 class ResetButton implements ButtonProviderInterface
 {
     /**
diff --git a/Block/Adminhtml/Edit/SaveAndContinueButton.php b/Block/Adminhtml/Edit/SaveAndContinueButton.php
@@ -8,9 +8,6 @@
 
 use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;
 
-/**
- * Class SaveAndContinueButton
- */
 class SaveAndContinueButton extends GenericButton implements ButtonProviderInterface
 {
 
diff --git a/Block/Adminhtml/Edit/SaveButton.php b/Block/Adminhtml/Edit/SaveButton.php
@@ -8,9 +8,6 @@
 
 use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;
 
-/**
- * Class SaveButton
- */
 class SaveButton extends GenericButton implements ButtonProviderInterface
 {
     /**
diff --git a/Block/Adminhtml/HyvaThemeChecker.php b/Block/Adminhtml/HyvaThemeChecker.php
@@ -45,8 +45,8 @@ public function __construct(
     }
 
         /**
-     * @return array
-     */
+         * @return array
+         */
     public function getWitchModuleIsInstalled(): array
     {
         $moduleGroups = [
diff --git a/Block/Adminhtml/Linv.php b/Block/Adminhtml/Linv.php
@@ -51,8 +51,8 @@ public function getItems(): array
         $path = '/g'.'en'.'er'.'al'.'/l'.'in'.'v';
         $select = $connection->select()
             ->from([$table])
-            ->where( 'path LIKE ?', '%' . $path )
-            ->where('value = ?',1);
+            ->where('path LIKE ?', '%' . $path)
+            ->where('value = ?', 1);
         $items = $connection->fetchAll($select);
         $result = [];
 
@@ -81,8 +81,8 @@ public function getMessages():array
         $path = '/g'.'en'.'er'.'al'.'/l'.'in'.'v'.'_'.'error_me'.'ss'.'ag'.'e';
         $select = $connection->select()
             ->from([$table])
-            ->where( 'path LIKE ?', '%' . $path )
-            ->where('value != ?','');
+            ->where('path LIKE ?', '%' . $path)
+            ->where('value != ?', '');
         $items = $connection->fetchAll($select);
         $result = [];
 
diff --git a/Block/Adminhtml/System/Config/Form/ExtensionsInfo.php b/Block/Adminhtml/System/Config/Form/ExtensionsInfo.php
@@ -91,7 +91,7 @@ public function render(AbstractElement $element)
                 if ($listKey == 'up_to_date' && version_compare($this->getModuleVersion->execute($moduleName), $moduleInfo->getVersion()) < 0) {
                     continue;
                 }
-                if ($listKey == 'need_update' && version_compare($this->getModuleVersion->execute($moduleName),  $moduleInfo->getVersion()) >= 0) {
+                if ($listKey == 'need_update' && version_compare($this->getModuleVersion->execute($moduleName), $moduleInfo->getVersion()) >= 0) {
                     continue;
                 }
 
@@ -117,5 +117,4 @@ public function render(AbstractElement $element)
         }
         return $html;
     }
-
 }
diff --git a/Block/Adminhtml/System/Config/Form/Info.php b/Block/Adminhtml/System/Config/Form/Info.php
@@ -143,21 +143,21 @@ public function render(\Magento\Framework\Data\Form\Element\AbstractElement $ele
     
         <div class="col-actions">
             <div class="actions">';
-            if ($canUpgradeToMaxPlan) {
-                $html .= '<button id="upgrade" title="Upgrade Plan" class="action-upgrade" onclick="window.open(\'' . $this->escapeHtml($moduleUrl . '/pricing'  . $utmParam) . '&utm_campaign=upgrade-plan\', \'_blank\'); return false;"><span>Upgrade Plan</span></button>';
-            }
+        if ($canUpgradeToMaxPlan) {
+            $html .= '<button id="upgrade" title="Upgrade Plan" class="action-upgrade" onclick="window.open(\'' . $this->escapeHtml($moduleUrl . '/pricing'  . $utmParam) . '&utm_campaign=upgrade-plan\', \'_blank\'); return false;"><span>Upgrade Plan</span></button>';
+        }
 
-            if ($newVersionAvailable) {
-                $html .= '<button id="update" title="Upgrade to new Version" class="action-update" onclick="window.open(\'https://mage' . 'fan.com/downloadable/customer/products' . $utmParam . '&utm_campaign=update-to-new-version\', \'_blank\'); return false;"><span>Upgrade to new Version</span><span class="mf-icon"><svg xmlns="http://www.w3.org/2000/svg" width="20" height="20" viewBox="0 0 20 20" fill="none">
+        if ($newVersionAvailable) {
+            $html .= '<button id="update" title="Upgrade to new Version" class="action-update" onclick="window.open(\'https://mage' . 'fan.com/downloadable/customer/products' . $utmParam . '&utm_campaign=update-to-new-version\', \'_blank\'); return false;"><span>Upgrade to new Version</span><span class="mf-icon"><svg xmlns="http://www.w3.org/2000/svg" width="20" height="20" viewBox="0 0 20 20" fill="none">
 <path d="M15.5664 10.9766H3.92578C3.38633 10.9766 2.94922 10.5395 2.94922 10C2.94922 9.46055 3.38633 9.02344 3.92578 9.02344H15.5664C16.1059 9.02344 16.543 9.46055 16.543 10C16.543 10.5395 16.1059 10.9766 15.5664 10.9766Z" fill="white"/>
 <path d="M11.0942 15.5938C10.8321 15.5938 10.5703 15.4887 10.3778 15.2813C10.011 14.886 10.0344 14.268 10.4297 13.9012L14.6278 10.0078L10.3028 6.10392C9.90236 5.7426 9.87072 5.12502 10.2321 4.72463C10.5934 4.32424 11.211 4.2926 11.6113 4.65393L16.7285 9.27307C16.9321 9.45666 17.0488 9.71721 17.0508 9.99142C17.0528 10.2656 16.9395 10.5278 16.7383 10.7141L11.7578 15.3332C11.5699 15.5074 11.3317 15.5938 11.0942 15.5938Z" fill="white"/>
 </svg></span></button>';
-            }
+        }
             $html .= '</div>
            ';
-           if ($newVersionAvailable) {
-               $html .= '<div class="available-version">Version v' . $this->escapeHtml($latestVersion) . ' is available</div>';
-           }
+        if ($newVersionAvailable) {
+            $html .= '<div class="available-version">Version v' . $this->escapeHtml($latestVersion) . ' is available</div>';
+        }
          $html .= ' </div>
 </div>
         <style>
@@ -217,5 +217,4 @@ protected function getModuleTitle()
     {
         return ucwords(str_replace('_', ' ', $this->getModuleName())) . ' Extension';
     }
-
 }
diff --git a/Block/Adminhtml/System/Config/Form/ProductKeyField.php b/Block/Adminhtml/System/Config/Form/ProductKeyField.php
@@ -11,9 +11,6 @@
 use Magefan\Community\Model\Section;
 use Magento\Framework\App\ObjectManager;
 
-/**
- * Class Product Key Field
- */
 class ProductKeyField extends Field
 {
     /**
@@ -40,7 +37,7 @@ public function render(\Magento\Framework\Data\Form\Element\AbstractElement $ele
         } else {
             $config = ObjectManager::getInstance()->get(\Magefan\Community\Model\Config::class);
             $bp = $section->getName() . '/' . 'g' . 'e' . 'n' . 'e' . 'r' . 'a' . 'l' . '/' ;
-            if (!$config->getConfig( $bp . Section::ACTIVE) && !$section->getType()) {
+            if (!$config->getConfig($bp . Section::ACTIVE) && !$section->getType()) {
                     $url = 'ht' . 'tps'. ':' . '/'. '/'. 'ma' . 'g' . 'ef' . 'an' . '.' . 'c' . 'o' . 'm' . '/' . 'mp' . 'k/a' . 'cti' . 'vat' . 'e/e' . 'xte' . 'nsi' . 'on/' . 'ret' . 'urn_' . 'ur' . 'l/' .
                         base64_encode($this->getUrl('m' . 'f' . 'co' . 'mm' . 'uni' . 'ty/' . 'act' . 'iva' . 'te/ext' . 'ens' . 'ion', ['section' => $section->getName()]))
                         . '/mo' . 'dul' . 'e/' . $section->getModuleName() . '/se' . 'cti' . 'on/' . $section->getName();
diff --git a/Controller/Adminhtml/Activate/Extension.php b/Controller/Adminhtml/Activate/Extension.php
@@ -45,8 +45,7 @@ public function __construct(
         WriterInterface $configWriter,
         TypeListInterface $cacheTypeList,
         DateTime $date
-    )
-    {
+    ) {
         $this->configWriter = $configWriter;
         $this->cacheTypeList = $cacheTypeList;
         $this->date = $date;
diff --git a/Cron/Sections.php b/Cron/Sections.php
@@ -11,10 +11,6 @@
 use Magefan\Community\Model\SetLinvFlag;
 use Magento\Framework\App\ResourceConnection;
 
-/**
- * Class Sections
- * @package Magefan\Community
- */
 class Sections
 {
     /**
diff --git a/Model/AbstractThemeDetection.php b/Model/AbstractThemeDetection.php
@@ -53,7 +53,6 @@ public function __construct(
         StoreManagerInterface $storeManager,
         ScopeConfigInterface $scopeConfig,
         ThemeProviderInterface $themeProvider
-
     ) {
         $this->moduleManager = $moduleManager;
         $this->storeManager = $storeManager;
@@ -91,7 +90,7 @@ public function execute($storeId = null): bool
                 $storeId = $this->storeManager->getStore()->getId();
             }
 
-            if (!$storeId){
+            if (!$storeId) {
                 $stores = $this->storeManager->getStores();
                 foreach ($stores as $store) {
                     if ($this->isThemeInUse($store->getId())) {
@@ -137,4 +136,4 @@ private function isThemeInUse($storeId)
         }
         return false;
     }
-}
+}
diff --git a/Model/BreezeThemeDetection.php b/Model/BreezeThemeDetection.php
@@ -9,6 +9,7 @@
 namespace Magefan\Community\Model;
 
 use Magefan\Community\Api\BreezeThemeDetectionInterface;
+
 class BreezeThemeDetection extends AbstractThemeDetection implements BreezeThemeDetectionInterface
 {
     /**
diff --git a/Model/GetModuleInfo.php b/Model/GetModuleInfo.php
@@ -124,4 +124,4 @@ private function loadFromCache(): array
 
         return [];
     }
-}
+}
diff --git a/Model/GetParentProductIds.php b/Model/GetParentProductIds.php
diff --git a/Model/GetWebsitesMap.php b/Model/GetWebsitesMap.php
diff --git a/Model/HyvaThemeDetection.php b/Model/HyvaThemeDetection.php
diff --git a/Model/View/Helper/SecureHtmlRenderer.php b/Model/View/Helper/SecureHtmlRenderer.php
diff --git a/Observer/ConfigObserver.php b/Observer/ConfigObserver.php
diff --git a/Plugin/Magento/Framework/View/TemplateEngine/Php.php b/Plugin/Magento/Framework/View/TemplateEngine/Php.php
diff --git a/Setup/Patch/Data/UpdateWelcomeBlogPost.php b/Setup/Patch/Data/UpdateWelcomeBlogPost.php

Original file line number	Diff line number	Diff line change
`@@ -16,5 +16,4 @@ interface BreezeThemeDetectionInterface`
`16`	`16`	`* @return bool`
`17`	`17`	`*/`
`18`	`18`	`public function execute($storeId = null): bool;`
`19`		`-`
`20`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,5 +16,4 @@ interface HyvaThemeDetectionInterface`
`16`	`16`	`* @return bool`
`17`	`17`	`*/`
`18`	`18`	`public function execute($storeId = null): bool;`
`19`		`-`
`20`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -25,5 +25,4 @@ public function renderTag(`
`25`	`25`	`?string $content = null,`
`26`	`26`	`bool $textContent = true`
`27`	`27`	`);`
`28`		`-`
`29`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,9 +8,6 @@`
`8`	`8`
`9`	`9`	`use Magento\Framework\View\Element\UiComponent\Control\ButtonProviderInterface;`
`10`	`10`
`11`		`-/**`
`12`		`- * Class BackButton`
`13`		`- */`
`14`	`11`	`class BackButton extends GenericButton implements ButtonProviderInterface`
`15`	`12`	`{`
`16`	`13`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -10,9 +10,6 @@`
`10`	`10`	`use Magento\Framework\Exception\NoSuchEntityException;`
`11`	`11`	`use Magento\Framework\AuthorizationInterface;`
`12`	`12`
`13`		`-/**`
`14`		`- * Class GenericButton`
`15`		`- */`
`16`	`13`	`class GenericButton`
`17`	`14`	`{`
`18`	`15`	`/**`