Ps2alerts\Api\Controller\Endpoint\Search\SearchEndpointController

Complexity

Total Complexity

18

Size/Duplication

Total Lines	183
Duplicated Lines	13.11 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	getPlayersByTerm()	0	16	2
A	getOutfitsByTerm()	0	16	2
A	searchForPlayer()	0	9	1
A	searchForOutfit()	0	20	2
A	parsePlayerName()	12	12	3
A	parseOutfitName()	12	12	3
A	parsePlayerID()	0	16	4

<?php

namespace Ps2alerts\Api\Controller\Endpoint\Search;

use Ps2alerts\Api\Controller\Endpoint\AbstractEndpointController;
use Ps2alerts\Api\Repository\Metrics\OutfitTotalRepository;
use Ps2alerts\Api\Repository\Metrics\PlayerTotalRepository;
use Ps2alerts\Api\Transformer\Search\OutfitSearchTransformer;
use Ps2alerts\Api\Transformer\Search\PlayerSearchTransformer;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Message\ResponseInterface;

class SearchEndpointController extends AbstractEndpointController
{
    /**
     * Construct
     *
     * @param OutfitSearchTransformer $outfitSearchTransformer
     * @param OutfitTotalRepository   $outfitTotalRepo
     * @param PlayerSearchTransformer $playerSearchTransformer
     * @param PlayerTotalRepository   $playerTotalRepo
     */
    public function __construct(
        OutfitTotalRepository   $outfitTotalRepo,
        PlayerTotalRepository   $playerTotalRepo,
        OutfitSearchTransformer $outfitSearchTransformer,
        PlayerSearchTransformer $playerSearchTransformer
    ) {
        $this->playerRepository        = $playerTotalRepo;

The property playerRepository does not seem to exist. Did you mean repository?

        $this->outfitRepository        = $outfitTotalRepo;

The property outfitRepository does not seem to exist. Did you mean repository?

        $this->playerSearchTransformer = $playerSearchTransformer;

The property playerSearchTransformer does not seem to exist. Did you mean transformer?

        $this->outfitSearchTransformer = $outfitSearchTransformer;

The property outfitSearchTransformer does not seem to exist. Did you mean transformer?

    }

    /**
     * Endpoint to return potential players based on search term
     *
     * @param  ServerRequestInterface $request
     * @param  ResponseInterface      $response
     * @param  array                  $args
     *
     * @return ResponseInterface
     */
    public function getPlayersByTerm(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.

    {
        // If a valid player name we're searching on
        $this->parsePlayerName($args['term']);
        $players = $this->searchForPlayer($args['term']);

        if (! empty($players)) {
            return $this->respond(
                'collection',
                $players,
                $this->playerSearchTransformer

The property playerSearchTransformer does not seem to exist. Did you mean transformer?

            );
        }

        return $this->respondWithError('Player could not be found', self::CODE_EMPTY);
    }

    /**
     * Endpoint to return potential players based on search term
     *
     * @param  ServerRequestInterface $request
     * @param  ResponseInterface      $response
     * @param  array                  $args
     *
     * @return ResponseInterface
     */
    public function getOutfitsByTerm(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.

    {
        $name = urldecode($args['term']); // Spaces will have to URL encoded

        $this->parseOutfitName($name);

        if (! empty($outfits)) {

The variable $outfits seems to never exist, and therefore empty should always return true. Did you maybe rename this variable?

            return $this->respond(
                'collection',
                $outfits,
                $this->outfitSearchTransformer

The property outfitSearchTransformer does not seem to exist. Did you mean transformer?

            );
        }

        return $this->respondWithError('Outfit could not be found', self::CODE_EMPTY);
    }

    /**
     * Takes a player name and searches for it
     *
     * @param  string $term
     *
     * @return array
     */
    public function searchForPlayer($term)
    {
        $query = $this->playerRepository->newQuery();

The property playerRepository does not seem to exist. Did you mean repository?

        $query->cols(['*']);
        $query->where('playerName LIKE :term');
        $query->bindValue('term', "%{$term}%");

        return $this->playerRepository->fireStatementAndReturn($query);

The property playerRepository does not seem to exist. Did you mean repository?

    }

    /**
     * Takes a outfit name and searches for it
     *
     * @param  string $term
     *
     * @return array
     */
    public function searchForOutfit($term)
    {
        $query = $this->outfitRepository->newQuery();

The property outfitRepository does not seem to exist. Did you mean repository?

        $query->cols(['*']);
        $query->where("outfitTag LIKE :term");
        $query->bindValue('term', "%{$term}%");

        $data = $this->outfitRepository->fireStatementAndReturn($query);

The property outfitRepository does not seem to exist. Did you mean repository?

        if (empty($data)) {
            $query = $this->outfitRepository->newQuery();

The property outfitRepository does not seem to exist. Did you mean repository?

            $query->cols(['*']);
            $query->where("outfitName LIKE :term");
            $query->bindValue('term', "%{$term}%");

            $data = $this->outfitRepository->fireStatementAndReturn($query);

The property outfitRepository does not seem to exist. Did you mean repository?

        }

        return $data;
    }

    /**
     * Parses a player name and makes sure it's valid
     *
     * @param  String $name
     *
     * @return ResponseInterface|boolean
     */
    public function parsePlayerName($name)

This method seems to be duplicated in your project.

    {
        if (empty($name)) {
            return $this->respondWithError('Player name needs to be present.', self::CODE_WRONG_ARGS);
        }

        if (strlen($name) > 24) {
            return $this->respondWithError('Player names cannot be longer than 24 characters.', self::CODE_WRONG_ARGS);
        }

        return true;
    }

    /**
     * Parses a outfit name and makes sure it's valid
     *
     * @param  String $name
     *
     * @return ResponseInterface|boolean
     */
    public function parseOutfitName($name)

This method seems to be duplicated in your project.

    {
        if (empty($name)) {
            return $this->respondWithError('Outfit name needs to be present.', self::CODE_WRONG_ARGS);
        }

        if (strlen($name) > 32) {
            return $this->respondWithError('Outfit names cannot be longer than 32 characters.', self::CODE_WRONG_ARGS);
        }

        return true;
    }

    /**
     * Runs checks on the player ID
     *
     * @param  string $id
     *
     * @return ResponseInterface|boolean
     */
    public function parsePlayerID($id)
    {
        if (empty($id)) {
            return $this->respondWithError('Player ID needs to be present.', self::CODE_WRONG_ARGS);
        }

        if (strlen($id > 19)) {
            return $this->respondWithError('Player ID cannot be longer than 19 characters.', self::CODE_WRONG_ARGS);
        }

        if (! is_numeric($id)) {
            return $this->respondWithError('Player ID must be numeric.', self::CODE_WRONG_ARGS);
        }

        return true;
    }
}