Dynamic\Foxy\Inventory\Test\TestOnly\Extension\TestVariationDataExtension

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	get_extra_config()	0	13	2

<?php

namespace Dynamic\Foxy\Inventory\Test\TestOnly\Extension;

use Dynamic\Foxy\Inventory\Test\TestOnly\Page\TestProduct;
use Dynamic\Foxy\Model\Variation;
use SilverStripe\ORM\DataExtension;

/**
 * Class TestVariationDataExtension
 * @package Dynamic\Foxy\Inventory\Test\TestOnly\Extension
 */
class TestVariationDataExtension extends DataExtension
{
    /**
     * @var string[]
     */
    private static $has_one = [

The private property $has_one is not used, and could be removed.

        'Product' => TestProduct::class,
    ];

    /**
     * @param null $class

Are you sure the doc-type for parameter $class is correct as it would always require null to be passed?

     * @param null $extensionClass

Are you sure the doc-type for parameter $extensionClass is correct as it would always require null to be passed?

     * @param null $args

Are you sure the doc-type for parameter $args is correct as it would always require null to be passed?

     * @return array
     */
    public static function get_extra_config($class = null, $extensionClass = null, $args = null)

The parameter $extensionClass is not used and could be removed.

The parameter $args is not used and could be removed.

    {
        $config = [];

        // Only add these extensions if the $class is set to DataExtensionTest_Player, to
        // test that the argument works.
        if (strcasecmp($class, Variation::class) === 0) {
            $config['has_one'] = [
                'Product' => TestProduct::class,
            ];
        }

        return $config;
    }
}