Dynamic\Salsify\Model\Mapper

Complexity

Total Complexity

154

Size/Duplication

Total Lines	938
Duplicated Lines	0 %

Importance

Changes	14
Bugs	2	Features	0

36 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	3
A	resetProductStream()	0	3	1
A	map()	0	27	6
A	updateMappingHash()	0	18	2
A	resetAssetStream()	0	3	1
A	getMappingHash()	0	3	1
A	clearValue()	0	17	4
A	classConfigHasSalsifyRelation()	0	20	6
A	getFieldsRequiringSalsifyObjects()	0	15	3
F	mapToObject()	0	104	23
A	mappingHasSalsifyRelation()	0	8	3
A	writeManyRelation()	0	22	6
A	objectDataUpToDate()	0	12	3
A	canModifyType()	0	9	2
A	typeFallback()	0	9	2
A	typeRequiresWrite()	0	9	2
A	objectRelationsUpToDate()	0	17	4
A	removeUnrelated()	0	15	3
A	getUniqueFilter()	0	18	3
A	hasFile()	0	3	1
A	getAssetStream()	0	3	1
A	typeRequiresSalsifyObjects()	0	9	2
C	getField()	0	46	14
A	findObjectByUnique()	0	12	3
A	isMapperHashUpToDate()	0	20	4
A	handleShouldSkip()	0	16	4
A	handleType()	0	17	4
A	getProductStream()	0	3	1
B	uniqueFields()	0	29	8
A	objectUpToDate()	0	24	6
A	getSortColumn()	0	11	3
A	hasValidUniqueFilter()	0	3	1
A	handleModification()	0	18	5
B	writeValue()	0	34	11
A	getFieldType()	0	15	4
A	findBySalsifyID()	0	19	4

<?php

namespace Dynamic\Salsify\Model;

use Dynamic\Salsify\ORM\SalsifyIDExtension;
use Dynamic\Salsify\Task\ImportTask;
use Exception;
use JsonMachine\JsonMachine;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\ORM\DataList;
use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\HasManyList;
use SilverStripe\ORM\ManyManyList;
use SilverStripe\Versioned\Versioned;

The type SilverStripe\Versioned\Versioned was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Class Mapper
 * @package Dynamic\Salsify\Model
 */
class Mapper extends Service
{

    /**
     * @var bool
     */
    public static $SINGLE = false;

    /**
     * @var bool
     */
    public static $MULTIPLE = true;

    /**
     * @var
     */
    private $file = null;

    /**
     * @var JsonMachine
     */
    private $productStream;

    /**
     * @var JsonMachine
     */
    private $assetStream;

    /**
     * @var array
     */
    private $currentUniqueFields = [];

    /**
     * @var int
     */
    private $importCount = 0;

    /**
     * @var bool
     */
    public $skipSilently = false;

    /**
     * Mapper constructor.
     * @param string $importerKey
     * @param $file
     * @throws \Exception
     */
    public function __construct($importerKey, $file = null)
    {
        parent::__construct($importerKey);

$importerKey of type string is incompatible with the type Dynamic\Salsify\Model\stirng expected by parameter $importerKey of Dynamic\Salsify\Model\Service::__construct().

        if (!$this->config()->get('mapping')) {
            throw  new Exception('A Mapper needs a mapping');
        }

        if ($file !== null) {
            $this->file = $file;
            $this->resetProductStream();
            $this->resetAssetStream();
        }
    }

    /**
     * Generates the current hash for the mapping
     *
     * @return string
     */
    private function getMappingHash()
    {
        return md5(serialize($this->config()->get('mapping')));
    }

    /**
     * Updates the mapping hash for the mapper
     * @param DataObject|SalsifyIDExtension $object
     * @param bool $relations
     */
    private function updateMappingHash($object, $relations)
    {
        $filter = [
            'MapperService' => $this->importerKey,
            'ForRelations' => $relations,
        ];
        /** @var MapperHash $mapperHash */
        $mapperHash = $object->MapperHashes()->filter($filter)->first();
        if ($mapperHash == null) {
            $mapperHash = MapperHash::create();
            $mapperHash->MappedObjectID = $object->ID;
            $mapperHash->MappedObjectClass = $object->ClassName;
            $mapperHash->MapperService = $this->importerKey;
            $mapperHash->ForRelations = $relations;
        }

        $mapperHash->MapperHash = $this->getMappingHash();
        $mapperHash->write();
    }

