astehlik/typo3-extension-mediaoembed

View on GitHub
Classes/Controller/OembedController.php

Summary

Maintainability
A
1 hr
Test Coverage
C
70%

Method startRequestLoop has 29 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    private function startRequestLoop(Configuration $configuration): void
    {
        $response = null;
        $request = null;

Severity: Minor
Found in Classes/Controller/OembedController.php - About 1 hr to fix

    The class OembedController has a coupling between objects value of 21. Consider to reduce the number of dependencies under 13.
    Open

    class OembedController extends ActionController
    {
        private ConfigurationFactory $configurationFactory;
    
        private ContainerInterface $container;

    CouplingBetweenObjects

    Since: 1.1.0

    A class with too many dependencies has negative impacts on several quality aspects of a class. This includes quality criteria like stability, maintainability and understandability

    Example

    class Foo {
        /**
         * @var \foo\bar\X
         */
        private $x = null;
    
        /**
         * @var \foo\bar\Y
         */
        private $y = null;
    
        /**
         * @var \foo\bar\Z
         */
        private $z = null;
    
        public function setFoo(\Foo $foo) {}
        public function setBar(\Bar $bar) {}
        public function setBaz(\Baz $baz) {}
    
        /**
         * @return \SplObjectStorage
         * @throws \OutOfRangeException
         * @throws \InvalidArgumentException
         * @throws \ErrorException
         */
        public function process(\Iterator $it) {}
    
        // ...
    }

    Source https://phpmd.org/rules/design.html#couplingbetweenobjects

    There are no issues that match your filters.

    Category
    Status