Sign upLogin

vpietri/magento2-developer-quickdevbar

View on GitHub
Star
Controller/Action/CacheCss.php

Summary

Maintainability
A
35 mins
Test Coverage

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

        \Magento\Framework\App\Action\Context $context,
        \ADM\QuickDevBar\Helper\Data $qdbHelper,
        \Magento\Framework\Controller\Result\RawFactory $resultRawFactory,
        \Magento\Framework\View\LayoutFactory $layoutFactory,
        \Magento\Framework\View\Asset\MergeService $mergeService
Severity: Minor
Found in Controller/Action/CacheCss.php - About 35 mins to fix

    The property $_mergeService is not named in camelCase.
    Open

    class CacheCss extends \ADM\QuickDevBar\Controller\Index
{


    protected $_mergeService;
    Severity: Minor
    Found in Controller/Action/CacheCss.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

    There are no issues that match your filters.

    Category
    Status