    /**
     *
     */
    public function resetProductStream()
    {
        $this->productStream = JsonMachine::fromFile($this->file, '/4/products');
    }

    /**
     *
     */
    public function resetAssetStream()
    {
        $this->assetStream = JsonMachine::fromFile($this->file, '/3/digital_assets');
    }

    /**
     * Maps the data
     * @throws \Exception
     */
    public function map()
    {
        $this->extend('onBeforeMap', $this->file, Mapper::$MULTIPLE);

        foreach ($this->yieldKeyVal($this->productStream) as $name => $data) {
            foreach ($this->yieldKeyVal($this->config()->get('mapping')) as $class => $mappings) {
                $this->mapToObject($class, $mappings, $data);

It seems like $class can also be of type true; however, parameter $class of Dynamic\Salsify\Model\Mapper::mapToObject() does only seem to accept SilverStripe\ORM\DataObject|string, maybe add an additional type check?

                $this->currentUniqueFields = [];
            }
        }

        if ($this->mappingHasSalsifyRelation()) {
            ImportTask::output("----------------");
            ImportTask::output("Setting up salsify relations");
            ImportTask::output("----------------");
            $this->resetProductStream();

            foreach ($this->yieldKeyVal($this->productStream) as $name => $data) {
                foreach ($this->yieldKeyVal($this->config()->get('mapping')) as $class => $mappings) {
                    $this->mapToObject($class, $mappings, $data, null, true);
                    $this->currentUniqueFields = [];
                }
            }
        }

        ImportTask::output("Imported and updated $this->importCount products.");
        $this->extend('onAfterMap', $this->file, Mapper::$MULTIPLE);
    }

