MovieTransformer::list() - Code Metrics - Inspection of "First Transformer + now we are using array instead..." - svbackend/my-art-lib - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9f513c...408a9b )

by Valentyn

created 2018-10-14 13:29 UTC

MovieTransformer::list() A

↳ Parent: MovieTransformer

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	15
ccs	4
cts	4
cp	1
rs	9.7666
c	0
b	0
f	0
cc	1
nc	1
nop	0
crap	1

<?php

namespace App\Movies\Transformer;

use App\Transformer\Transformer;

class MovieTransformer implements Transformer
{
    private $hiddenFields = [];

    public static function list(): self
    {
        $transformer = new self();
        $transformer->hiddenFields = [
            'budget',
            'genres',
            'imdbId',
            'overview',
            'runtime',
            'tmdb.id',
            'tmdb.voteAverage',
            'tmdb.voteCount',
        ];
        return $transformer;
    }

    public function process(array $data): array
    {
        if (!isset($data['id'])) {
            return $data;
        }

        if (isset($data['userWatchedMovie'])) {
            $data['isWatched'] = $data['userWatchedMovie'] !== null;
        }

        if (isset($data['tmdb.id'])) {
            $data['tmdb']['id'] = $data['tmdb.id'];
            $data['tmdb']['voteAverage'] = $data['tmdb.voteAverage'];
            $data['tmdb']['voteCount'] = $data['tmdb.voteCount'];
        }

        foreach ($this->hiddenFields as $hiddenField) {
            if (isset($data[$hiddenField])) {
                unset($data[$hiddenField]);
            }
        }

        return $data;
    }
}

1		<?php
2
3		namespace App\Movies\Transformer;
4
5		use App\Transformer\Transformer;
6
7		class MovieTransformer implements Transformer
8		{
9		private $hiddenFields = [];
10
11	17	public static function list(): self
12		{
13	17	$transformer = new self();
14	17	$transformer->hiddenFields = [
15		'budget',
16		'genres',
17		'imdbId',
18		'overview',
19		'runtime',
20		'tmdb.id',
21		'tmdb.voteAverage',
22		'tmdb.voteCount',
23		];
24	17	return $transformer;
25		}
26
27	17	public function process(array $data): array
28		{
29	17	if (!isset($data['id'])) {
30	17	return $data;
31		}
32
33	17	if (isset($data['userWatchedMovie'])) {
34		$data['isWatched'] = $data['userWatchedMovie'] !== null;
35		}
36
37	17	if (isset($data['tmdb.id'])) {
38	17	$data['tmdb']['id'] = $data['tmdb.id'];
39	17	$data['tmdb']['voteAverage'] = $data['tmdb.voteAverage'];
40	17	$data['tmdb']['voteCount'] = $data['tmdb.voteCount'];
41		}
42
43	17	foreach ($this->hiddenFields as $hiddenField) {
44	17	if (isset($data[$hiddenField])) {
45	17	unset($data[$hiddenField]);
46		}
47		}
48
49	17	return $data;
50		}
51		}

svbackend / my-art-lib

Push — master ( 9f513c...408a9b )

MovieTransformer::list() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like