WSW\SiftScience\Transformers\AbstractTransformer

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	loadItem()	0	7	1
A	loadCollection()	0	9	2

<?php

namespace WSW\SiftScience\Transformers;

use League\Fractal\Serializer\ArraySerializer;
use League\Fractal\TransformerAbstract;

/**
 * Class AbstractTransformer
 *
 * @package WSW\SiftScience\Transformers
 * @author Ronaldo Matos Rodrigues <ronaldo@whera.com.br>
 */
abstract class AbstractTransformer extends TransformerAbstract
{
    /**
     * @param mixed $data
     * @param AbstractTransformer $transformer
     *
     * @return array
     */
    protected function loadItem($data = null, AbstractTransformer $transformer = null)
    {
        return $this->getCurrentScope()
                    ->getManager()
                    ->createData($this->item($data, $transformer))

It seems like $transformer defined by parameter $transformer on line 22 can be null; however, League\Fractal\TransformerAbstract::item() does not accept null, maybe add an additional type check?

                    ->toArray();
    }

    /**
     * @param mixed $data
     * @param AbstractTransformer $transformer
     *
     * @return array
     */
    protected function loadCollection($data = null, AbstractTransformer $transformer = null)
    {
        $result = $this->getCurrentScope()
                    ->getManager()
                    ->setSerializer(new ArraySerializer)
                    ->createData($this->collection($data, $transformer))->toArray();

It seems like $transformer defined by parameter $transformer on line 36 can be null; however, League\Fractal\TransformerAbstract::collection() does not accept null, maybe add an additional type check?

        return (isset($result['data'])) ? $result['data'] : $result;
    }
}