    /**
     * @param string|DataObject $class
     * @param array $mappings The mapping for a specific class
     * @param array $data
     * @param DataObject|null $object
     * @param bool $salsifyRelations
     * @param bool $forceUpdate
     *
     * @return DataObject|null
     * @throws \Exception
     */
    public function mapToObject(
        $class,
        $mappings,
        $data,
        $object = null,
        $salsifyRelations = false,
        $forceUpdate = false
    ) {
        if ($salsifyRelations) {
            if (!$this->classConfigHasSalsifyRelation($mappings)) {
                return null;
            }
        }

        // if object was not passed
        if ($object === null) {
            $object = $this->findObjectByUnique($class, $mappings, $data);

            // if no existing object was found but a unique filter is valid (not empty)
            if (!$object) {

$object is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

                // don't try to create related objects that don't exist
                if (!$salsifyRelations) {
                    return null;
                }

                if (!$this->hasValidUniqueFilter($class, $mappings, $data)) {
                    return null;
                }
                
                $object = $class::create();
            }
        }

        $wasPublished = $object->hasExtension(Versioned::class) ? $object->isPublished() : false;
        $wasWritten = $object->isInDB();

        $firstUniqueKey = array_keys($this->uniqueFields($class, $mappings))[0];
        if (array_key_exists($mappings[$firstUniqueKey]['salsifyField'], $data)) {
            $firstUniqueValue = $data[$mappings[$firstUniqueKey]['salsifyField']];
        } else {
            $firstUniqueValue = 'NULL';
        }
        ImportTask::output("Updating $class $firstUniqueKey $firstUniqueValue");

        if (
            !$forceUpdate &&
            $this->objectUpToDate($object, $data, $firstUniqueKey, $firstUniqueValue, $salsifyRelations)
        ) {
            return $object;
        }

        foreach ($this->yieldKeyVal($mappings) as $dbField => $salsifyField) {
            $field = $this->getField($salsifyField, $data);
            if ($field === false) {
                $this->clearValue($object, $dbField, $salsifyField);
                continue;
            }

            $type = $this->getFieldType($salsifyField);
            // skip all but salsify relations types if not doing relations
            if ($salsifyRelations && !$this->typeRequiresSalsifyObjects($type)) {
                continue;
            }

            // skip salsify relations types if not doing relations
            if (!$salsifyRelations && $this->typeRequiresSalsifyObjects($type)) {
                continue;
            }

            $value = null;
            $objectData = $data;

The assignment to $objectData is dead and can be removed.

            if ($this->handleShouldSkip($class, $dbField, $salsifyField, $data)) {

It seems like $dbField can also be of type true; however, parameter $dbField of Dynamic\Salsify\Model\Mapper::handleShouldSkip() does only seem to accept string, maybe add an additional type check?

                if (!$this->skipSilently) {
                    ImportTask::output("Skipping $class $firstUniqueKey $firstUniqueValue");
                    $this->skipSilently = false;
                }
                return false;

The expression return false returns the type false which is incompatible with the documented return type SilverStripe\ORM\DataObject|null.

            };

            $objectData = $this->handleModification($type, $class, $dbField, $salsifyField, $data);

It seems like $dbField can also be of type true; however, parameter $dbField of Dynamic\Salsify\Model\Mapper::handleModification() does only seem to accept string, maybe add an additional type check?

            $sortColumn = $this->getSortColumn($salsifyField);

            if ($salsifyRelations == false && !array_key_exists($field, $objectData)) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

                continue;
            }

            $value = $this->handleType($type, $class, $objectData, $field, $salsifyField, $dbField);

It seems like $dbField can also be of type true; however, parameter $dbField of Dynamic\Salsify\Model\Mapper::handleType() does only seem to accept string, maybe add an additional type check?

            $this->writeValue($object, $type, $dbField, $value, $sortColumn);

It seems like $dbField can also be of type true; however, parameter $dbField of Dynamic\Salsify\Model\Mapper::writeValue() does only seem to accept string, maybe add an additional type check?

        }

        if (!$this->isMapperHashUpToDate($object, $salsifyRelations)) {
            $this->updateMappingHash($object, $salsifyRelations);
        }

