vpietri/magento2-developer-quickdevbar

View on GitHub
Block/Tab/Content/Config.php

Summary

Maintainability
A
35 mins
Test Coverage

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.
Open

        \Magento\Framework\View\Element\Template\Context $context,
        \Magento\Framework\App\Config $appConfig,
        \ADM\QuickDevBar\Helper\Data $qdbHelper,
        \ADM\QuickDevBar\Helper\Register $qdbHelperRegister,
        array $data = []
Severity: Minor
Found in Block/Tab/Content/Config.php - About 35 mins to fix

    The property $_appConfig is not named in camelCase.
    Open

    class Config extends \ADM\QuickDevBar\Block\Tab\Panel
    {
        protected $_config_values;
    
        protected $_appConfig;
    Severity: Minor
    Found in Block/Tab/Content/Config.php by phpmd

    CamelCasePropertyName

    Since: 0.2

    It is considered best practice to use the camelCase notation to name attributes.

    Example

    class ClassName {
        protected $property_name;
    }

    Source

    The property $_config_values is not named in camelCase.
    Open

    class Config extends \ADM\QuickDevBar\Block\Tab\Panel
    {
        protected $_config_values;
    
        protected $_appConfig;
    Severity: Minor
    Found in Block/Tab/Content/Config.php by phpmd

    CamelCasePropertyName

    Since: 0.2

    It is considered best practice to use the camelCase notation to name attributes.

    Example

    class ClassName {
        protected $property_name;
    }

    Source

    The method _buildFlatConfig is not named in camelCase.
    Open

        protected function _buildFlatConfig($scope, $path = '')
        {
            $flatConfig = [];
            if (is_array($scope)) {
                foreach ($scope as $scopeKey => $scopeValue) {
    Severity: Minor
    Found in Block/Tab/Content/Config.php by phpmd

    CamelCaseMethodName

    Since: 0.2

    It is considered best practice to use the camelCase notation to name methods.

    Example

    class ClassName {
        public function get_name() {
        }
    }

    Source

    There are no issues that match your filters.

    Category
    Status