Borobudur\Cqrs\ReadModel\AbstractReadModel

Complexity

Total Complexity

5

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	deserialize()	0	8	2
A	serialize()	0	10	2
A	map()	0	4	1

<?php
/*
 * This file is part of the Borobudur-Cqrs package.
 *
 * (c) Hexacodelabs <http://hexacodelabs.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Borobudur\Cqrs\ReadModel;

use Borobudur\Bus\Message\MessageMapperTrait;
use Borobudur\Parameterize\Mapper\ParameterMapper;
use Borobudur\Parameterize\ParameterBag;
use Borobudur\Serialization\Serializer\Mixin\DeserializerTrait;
use Borobudur\Serialization\Serializer\Mixin\SerializerTrait;

/**
 * @author      Iqbal Maulana <iq.bluejack@gmail.com>
 * @created     8/18/15
 */
abstract class AbstractReadModel implements ReadModelInterface
{
    use SerializerTrait, DeserializerTrait, MessageMapperTrait {
        SerializerTrait::serialize as serializeData;
        DeserializerTrait::deserialize as deserializeData;
    }

    /**
     * @var array
     */
    protected static $mappings = array();

    /**
     * {@inheritdoc}
     */
    public static function deserialize(array $attributes)
    {
        if (static::$mappings) {

The expression static::$mappings of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $attributes = static::map($attributes, array_flip(static::$mappings));
        }

        return static::deserializeData($attributes);
    }

    /**
     * {@inheritdoc}
     */
    public function serialize()
    {
        $serialized = $this->serializeData();

        if (static::$mappings) {

The expression static::$mappings of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $serialized = static::map($serialized, static::$mappings);
        }

        return $serialized;
    }

    /**
     * Mapping data base on mapping keys.
     *
     * @param array $attributes
     * @param array $mappingKeys
     *
     * @return array
     */
    protected static function map(array $attributes, array $mappingKeys)
    {
        return (array) ParameterMapper::map(new ParameterBag($attributes), $mappingKeys);
    }
}