        $this->extend('beforeObjectWrite', $object);
        if ($object->isChanged()) {
            $object->write();
            $this->importCount++;
            $this->extend('afterObjectWrite', $object, $wasWritten, $wasPublished);
        } else {
            ImportTask::output("$class $firstUniqueKey $firstUniqueValue was not changed.");
        }
        return $object;
    }

    /**
     * @param DataObject $object
     * @param array $data
     * @param string $firstUniqueKey
     * @param string $firstUniqueValue
     * @param bool $salsifyRelations
     * @return bool
     */
    private function objectUpToDate($object, $data, $firstUniqueKey, $firstUniqueValue, $salsifyRelations = false)
    {
        if ($this->config()->get('skipUpToDate') == false) {
            return false;
        }

        if (!$this->isMapperHashUpToDate($object, $salsifyRelations)) {
            ImportTask::output("Forcing update for $firstUniqueKey $firstUniqueValue. Mappings changed.");
            return false;
        }

        if ($salsifyRelations == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            if ($this->objectDataUpToDate($object, $data, $firstUniqueKey, $firstUniqueValue)) {
                ImportTask::output("Skipping $firstUniqueKey $firstUniqueValue. It is up to Date.");
                return true;
            }
        } else {
            if ($this->objectRelationsUpToDate($object, $data, $firstUniqueKey, $firstUniqueValue)) {
                ImportTask::output("Skipping $firstUniqueKey $firstUniqueValue relations. It is up to Date.");
                return true;
            }
        }

        return false;
    }

    /**
     * @param DataObject $object
     * @param array $data
     * @param string $firstUniqueKey
     * @param string $firstUniqueValue
     * @return bool
     */
    public function objectDataUpToDate($object, $data, $firstUniqueKey, $firstUniqueValue)
    {
        // assume not up to date if field does not exist on object
        if (!$object->hasField('SalsifyUpdatedAt')) {
            return false;
        }

        if ($data['salsify:updated_at'] != $object->getField('SalsifyUpdatedAt')) {
            return false;
        }

        return true;
    }

    /**
     * @param DataObject $object
     * @param array $data
     * @param string $firstUniqueKey
     * @param string $firstUniqueValue
     * @return bool
     */
    public function objectRelationsUpToDate($object, $data, $firstUniqueKey, $firstUniqueValue)
    {
        // assume not up to date if field does not exist on object
        if (!$object->hasField('SalsifyRelationsUpdatedAt')) {
            return false;
        }

        // relations were never updated, so its up to date
        if (!isset($data['salsify:relations_updated_at'])) {
            return true;
        }

        if ($data['salsify:relations_updated_at'] != $object->getField('SalsifyRelationsUpdatedAt')) {
            return false;
        }

        return true;
    }

    /**
     * @param array $salsifyField
     * @param array $data
     *
     * @return string|false
     */
    private function getField($salsifyField, $data)
    {
        if (!is_array($salsifyField)) {

The condition is_array($salsifyField) is always true.

            return array_key_exists($salsifyField, $data) ? $salsifyField : false;
        }

        $hasSalsifyField = array_key_exists('salsifyField', $salsifyField);
        $isLiteralField = (
            $this->getFieldType($salsifyField) === 'Literal' &&

The condition $this->getFieldType($salsifyField) === 'Literal' is always false.

            array_key_exists('value', $salsifyField)
        );
        $isSalsifyRelationField = (
            $this->getFieldType($salsifyField) === 'SalsifyRelation' &&

The condition $this->getFieldType($salsifyField) === 'SalsifyRelation' is always false.

            $hasSalsifyField
        );

        if ($isLiteralField) {

The condition $isLiteralField is always false.

            return $salsifyField['value'];
        }

        if ($isSalsifyRelationField) {

The condition $isSalsifyRelationField is always false.

            return $salsifyField['salsifyField'];
        }

        if (!$hasSalsifyField) {
            return false;
        }

        if (array_key_exists($salsifyField['salsifyField'], $data)) {
            return $salsifyField['salsifyField'];
        } elseif (array_key_exists('fallback', $salsifyField)) {
            // make fallback an array
            if (!is_array($salsifyField['fallback'])) {
                $salsifyField['fallback'] = [$salsifyField['fallback']];
            }

            foreach ($this->yieldSingle($salsifyField['fallback']) as $fallback) {
                if (array_key_exists($fallback, $data)) {
                    return $fallback;
                }
            }
        } elseif (array_key_exists('modification', $salsifyField)) {
            return $salsifyField['salsifyField'];
        }

        return false;
    }

    /**
     * @param string $class
     * @param array $mappings
     * @param array $data
     *
     * @return bool
     */
    private function hasValidUniqueFilter($class, $mappings, $data)
    {
        return (bool) count(array_filter($this->getUniqueFilter($class, $mappings, $data)));
    }

    /**
     * @param string $class
     * @param array $mappings
     * @param array $data
     *
     * @return array
     */
    private function getUniqueFilter($class, $mappings, $data)
    {
        $uniqueFields = $this->uniqueFields($class, $mappings);
        // creates a filter
        $filter = [];
        foreach ($this->yieldKeyVal($uniqueFields) as $dbField => $salsifyField) {
            $modifiedData = $data;
            $fieldMapping = $mappings[$dbField];
            $fieldType = $this->getFieldType($salsifyField);
            $modifiedData = $this->handleModification($fieldType, $class, $dbField, $fieldMapping, $modifiedData);

It seems like $dbField can also be of type true; however, parameter $dbField of Dynamic\Salsify\Model\Mapper::handleModification() does only seem to accept string, maybe add an additional type check?

            // adds unique fields to filter
            if (array_key_exists($salsifyField, $modifiedData)) {
                $filter[$dbField] = $modifiedData[$salsifyField];
            }
        }

        return $filter;
    }

    /**
     * @param string $class
     * @param array $mappings
     * @param array $data
     *
     * @return \SilverStripe\ORM\DataObject
     */
    private function findObjectByUnique($class, $mappings, $data)
    {
        if ($obj = $this->findBySalsifyID($class, $mappings, $data)) {
            return $obj;
        }

        if (!$this->hasValidUniqueFilter($class, $mappings, $data)) {
            return null;
        }

        $filter = $this->getUniqueFilter($class, $mappings, $data);
        return DataObject::get($class)->filter($filter)->first();
    }

    /**
     * @param string $class
     * @param array $mappings
     * @param array $data
     *
     * @return \SilverStripe\ORM\DataObject|bool
     */
    private function findBySalsifyID($class, $mappings, $data)
    {
        /** @var DataObject $genericObject */
        $genericObject = Injector::inst()->get($class);
        if (
            !$genericObject->hasExtension(SalsifyIDExtension::class) &&
            !$genericObject->hasField('SalsifyID')
        ) {
            return false;
        }

        $obj = DataObject::get($class)->filter([
            'SalsifyID' => $data['salsify:id'],
        ])->first();
        if ($obj) {
            return $obj;
        }

        return false;
    }

    /**
     * Gets a list of all the unique field keys
     *
     * @param string class

The type Dynamic\Salsify\Model\class was not found. Maybe you did not declare it correctly or list all dependencies?

     * @param array $mappings
     * @return array
     */
    private function uniqueFields($class, $mappings)
    {
        // cached after first map
        if (array_key_exists($class, $this->currentUniqueFields) && !empty($this->currentUniqueFields[$class])) {
            return $this->currentUniqueFields[$class];
        }

        $uniqueFields = [];
        foreach ($this->yieldKeyVal($mappings) as $dbField => $salsifyField) {
            if (!is_array($salsifyField)) {
                continue;
            }

            if (
                !array_key_exists('unique', $salsifyField) ||
                !array_key_exists('salsifyField', $salsifyField)
            ) {
                continue;
            }

            if ($salsifyField['unique'] !== true) {
                continue;
            }

            $uniqueFields[$dbField] = $salsifyField['salsifyField'];
        }

        $this->currentUniqueFields[$class] = $uniqueFields;
        return $uniqueFields;
    }

    /**
     * @param array|string $salsifyField
     * @return bool|mixed
     */
    private function getSortColumn($salsifyField)
    {
        if (!is_array($salsifyField)) {
            return false;
        }

        if (array_key_exists('sortColumn', $salsifyField)) {
            return $salsifyField['sortColumn'];
        }

        return false;
    }

    /**
     * @return bool
     */
    private function mappingHasSalsifyRelation()
    {
        foreach ($this->yieldKeyVal($this->config()->get('mapping')) as $class => $mappings) {
            if ($this->classConfigHasSalsifyRelation($mappings)) {
                return true;
            }
        }
        return false;
    }

    /**
     * @param array $classConfig
     * @return bool
     */
    private function classConfigHasSalsifyRelation($classConfig)
    {
        foreach ($this->yieldKeyVal($classConfig) as $field => $config) {
            if (!is_array($config)) {
                continue;
            }

            if (!array_key_exists('salsifyField', $config)) {
                continue;
            }

            if (!array_key_exists('type', $config)) {
                continue;
            }

            if (in_array($config['type'], $this->getFieldsRequiringSalsifyObjects())) {
                return true;
            }
        }
        return false;
    }

    /**
     * @return array
     */
    private function getFieldsRequiringSalsifyObjects()
    {
        $fieldTypes = $this->config()->get('field_types');

The assignment to $fieldTypes is dead and can be removed.

        $types = [];
        foreach ($this->yieldKeyVal($this->config()->get('field_types')) as $field => $config) {
            $type = [
                'type' => $field,
                'config' => $config,
            ];
            if ($this->typeRequiresSalsifyObjects($type)) {
                $types[] = $field;
            }
        }

        return $types;
    }

    /**
     * @param array $type
     * @return bool
     */
    private function typeRequiresWrite($type)
    {
        $config = $type['config'];

        if (array_key_exists('requiresWrite', $config)) {
            return $config['requiresWrite'];
        }

        return false;
    }

    /**
     * @param array $type
     * @return bool
     */
    private function typeRequiresSalsifyObjects($type)
    {
        $config = $type['config'];

        if (array_key_exists('requiresSalsifyObjects', $config)) {
            return $config['requiresSalsifyObjects'];
        }

        return false;
    }

    /**
     * @param array $type
     * @return string|bool
     */
    private function typeFallback($type)
    {
        $config = $type['config'];

        if (array_key_exists('fallback', $config)) {
            return $config['fallback'];
        }

        return false;
    }

    /**
     * @param array $type
     * @return bool
     */
    private function canModifyType($type)
    {
        $config = $type['config'];

        if (array_key_exists('allowsModification', $config)) {
            return $config['allowsModification'];
        }

        return true;
    }

    /**
     * @param array $type
     * @param string $class
     * @param string $dbField
     * @param array $config
     * @param array $data
     * @return array
     */
    private function handleModification($type, $class, $dbField, $config, $data)
    {
        if (!is_array($config)) {

The condition is_array($config) is always true.

            return $data;
        }

        if (!$this->canModifyType($type)) {
            return $data;
        }

        if (array_key_exists('modification', $config)) {
            $mod = $config['modification'];
            if ($this->hasMethod($mod)) {
                return $this->{$mod}($class, $dbField, $config, $data);
            }
            ImportTask::output("{$mod} is not a valid field modifier. skipping modification for field {$dbField}.");
        }
        return $data;
    }

    /**
     * @param string $class
     * @param string $dbField
     * @param array $config
     * @param array $data
     * @return boolean
     */
    public function handleShouldSkip($class, $dbField, $config, $data)
    {
        if (!is_array($config)) {

The condition is_array($config) is always true.

            return false;
        }

        if (array_key_exists('shouldSkip', $config)) {
            $skipMethod = $config['shouldSkip'];
            if ($this->hasMethod($skipMethod)) {
                return $this->{$skipMethod}($class, $dbField, $config, $data);
            }
            ImportTask::output(
                "{$skipMethod} is not a valid skip test method. Skipping skip test for field {$dbField}."
            );
        }
        return false;
    }

    /**
     * @param string|array $field
     * @return array
     */
    public function getFieldType($field)
    {
        $fieldTypes = $this->config()->get('field_types');
        if (is_array($field) && array_key_exists('type', $field)) {
            if (array_key_exists($field['type'], $fieldTypes)) {
                return [
                    'type' => $field['type'],
                    'config' => $fieldTypes[$field['type']],
                ];
            }
        }
        // default to raw
        return [
            'type' => 'Raw',
            'config' => $fieldTypes['Raw'],
        ];
    }

    /**
     * @param array $type
     * @param string|DataObject $class
     * @param array $salsifyData
     * @param string $salsifyField
     * @param array $dbFieldConfig
     * @param string $dbField
     *
     * @return mixed
     */
    private function handleType($type, $class, $salsifyData, $salsifyField, $dbFieldConfig, $dbField)
    {
        $typeName = $type['type'];
        $typeConfig = $type['config'];
        if ($this->hasMethod("handle{$typeName}Type")) {
            return $this->{"handle{$typeName}Type"}($class, $salsifyData, $salsifyField, $dbFieldConfig, $dbField);
        }

        if (array_key_exists('fallback', $typeConfig)) {
            $fallback = $typeConfig['fallback'];
            if ($this->hasMethod("handle{$fallback}Type")) {
                return $this->{"handle{$fallback}Type"}($class, $salsifyData, $salsifyField, $dbFieldConfig, $dbField);
            }
        }

        ImportTask::output("{$typeName} is not a valid type. skipping field {$dbField}.");
        return '';
    }

    /**
     * @param DataObject $object
     * @param array $type
     * @param string $dbField
     * @param mixed $value
     * @param string|bool $sortColumn
     *
     * @throws \Exception
     */
    private function writeValue($object, $type, $dbField, $value, $sortColumn)
    {
        $isManyRelation = array_key_exists($dbField, $object->config()->get('has_many')) ||
            array_key_exists($dbField, $object->config()->get('many_many')) ||
            array_key_exists($dbField, $object->config()->get('belongs_many_many'));

        $isSingleRelation = array_key_exists(rtrim($dbField, 'ID'), $object->config()->get('has_one'));

        // write the object so relations can be written
        if ($this->typeRequiresWrite($type) && !$object->exists()) {
            $this->extend('beforeObjectWrite', $object);
            $object->write();
        }

        if (!$isManyRelation) {
            if (!$isSingleRelation || ($isSingleRelation && $value !== false)) {
                $object->$dbField = $value;
            }
            return;
        }

        // change to an array and filter out empty values
        if (!is_array($value)) {
            $value = [$value];
        }
        $value = array_filter($value);

        // don't try to write an empty set
        if (!count($value)) {
            return;
        }

        $this->removeUnrelated($object, $dbField, $value);
        $this->writeManyRelation($object, $dbField, $value, $sortColumn);
    }

    /**
     * @param DataObject $object
     * @param string $dbField
     * @param array $value
     * @param string|bool $sortColumn
     *
     * @throws \Exception
     */
    private function writeManyRelation($object, $dbField, $value, $sortColumn)
    {
        /** @var DataList|HasManyList|ManyManyList $relation */
        $relation = $object->{$dbField}();

        if ($sortColumn && $relation instanceof ManyManyList) {
            for ($i = 0; $i < count($value); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

                $relation->add($value[$i], [$sortColumn => $i]);
            }
            return;
        }

        // HasManyList, so it exists on the value
        if ($sortColumn) {
            for ($i = 0; $i < count($value); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

                $value[$i]->{$sortColumn} = $i;
                $relation->add($value[$i]);
            }
            return;
        }

        $relation->addMany($value);
    }

    /**
     * Removes unrelated objects in the relation that were previously related
     * @param DataObject $object
     * @param string $dbField
     * @param array $value
     */
    private function removeUnrelated($object, $dbField, $value)
    {
        $ids = [];
        foreach ($value as $v) {
            $ids[] = $v->ID;
        }

        /** @var DataList $relation */
        $relation = $object->{$dbField}();
        // remove all unrelated - removeAll had an odd side effect (relations only got added back half the time)
        if (!empty($ids)) {
            $relation->removeMany(
                $relation->exclude([
                    'ID' => $ids,
                ])->column('ID')
            );
        }
    }

    /**
     * @param string|array $salsifyField
     * @throws Exception
     */
    private function clearValue($object, $dbField, $salsifyField)
    {
        if (
            is_array($salsifyField) &&
            array_key_exists('keepExistingValue', $salsifyField) &&
            $salsifyField['keepExistingValue']
        ) {
            return;
        }

        $type = [
            'type' => 'null',
            'config' => [],
        ];

        // clear any existing value
        $this->writeValue($object, $type, $dbField, null, null);
    }

    /**
     * @return JsonMachine
     */
    public function getProductStream()
    {
        return $this->productStream;
    }

    /**
     * @return JsonMachine
     */
    public function getAssetStream()
    {
        return $this->assetStream;
    }

    /**
     * @return bool
     */
    public function hasFile()
    {
        return $this->file !== null;
    }

    /**
     * @param DataObject|SalsifyIDExtension $object
     * @return bool
     */
    private function isMapperHashUpToDate($object, $relations)
    {
        if (!$object->hasMethod('MapperHashes')) {

The method hasMethod() does not exist on Dynamic\Salsify\ORM\SalsifyIDExtension.

            return true;
        }

        $filter = [
            'MapperService' => $this->importerKey,
            'ForRelations' => $relations,
        ];
        /** @var MapperHash $hash */
        if ($hash = $object->MapperHashes()->filter($filter)->first()) {
            if ($hash->MapperHash != $this->getMappingHash()) {
                return false;
            }
        } else {
            return false;
        }

        return true;
    }
}