TechDivision\Import\Attribute\Set\Actions\EavAttributeGroupAction

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	3	1
A	update()	0	3	1
A	persist()	0	8	1

<?php

/**
 * TechDivision\Import\Attribute\Set\Actions\EavAttributeGroupAction
 *
 * NOTICE OF LICENSE
 *
 * This source file is subject to the Open Software License (OSL 3.0)
 * that is available through the world-wide-web at this URL:
 * http://opensource.org/licenses/osl-3.0.php
 *
 * PHP version 5
 *
 * @author    Tim Wagner <t.wagner@techdivision.com>
 * @copyright 2019 TechDivision GmbH <info@techdivision.com>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/techdivision/import-attribute-set
 * @link      http://www.techdivision.com
 */

namespace TechDivision\Import\Attribute\Set\Actions;

use TechDivision\Import\Utils\EntityStatus;
use TechDivision\Import\Actions\AbstractAction;

/**
 * An action implementation that provides CRUD functionality for EAV attribute groups.
 *
 * @author    Tim Wagner <t.wagner@techdivision.com>
 * @copyright 2019 TechDivision GmbH <info@techdivision.com>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/techdivision/import-attribute-set
 * @link      http://www.techdivision.com
 */
class EavAttributeGroupAction extends AbstractAction implements EavAttributeGroupActionInterface
{

    /**
     * Helper method that create/update the passed entity, depending on
     * the entity's status.
     *
     * @param array $row The entity data to create/update
     *
     * @return string The last inserted ID
     */
    public function persist(array $row)
    {

        // load the method name
        $methodName = $row[EntityStatus::MEMBER_NAME];

        // invoke the method
        return $this->$methodName($row);
    }

    /**
     * Creates's the entity with the passed attributes.
     *
     * @param array       $row  The attributes of the entity to create
     * @param string|null $name The name of the prepared statement that has to be executed
     *
     * @return string The last inserted ID
     */
    public function create($row, $name = null)
    {
        return $this->getCreateProcessor()->execute($row, $name);

The expression return $this->getCreateProcessor()->execute($row, $name) returns the type void which is incompatible with the documented return type string.

Are you sure the usage of $this->getCreateProcessor()->execute($row, $name) targeting TechDivision\Import\Actions\Processors\ProcessorInterface::execute() seems to always return null.

    }

    /**
     * Update's the entity with the passed attributes.
     *
     * @param array       $row  The attributes of the entity to update
     * @param string|null $name The name of the prepared statement that has to be executed
     *
     * @return string The ID of the updated product
     */
    public function update($row, $name = null)
    {
        return $this->getUpdateProcessor()->execute($row, $name);

The expression return $this->getUpdateProcessor()->execute($row, $name) returns the type void which is incompatible with the documented return type string.

Are you sure the usage of $this->getUpdateProcessor()->execute($row, $name) targeting TechDivision\Import\Actions\Processors\ProcessorInterface::execute() seems to always return null.

    }
}