Dynamic\Salsify\TypeHandler\Relation\HasOneHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handleHasOneHandlerType()	0	10	5

<?php

namespace Dynamic\Salsify\TypeHandler\Relation;

use SilverStripe\Core\Extension;
use SilverStripe\ORM\DataObject;

/**
 * Class HasOneRelationHandler
 *
 * @property-read \Dynamic\Salsify\TypeHandler\Relation\HasOneHandler|\Dynamic\Salsify\Model\Mapper $owner
 */
class HasOneHandler extends Extension
{
    /**
     * @var array
     */
    private static $field_types = [

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

        'HasOne',
    ];

    /**
     * @param $data
     * @param $dataField
     * @param $config
     * @param $dbField
     * @param string |\SilverStripe\ORM\DataObject $class
     *
     * @return int|DataObject
     * @throws \Exception
     */
    public function handleHasOneHandlerType($data, $dataField, $config, $dbField, $class)

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

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

    {
        if (!array_key_exists('relation', $config) || !is_array($config['relation'])) {
            return preg_match('/ID$/', $dbField) ? 0 : null;
        }

        $relationConfig = $config['relation'];
        $relatedClass = array_key_first($relationConfig);
        $object = $this->owner->mapToObject($relatedClass, $relationConfig[$relatedClass], $data);

The method mapToObject() does not exist on Dynamic\Salsify\TypeHandler\Relation\HasOneHandler.

        return preg_match('/ID$/', $dbField) ? $object->ID : $object;
    }
}