diff --git a/app/code/Magento/Config/Block/System/Config/Edit.php b/app/code/Magento/Config/Block/System/Config/Edit.php index e292f81e7b13f..e7e56daea749f 100644 --- a/app/code/Magento/Config/Block/System/Config/Edit.php +++ b/app/code/Magento/Config/Block/System/Config/Edit.php @@ -6,8 +6,6 @@ /** * Config edit page - * - * @author Magento Core Team */ namespace Magento\Config\Block\System\Config; @@ -21,7 +19,7 @@ */ class Edit extends \Magento\Backend\Block\Widget { - const DEFAULT_SECTION_BLOCK = \Magento\Config\Block\System\Config\Form::class; + public const DEFAULT_SECTION_BLOCK = \Magento\Config\Block\System\Config\Form::class; /** * Form block class name @@ -120,6 +118,8 @@ public function getSaveUrl() } /** + * Return search parameters in JSON format + * * @return string * @since 101.1.0 */ diff --git a/app/code/Magento/Config/Block/System/Config/Form/Field/FieldArray/AbstractFieldArray.php b/app/code/Magento/Config/Block/System/Config/Form/Field/FieldArray/AbstractFieldArray.php index a8e0ecd71912d..f24013868de99 100644 --- a/app/code/Magento/Config/Block/System/Config/Form/Field/FieldArray/AbstractFieldArray.php +++ b/app/code/Magento/Config/Block/System/Config/Form/Field/FieldArray/AbstractFieldArray.php @@ -9,7 +9,8 @@ /** * Backend system config array field renderer * - * @author Magento Core Team + * phpcs:disable Magento2.CodeAnalysis.EmptyBlock.DetectedFunction + * phpcs:ignore Magento2.Classes.AbstractApi.AbstractApi * @api * @since 100.0.2 */ @@ -196,7 +197,7 @@ protected function _getCellInputElementName($columnName) public function renderCellTemplate($columnName) { if (empty($this->_columns[$columnName])) { - throw new \Exception('Wrong column name specified.'); + throw new \Exception('Wrong column name specified.'); // phpcs:ignore } $column = $this->_columns[$columnName]; $inputName = $this->_getCellInputElementName($columnName); @@ -255,7 +256,7 @@ protected function _toHtml() $this->_isPreparedToRender = true; } if (empty($this->_columns)) { - throw new \Exception('At least one column must be defined.'); + throw new \Exception('At least one column must be defined.'); // phpcs:ignore } return parent::_toHtml(); } @@ -281,6 +282,8 @@ public function getColumns() } /** + * Retrieve the label for the 'add' button + * * @return string * @since 101.0.0 */ diff --git a/app/code/Magento/Config/Block/System/Config/Form/Field/File.php b/app/code/Magento/Config/Block/System/Config/Form/Field/File.php index c1db2225af984..8a2e26dc0f02a 100755 --- a/app/code/Magento/Config/Block/System/Config/Form/Field/File.php +++ b/app/code/Magento/Config/Block/System/Config/Form/Field/File.php @@ -6,8 +6,6 @@ /** * File config field renderer - * - * @author Magento Core Team */ declare(strict_types=1); diff --git a/app/code/Magento/Config/Block/System/Config/Form/Field/Heading.php b/app/code/Magento/Config/Block/System/Config/Form/Field/Heading.php index 1309b8e4e1831..021ffec466d40 100644 --- a/app/code/Magento/Config/Block/System/Config/Form/Field/Heading.php +++ b/app/code/Magento/Config/Block/System/Config/Form/Field/Heading.php @@ -6,8 +6,6 @@ /** * Renderer for sub-heading in fieldset - * - * @author Magento Core Team */ namespace Magento\Config\Block\System\Config\Form\Field; diff --git a/app/code/Magento/Config/Block/System/Config/Form/Field/Select/Allowspecific.php b/app/code/Magento/Config/Block/System/Config/Form/Field/Select/Allowspecific.php index 5b442e0f583f6..549a5888233ed 100644 --- a/app/code/Magento/Config/Block/System/Config/Form/Field/Select/Allowspecific.php +++ b/app/code/Magento/Config/Block/System/Config/Form/Field/Select/Allowspecific.php @@ -6,8 +6,6 @@ /** * System configuration shipping methods allow all countries select - * - * @author Magento Core Team */ namespace Magento\Config\Block\System\Config\Form\Field\Select; diff --git a/app/code/Magento/Config/Block/System/Config/Tabs.php b/app/code/Magento/Config/Block/System/Config/Tabs.php index 1427552785960..3671d57091c52 100644 --- a/app/code/Magento/Config/Block/System/Config/Tabs.php +++ b/app/code/Magento/Config/Block/System/Config/Tabs.php @@ -8,8 +8,6 @@ * System configuration tabs block * * @method setTitle(string $title) - * - * @author Magento Core Team */ namespace Magento\Config\Block\System\Config; @@ -21,8 +19,6 @@ class Tabs extends \Magento\Backend\Block\Widget { /** - * Tabs - * * @var \Magento\Config\Model\Config\Structure\Element\Iterator */ protected $_tabs; diff --git a/app/code/Magento/Config/Controller/Adminhtml/System/Config/Save.php b/app/code/Magento/Config/Controller/Adminhtml/System/Config/Save.php index dd75a9078ad0e..c21e20dc1e70f 100644 --- a/app/code/Magento/Config/Controller/Adminhtml/System/Config/Save.php +++ b/app/code/Magento/Config/Controller/Adminhtml/System/Config/Save.php @@ -13,7 +13,6 @@ /** * System Configuration Save Controller * - * @author Magento Core Team * @SuppressWarnings(PHPMD.CouplingBetweenObjects) */ class Save extends AbstractConfig implements HttpPostActionInterface diff --git a/app/code/Magento/Config/Model/Config.php b/app/code/Magento/Config/Model/Config.php index 6b3cdd05c54c4..d32ff37420337 100644 --- a/app/code/Magento/Config/Model/Config.php +++ b/app/code/Magento/Config/Model/Config.php @@ -19,7 +19,6 @@ * * Used to save configuration * - * @author Magento Core Team * @SuppressWarnings(PHPMD.CouplingBetweenObjects) * @api * @since 100.0.2 diff --git a/app/code/Magento/Config/Model/Config/Backend/Admin/Custom.php b/app/code/Magento/Config/Model/Config/Backend/Admin/Custom.php index 7da29f487212e..61827a50a533c 100644 --- a/app/code/Magento/Config/Model/Config/Backend/Admin/Custom.php +++ b/app/code/Magento/Config/Model/Config/Backend/Admin/Custom.php @@ -6,8 +6,6 @@ /** * Config backend model for "Custom Admin URL" option - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Backend\Admin; diff --git a/app/code/Magento/Config/Model/Config/Backend/Admin/Custompath.php b/app/code/Magento/Config/Model/Config/Backend/Admin/Custompath.php index c21b131e80ed5..7164db0a5acdb 100644 --- a/app/code/Magento/Config/Model/Config/Backend/Admin/Custompath.php +++ b/app/code/Magento/Config/Model/Config/Backend/Admin/Custompath.php @@ -6,8 +6,6 @@ /** * Config backend model for "Custom Admin Path" option - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Backend\Admin; diff --git a/app/code/Magento/Config/Model/Config/Backend/Admin/Password/Link/Expirationperiod.php b/app/code/Magento/Config/Model/Config/Backend/Admin/Password/Link/Expirationperiod.php index de19dac52947c..b550797d9ff9a 100644 --- a/app/code/Magento/Config/Model/Config/Backend/Admin/Password/Link/Expirationperiod.php +++ b/app/code/Magento/Config/Model/Config/Backend/Admin/Password/Link/Expirationperiod.php @@ -6,8 +6,6 @@ /** * Admin Reset Password Link Expiration period backend model - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Backend\Admin\Password\Link; diff --git a/app/code/Magento/Config/Model/Config/Backend/Currency/AbstractCurrency.php b/app/code/Magento/Config/Model/Config/Backend/Currency/AbstractCurrency.php index e51c433918f67..8442f8c53900d 100644 --- a/app/code/Magento/Config/Model/Config/Backend/Currency/AbstractCurrency.php +++ b/app/code/Magento/Config/Model/Config/Backend/Currency/AbstractCurrency.php @@ -8,14 +8,13 @@ * \Directory currency abstract backend model * * Allows dispatching before and after events for each controller action - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Backend\Currency; /** * Base currency class * + * phpcs:ignore Magento2.Classes.AbstractApi.AbstractApi * @api * @since 100.0.2 */ diff --git a/app/code/Magento/Config/Model/Config/Backend/Datashare.php b/app/code/Magento/Config/Model/Config/Backend/Datashare.php index 4eb832be0deb1..dfde4d2b94efe 100644 --- a/app/code/Magento/Config/Model/Config/Backend/Datashare.php +++ b/app/code/Magento/Config/Model/Config/Backend/Datashare.php @@ -6,8 +6,6 @@ /** * Config category field backend - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Backend; @@ -18,6 +16,8 @@ class Datashare extends \Magento\Framework\App\Config\Value { /** + * Do nothing after save + * * @return $this */ public function afterSave() diff --git a/app/code/Magento/Config/Model/Config/Backend/Email/Logo.php b/app/code/Magento/Config/Model/Config/Backend/Email/Logo.php index 6fec9141d9de2..8a5f6bf69cae0 100644 --- a/app/code/Magento/Config/Model/Config/Backend/Email/Logo.php +++ b/app/code/Magento/Config/Model/Config/Backend/Email/Logo.php @@ -6,20 +6,19 @@ /** * Backend model for uploading transactional emails custom logo image - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Backend\Email; /** * @deprecated 100.1.5 + * @see nothing */ class Logo extends \Magento\Config\Model\Config\Backend\Image { /** * The tail part of directory path for uploading */ - const UPLOAD_DIR = 'email/logo'; + public const UPLOAD_DIR = 'email/logo'; /** * Upload max file size in kilobytes @@ -49,12 +48,16 @@ protected function _addWhetherScopeInfo() } /** + * Return temporary name of file + * * @return string|null */ protected function getTmpFileName() { $tmpName = null; + // @phpcs:ignore Magento2.Security.Superglobal.SuperglobalUsageError if (isset($_FILES['groups'])) { + // @phpcs:ignore Magento2.Security.Superglobal.SuperglobalUsageError $tmpName = $_FILES['groups']['tmp_name'][$this->getGroupId()]['fields'][$this->getField()]['value']; } else { $tmpName = is_array($this->getValue()) ? $this->getValue()['tmp_name'] : null; diff --git a/app/code/Magento/Config/Model/Config/Backend/Image/Adapter.php b/app/code/Magento/Config/Model/Config/Backend/Image/Adapter.php index 28b0b6275bb4c..e34b99900d73d 100644 --- a/app/code/Magento/Config/Model/Config/Backend/Image/Adapter.php +++ b/app/code/Magento/Config/Model/Config/Backend/Image/Adapter.php @@ -6,8 +6,6 @@ /** * System config image field backend model for Zend PDF generator - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Backend\Image; diff --git a/app/code/Magento/Config/Model/Config/Source/Email/Method.php b/app/code/Magento/Config/Model/Config/Source/Email/Method.php index 36fb9f2b93592..e53237c30d3ea 100644 --- a/app/code/Magento/Config/Model/Config/Source/Email/Method.php +++ b/app/code/Magento/Config/Model/Config/Source/Email/Method.php @@ -4,20 +4,19 @@ * All Rights Reserved. */ -/** - * Source for email send method - * - * @author Magento Core Team - */ namespace Magento\Config\Model\Config\Source\Email; /** + * Source for email send method + * * @api * @since 100.0.2 */ class Method implements \Magento\Framework\Option\ArrayInterface { /** + * Return option array + * * @return array */ public function toOptionArray() diff --git a/app/code/Magento/Config/Model/Config/Source/Reports/Scope.php b/app/code/Magento/Config/Model/Config/Source/Reports/Scope.php index 057a4dd9f5ec2..7cada8656fc45 100644 --- a/app/code/Magento/Config/Model/Config/Source/Reports/Scope.php +++ b/app/code/Magento/Config/Model/Config/Source/Reports/Scope.php @@ -6,8 +6,6 @@ /** * Config source reports event store filter - * - * @author Magento Core Team */ namespace Magento\Config\Model\Config\Source\Reports; diff --git a/app/code/Magento/Config/Model/ResourceModel/Config.php b/app/code/Magento/Config/Model/ResourceModel/Config.php index de4a54dfc6f75..0c10da244dd82 100644 --- a/app/code/Magento/Config/Model/ResourceModel/Config.php +++ b/app/code/Magento/Config/Model/ResourceModel/Config.php @@ -11,7 +11,6 @@ /** * Core Resource Resource Model * - * @author Magento Core Team * @api * @since 100.0.2 */ diff --git a/app/code/Magento/Config/Model/ResourceModel/Config/Data.php b/app/code/Magento/Config/Model/ResourceModel/Config/Data.php index 3df1976b7090f..8dc8f787e7c57 100644 --- a/app/code/Magento/Config/Model/ResourceModel/Config/Data.php +++ b/app/code/Magento/Config/Model/ResourceModel/Config/Data.php @@ -8,7 +8,6 @@ /** * Core config data resource model * - * @author Magento Core Team * @api * @since 100.0.2 */ @@ -44,6 +43,7 @@ protected function _beforeSave(\Magento\Framework\Model\AbstractModel $object) /** * Validate unique configuration data before save + * * Set id to object if exists configuration instead of throw exception * * @param \Magento\Framework\Model\AbstractModel $object diff --git a/app/code/Magento/Config/Model/ResourceModel/Config/Data/Collection.php b/app/code/Magento/Config/Model/ResourceModel/Config/Data/Collection.php index 7b73570932424..f8c3af22140a8 100644 --- a/app/code/Magento/Config/Model/ResourceModel/Config/Data/Collection.php +++ b/app/code/Magento/Config/Model/ResourceModel/Config/Data/Collection.php @@ -8,7 +8,6 @@ /** * Config data collection * - * @author Magento Core Team * @api * @since 100.0.2 */ diff --git a/app/code/Magento/ConfigurableProduct/Block/Adminhtml/Product/Attribute/Edit/Tab/Variations/Main.php b/app/code/Magento/ConfigurableProduct/Block/Adminhtml/Product/Attribute/Edit/Tab/Variations/Main.php index 961dfc20dd00c..9d50a0ba65d96 100644 --- a/app/code/Magento/ConfigurableProduct/Block/Adminhtml/Product/Attribute/Edit/Tab/Variations/Main.php +++ b/app/code/Magento/ConfigurableProduct/Block/Adminhtml/Product/Attribute/Edit/Tab/Variations/Main.php @@ -6,8 +6,6 @@ /** * Product attribute add form variations main tab - * - * @author Magento Core Team */ namespace Magento\ConfigurableProduct\Block\Adminhtml\Product\Attribute\Edit\Tab\Variations; diff --git a/app/code/Magento/ConfigurableProduct/Model/ResourceModel/Indexer/Stock/Configurable.php b/app/code/Magento/ConfigurableProduct/Model/ResourceModel/Indexer/Stock/Configurable.php index a7d2bdf02bc81..a5b29711fb968 100644 --- a/app/code/Magento/ConfigurableProduct/Model/ResourceModel/Indexer/Stock/Configurable.php +++ b/app/code/Magento/ConfigurableProduct/Model/ResourceModel/Indexer/Stock/Configurable.php @@ -7,8 +7,6 @@ /** * CatalogInventory Configurable Products Stock Status Indexer Resource Model - * - * @author Magento Core Team */ use Magento\Catalog\Model\Product\Attribute\Source\Status as ProductStatus; use Magento\CatalogInventory\Model\Indexer\Stock\Action\Full; @@ -32,7 +30,7 @@ class Configurable extends \Magento\CatalogInventory\Model\ResourceModel\Indexer * @param \Magento\Framework\Indexer\Table\StrategyInterface $tableStrategy * @param \Magento\Eav\Model\Config $eavConfig * @param \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig - * @param null $connectionName + * @param string $connectionName * @param \Magento\Catalog\Model\ResourceModel\Indexer\ActiveTableSwitcher|null $activeTableSwitcher */ public function __construct( diff --git a/app/code/Magento/Cron/Model/Config/Backend/Product/Alert.php b/app/code/Magento/Cron/Model/Config/Backend/Product/Alert.php index 4db261b386cbf..5e340bf4eebe7 100644 --- a/app/code/Magento/Cron/Model/Config/Backend/Product/Alert.php +++ b/app/code/Magento/Cron/Model/Config/Backend/Product/Alert.php @@ -6,8 +6,6 @@ /** * Backend Model for product alerts - * - * @author Magento Core Team */ namespace Magento\Cron\Model\Config\Backend\Product; @@ -21,12 +19,12 @@ class Alert extends \Magento\Framework\App\Config\Value /** * Cron string path for product alerts */ - const CRON_STRING_PATH = 'crontab/default/jobs/catalog_product_alert/schedule/cron_expr'; + public const CRON_STRING_PATH = 'crontab/default/jobs/catalog_product_alert/schedule/cron_expr'; /** * Cron model path for product alerts */ - const CRON_MODEL_PATH = 'crontab/default/jobs/catalog_product_alert/run/model'; + public const CRON_MODEL_PATH = 'crontab/default/jobs/catalog_product_alert/run/model'; /** * @var \Magento\Framework\App\Config\ValueFactory diff --git a/app/code/Magento/Cron/Model/Config/Backend/Sitemap.php b/app/code/Magento/Cron/Model/Config/Backend/Sitemap.php index c5533ea50c4cd..37334049dea4a 100644 --- a/app/code/Magento/Cron/Model/Config/Backend/Sitemap.php +++ b/app/code/Magento/Cron/Model/Config/Backend/Sitemap.php @@ -6,8 +6,6 @@ /** * Backend Model for Currency import options - * - * @author Magento Core Team */ namespace Magento\Cron\Model\Config\Backend; diff --git a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency.php b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency.php index f704ddf9e456e..f149da02ee1b2 100644 --- a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency.php +++ b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency.php @@ -6,8 +6,6 @@ /** * Manage currency block - * - * @author Magento Core Team */ namespace Magento\CurrencySymbol\Block\Adminhtml\System; diff --git a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Matrix.php b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Matrix.php index 69dd81e5d0e09..0d775fbdc4e9f 100644 --- a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Matrix.php +++ b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Matrix.php @@ -6,8 +6,6 @@ /** * Manage currency block - * - * @author Magento Core Team */ namespace Magento\CurrencySymbol\Block\Adminhtml\System\Currency\Rate; diff --git a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Services.php b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Services.php index 90a4570f7757d..3b76c331de6d5 100644 --- a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Services.php +++ b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currency/Rate/Services.php @@ -6,8 +6,6 @@ /** * Manage currency import services block - * - * @author Magento Core Team */ namespace Magento\CurrencySymbol\Block\Adminhtml\System\Currency\Rate; diff --git a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currencysymbol.php b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currencysymbol.php index b0f849a7273a0..39faebe6b9554 100644 --- a/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currencysymbol.php +++ b/app/code/Magento/CurrencySymbol/Block/Adminhtml/System/Currencysymbol.php @@ -6,8 +6,6 @@ /** * Manage currency symbols block - * - * @author Magento Core Team */ namespace Magento\CurrencySymbol\Block\Adminhtml\System; @@ -22,11 +20,6 @@ class Currencysymbol extends \Magento\Backend\Block\Widget\Form */ protected $_symbolSystemFactory; - /** - * @var string - */ - private $_controller; - /** * @param \Magento\Backend\Block\Template\Context $context * @param \Magento\CurrencySymbol\Model\System\CurrencysymbolFactory $symbolSystemFactory diff --git a/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currency.php b/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currency.php index 3e967f1637d69..62bf641fad5a6 100644 --- a/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currency.php +++ b/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currency.php @@ -6,12 +6,11 @@ /** * Currency controller - * - * @author Magento Core Team */ namespace Magento\CurrencySymbol\Controller\Adminhtml\System; /** + * @phpcs:ignore Magento2.Classes.AbstractApi.AbstractApi * @api * @since 100.0.2 */ @@ -22,11 +21,9 @@ abstract class Currency extends \Magento\Backend\App\Action * * @see _isAllowed() */ - const ADMIN_RESOURCE = 'Magento_CurrencySymbol::currency_rates'; + public const ADMIN_RESOURCE = 'Magento_CurrencySymbol::currency_rates'; /** - * Core registry - * * @var \Magento\Framework\Registry */ protected $_coreRegistry = null; diff --git a/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currencysymbol.php b/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currencysymbol.php index c79abe8ded7cf..d7229773f0168 100644 --- a/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currencysymbol.php +++ b/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currencysymbol.php @@ -4,14 +4,12 @@ * All Rights Reserved. */ -/** - * Adminhtml Currency Symbols Controller - * - * @author Magento Core Team - */ namespace Magento\CurrencySymbol\Controller\Adminhtml\System; /** + * Adminhtml Currency Symbols Controller + * + * @phpcs:ignore Magento2.Classes.AbstractApi.AbstractApi * @api * @since 100.0.2 */ @@ -22,5 +20,5 @@ abstract class Currencysymbol extends \Magento\Backend\App\Action * * @see _isAllowed() */ - const ADMIN_RESOURCE = 'Magento_CurrencySymbol::symbols'; + public const ADMIN_RESOURCE = 'Magento_CurrencySymbol::symbols'; }