Ps2alerts\Api\Controller\Endpoint\Profiles\PlayerProfileEndpointController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	36
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	7	7	1
A	getPlayer()	10	10	1

<?php

namespace Ps2alerts\Api\Controller\Endpoint\Profiles;

use Ps2alerts\Api\Controller\Endpoint\AbstractEndpointController;
use Ps2alerts\Api\Repository\Metrics\PlayerTotalRepository;
use Ps2alerts\Api\Transformer\Profiles\PlayerProfileTransformer;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Message\ResponseInterface;

class PlayerProfileEndpointController extends AbstractEndpointController

This class seems to be duplicated in your project.

{
    /**
     * Construct
     *
     * @param PlayerTotalRepository    $playerTotalRepo
     * @param PlayerProfileTransformer $playerProfileTransformer
     */
    public function __construct(
        PlayerTotalRepository    $playerTotalRepo,
        PlayerProfileTransformer $playerProfileTransformer
    ) {
        $this->repository = $playerTotalRepo;
        $this->transformer = $playerProfileTransformer;
    }

    /**
     * Gets a player
     *
     * @param  ServerRequestInterface $request
     * @param  ResponseInterface      $response
     * @param  array                  $args
     *
     * @return ResponseInterface
     */
    public function getPlayer(ServerRequestInterface $request, ResponseInterface $response, array $args)

The parameter $request is not used and could be removed.

The parameter $response is not used and could be removed.

    {
        $player = $this->repository->readSinglebyId($args['id']);

        return $this->respond(
            'item',
            $player,
            $this->transformer
        );
    }
}