LeagueOverseerHookController::matchReportAction() - Code Metrics - allejo/bzion - Measure and Improve Code Quality continuously with Scrutinizer

LeagueOverseerHookController::matchReportAction() F
last analyzed 2018-04-10 03:56 UTC

↳ Parent: LeagueOverseerHookController

Complexity

Conditions	20
Paths	266

Size

Total Lines	93
Code Lines	64

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	55
CRAP Score	20.0021

Importance

Changes

Metric	Value
dl	0
loc	93
ccs	55
cts	56
cp	0.9821
rs	3.6338
c	0
b	0
f	0
cc	20
eloc	64
nc	266
nop	2
crap	20.0021

How to fix Long Method Complexity

<?php

use Monolog\Logger;
use Nelmio\ApiDocBundle\Annotation\ApiDoc;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;

class LeagueOverseerHookController extends PlainTextController
{
    /**
     * The API version of the server performing the request
     * @var int
     */
    private $version;

    /**
     * The parameter bag representing the $_GET or $_POST array
     * @var Symfony\Component\HttpFoundation\ParameterBag
     */
    private $params;

    /**
     * {@inheritdoc}
     */
    public function setUp()
    {
        $request = $this->getRequest();

        // To prevent abuse of the automated system, we need to make sure that
        // the IP making the request is one of the IPs we allowed in the config
        $allowedIPs = array_map('trim', $this->container->getParameter('bzion.api.allowed_ips'));
        $clientIP   = $request->getClientIp();

        if (!$this->isDebug() && // Don't care about IPs if we're in debug mode
           !in_array($clientIP, $allowedIPs)) {
            // If server making the request isn't an official server, then log the unauthorized attempt and kill the script

            $this->getLogger()->addNotice("Unauthorized access attempt from $clientIP");
            throw new ForbiddenException("Error: 403 - Forbidden");
        }

        // We will be looking at either $_POST or $_GET depending on the status, production or development
        $this->params = $request->request; // $_POST

        if (!$this->params->has('query')) {
            // There seems to be nothing in $_POST. If we are in debug mode
            // however, we might have a debug request with data in $_GET
            if ($this->isDebug() && $request->query->has('query')) {
                $this->params = $request->query; // $_GET
            } else {
                throw new BadRequestException();
            }
        }

        // After the first major rewrite of the league overseer plugin, the
        // API was introduced in order to provide backwards compatibility for
        // servers that have not updated to the latest version of the plugin.
        $this->version = $this->params->get('apiVersion', 0);
    }

    /**
     * @ApiDoc(
     *  description="Query the LeagueOverseer API",
     *  parameters={
     *      {"name"="query", "dataType"="string", "required"=true, "description"="query type"},
     *      {"name"="apiVersion", "dataType"="integer", "required"=false, "description"="LeagueOverseer API version"}
     *  }
     * )
     * @todo Test/improve/revoke support for API version 0
     */
    public function queryAction()
    {
        $matchReportQuery = $this->version == 1 ? 'reportMatch' : 'matchReport';
        $teamNameQuery = $this->version == 1 ? 'teamNameQuery' : 'teamName';
        $teamNameDumpQuery = $this->version == 1 ? 'teamDump' : 'teamInfoDump';

        switch ($this->params->get('query')) {
            case $matchReportQuery:
                return $this->forward('matchReport');
            case $teamNameQuery:
                return $this->forward('teamName');
            case $teamNameDumpQuery:
                return $this->forward('teamNameDump');
            default:
                throw new BadRequestException();
            }
    }

    public function teamNameAction()
    {
        if ($this->version < 1) {
            throw new BadRequestException();
        }

        $param = $this->version == 1 ? 'teamPlayers' : 'bzid';

        $bzid = $this->params->get($param);
        $team = Player::getFromBZID($bzid)->getTeam();

        $teamName = ($team->isValid()) ? preg_replace("/&[^\s]*;/", "", $team->getName()) : '';

        return new JsonResponse(array(
            // API v1 legacy support
            "bzid" => $bzid,     // Replaced with "team_name" in API v2+
            "team" => $teamName, // Replaced with "player_bzid" in API v2+

            // API v2+
            "player_bzid" => $bzid,
            "team_id"     => $team->getId(),
            "team_name"   => $teamName,
        ));
    }

    public function teamNameDumpAction()
    {
        if ($this->version < 1) {
            throw new BadRequestException();
        }

        // Create an array to store all teams and the BZIDs
        $teamArray = array();

        foreach (Team::getTeams() as $team) {
            $memberList = "";

            foreach ($team->getMembers() as $member) {
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
                $memberList .= $member->getBZID() . ",";
            }

            $teamName    = preg_replace("/&[^\s]*;/", "", $team->getName());
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            $teamID      = $team->getId();
            $teamMembers = rtrim($memberList, ",");

            $teamArray[] = array(
                // API v1 legacy support
                "team"    => $teamName,
                "members" => $teamMembers,

                // API v2+
                "team_id"      => $teamID,
                "team_name"    => $teamName,
                "team_members" => $teamMembers
            );
        }

        return new JsonResponse(array(
            // API v1 legacy support
            "teamDump" => &$teamArray,

            // API v2+
            "team_list" => &$teamArray
        ));
    }

    public function matchReportAction(Logger $log, Request $request)
    {
        $log->addNotice("Match data received from " . $request->getClientIp());
        $log->addDebug("Debug match data query: " . http_build_query($this->params->all()));

        $matchType = $this->params->get('matchType', Match::OFFICIAL);

        $teamOnePlayers = $this->bzidsToIdArray('teamOnePlayers');
        $teamTwoPlayers = $this->bzidsToIdArray('teamTwoPlayers');

        $teamOne = $teamTwo = null;

        if (Match::OFFICIAL === $matchType) {
            $teamOne = $this->getTeam($teamOnePlayers);
            $teamTwo = $this->getTeam($teamTwoPlayers);

            if ($teamOne->isValid() && $teamTwo->isValid() && $teamOne->isSameAs($teamTwo)) {
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
                $log->addNotice("The '" . $teamOne->getName() . "' team played against each other in an official match. Match invalidated.");
                throw new ForbiddenException("Holy sanity check, Batman! The same team can't play against each other in an official match.");
            }
        } elseif (Match::FUN === $matchType) {
            if (count($teamOnePlayers)  < 2 || count($teamTwoPlayers) < 2) {
                throw new ForbiddenException("You are not allowed to report a match with less than 2 players per team.");
            }
        }

        $map = Map::fetchFromAlias($this->params->get('mapPlayed'));
        $server = Server::fetchFromAddress($this->params->get('server'));

        $match = Match::enterMatch(
            ($teamOne !== null) ? $teamOne->getId() : null,
            ($teamTwo !== null) ? $teamTwo->getId() : null,
            $this->params->get('teamOneWins'),
            $this->params->get('teamTwoWins'),
            $this->params->get('duration'),
            null,
            $this->params->get('matchTime'),
            $teamOnePlayers,
            $teamTwoPlayers,
            $this->params->get('server'),
            $this->params->get('replayFile'),
            $map->getId(),
            $this->params->get('matchType'),
            $this->params->get('teamOneColor'),
            $this->params->get('teamTwoColor'),
            $this->explodeQueryParam('teamOneIPs'),
            $this->explodeQueryParam('teamTwoIPs'),
            $this->explodeQueryParam('teamOneCallsigns'),
            $this->explodeQueryParam('teamTwoCallsigns')
        );

        if ($server->isValid()) {
            $match->setServer($server->getId());
        }

        $log->addNotice("Match reported automatically", array(
            'winner' => array(
                'name'  => $match->getWinner()->getName(),
                'score' => $match->getScore($match->getWinner()),
            ),
            'loser' => array(
                'name'  => $match->getLoser()->getName(),
                'score' => $match->getScore($match->getLoser())
            ),
            'eloDiff' => $match->getEloDiff(),
            'type'    => $matchType,
            'map'     => $map->getName()
        ));

        $bzfsAnnouncement = $match->getName();

        if (!$match->getWinner()->supportsMatchCount() || !$match->getLoser()->supportsMatchCount()) {
            if ($match->getWinner()->supportsMatchCount()) {
                $bzfsAnnouncement .= sprintf("\n  %s: +%d", $match->getWinner()->getName(), $match->getEloDiff());
            } elseif ($match->getLoser()->supportsMatchCount()) {
                $diff = -$match->getEloDiff();
                $bzfsAnnouncement .= sprintf("\n  %s: %d", $match->getLoser()->getName(), $diff);
            }
        }

        if ($match->isOfficial()) {
            $inverseSymbol = (
                $match->getTeamMatchType() === Match::TEAM_V_TEAM && $match->isDraw() &&
                ($match->getWinner()->isSameAs($match->getTeamB()) || $match->getPlayerEloDiff(false) < 0)
            );

            $symbol = ($inverseSymbol) ? '-/+' : '+/-';
            $bzfsAnnouncement .= sprintf("\n  player elo: %s %d", $symbol, $match->getPlayerEloDiff());
        }

        // Output the match stats that will be sent back to BZFS
        return $bzfsAnnouncement;
    }

    /**
     * Split a query parameter with a delimiter.
     *
     * @param string $parameter The query parameter to get and split
     * @param string $delimiter A delimiter for splitting the string
     *
     * @return array
     */
    private function explodeQueryParam($parameter, $delimiter = ',')
    {
        $split = explode($delimiter, $this->params->get($parameter, ''));

        if (!is_array($split)) {
            return [];
        }

        return $split;
    }

    /**
     * Convert a comma-separated list of bzids to player IDs so we can pass them to Match::enterMatch()
     *
     * @param  string $queryParam The query parameter storing a comma-separated list of BZIDs
     *
     * @return int[]  A list of Player IDs
     */
    private function bzidsToIdArray($queryParam)
    {
        $players = $this->explodeQueryParam($queryParam);

        foreach ($players as &$player) {
            $player = Player::getFromBZID($player)->getId();
        }

        return $players;
    }

    /**
     * Queries the database to get the team which a conversation of players belong to
     *
     * @param  int[] $players The IDs of players
     * @return Team  The team
     */
    private function getTeam($players)
    {
        $team = null;

        foreach ($players as $id) {
            $player = Player::get($id);

            if ($player->isTeamless()) {
                return Team::invalid();
            } elseif ($team == null) {
                $team = $player->getTeam();
            } elseif ($team->getId() != $player->getTeam()->getId()) {
                // This player is on a different team from the previous player!
                return Team::invalid();
            }
        }

        return $team;
    }

    /**
     * {@inheritdoc}
     */
    protected static function getLogChannel()
    {
        return 'api';
    }
}


LeagueOverseerHookController::matchReportAction() F
last analyzed 2018-04-10 03:56 UTC

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Available Fixes

1		<?php
2
3		use Monolog\Logger;
4		use Nelmio\ApiDocBundle\Annotation\ApiDoc;
5		use Symfony\Component\HttpFoundation\JsonResponse;
6		use Symfony\Component\HttpFoundation\Request;
7
8		class LeagueOverseerHookController extends PlainTextController
9		{
10		/**
11		* The API version of the server performing the request
12		* @var int
13		*/
14		private $version;
15
16		/**
17		* The parameter bag representing the $_GET or $_POST array
18		* @var Symfony\Component\HttpFoundation\ParameterBag
19		*/
20		private $params;
21
22		/**
23		* {@inheritdoc}
24		*/
25	22	public function setUp()
26		{
27	22	$request = $this->getRequest();
28
29		// To prevent abuse of the automated system, we need to make sure that
30		// the IP making the request is one of the IPs we allowed in the config
31	22	$allowedIPs = array_map('trim', $this->container->getParameter('bzion.api.allowed_ips'));
32	22	$clientIP = $request->getClientIp();
33
34	22	if (!$this->isDebug() && // Don't care about IPs if we're in debug mode
35	22	!in_array($clientIP, $allowedIPs)) {
36		// If server making the request isn't an official server, then log the unauthorized attempt and kill the script
37
38		$this->getLogger()->addNotice("Unauthorized access attempt from $clientIP");
39		throw new ForbiddenException("Error: 403 - Forbidden");
40		}
41
42		// We will be looking at either $_POST or $_GET depending on the status, production or development
43	22	$this->params = $request->request; // $_POST
44
45	22	if (!$this->params->has('query')) {
46		// There seems to be nothing in $_POST. If we are in debug mode
47		// however, we might have a debug request with data in $_GET
48		if ($this->isDebug() && $request->query->has('query')) {
49		$this->params = $request->query; // $_GET
50		} else {
51		throw new BadRequestException();
52		}
53		}
54
55		// After the first major rewrite of the league overseer plugin, the
56		// API was introduced in order to provide backwards compatibility for
57		// servers that have not updated to the latest version of the plugin.
58	22	$this->version = $this->params->get('apiVersion', 0);
59	22	}
60
61		/**
62		* @ApiDoc(
63		* description="Query the LeagueOverseer API",
64		* parameters={
65		* {"name"="query", "dataType"="string", "required"=true, "description"="query type"},
66		* {"name"="apiVersion", "dataType"="integer", "required"=false, "description"="LeagueOverseer API version"}
67		* }
68		* )
69		* @todo Test/improve/revoke support for API version 0
70		*/
71	22	public function queryAction()
72		{
73	22	$matchReportQuery = $this->version == 1 ? 'reportMatch' : 'matchReport';
74	22	$teamNameQuery = $this->version == 1 ? 'teamNameQuery' : 'teamName';
75	22	$teamNameDumpQuery = $this->version == 1 ? 'teamDump' : 'teamInfoDump';
76
77	22	switch ($this->params->get('query')) {
78	22	case $matchReportQuery:
79	22	return $this->forward('matchReport');
80		case $teamNameQuery:
81		return $this->forward('teamName');
82		case $teamNameDumpQuery:
83		return $this->forward('teamNameDump');
84		default:
85		throw new BadRequestException();
86		}
87		}
88
89		public function teamNameAction()
90		{
91		if ($this->version < 1) {
92		throw new BadRequestException();
93		}
94
95		$param = $this->version == 1 ? 'teamPlayers' : 'bzid';
96
97		$bzid = $this->params->get($param);
98		$team = Player::getFromBZID($bzid)->getTeam();
99
100		$teamName = ($team->isValid()) ? preg_replace("/&[^\s]*;/", "", $team->getName()) : '';
101
102		return new JsonResponse(array(
103		// API v1 legacy support
104		"bzid" => $bzid, // Replaced with "team_name" in API v2+
105		"team" => $teamName, // Replaced with "player_bzid" in API v2+
106
107		// API v2+
108		"player_bzid" => $bzid,
109		"team_id" => $team->getId(),
110		"team_name" => $teamName,
111		));
112		}
113
114		public function teamNameDumpAction()
115		{
116		if ($this->version < 1) {
117		throw new BadRequestException();
118		}
119
120		// Create an array to store all teams and the BZIDs
121		$teamArray = array();
122
123		foreach (Team::getTeams() as $team) {
124		$memberList = "";
125
126		foreach ($team->getMembers() as $member) {
		0 ignored issues – show Bug introduced 2016-08-28 12:41 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Model` as the method `getMembers()` does only exist in the following sub-classes of `Model`: `Conversation`, `Team`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
127		$memberList .= $member->getBZID() . ",";
128		}
129
130		$teamName = preg_replace("/&[^\s]*;/", "", $team->getName());
		0 ignored issues – show Bug introduced 2016-08-28 12:41 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Model` as the method `getName()` does only exist in the following sub-classes of `Model`: `AliasModel`, `ApiKey`, `AvatarModel`, `Ban`, `Conversation`, `Country`, `Map`, `Match`, `News`, `NewsCategory`, `Page`, `Permission`, `Player`, `Role`, `Server`, `Team`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
131		$teamID = $team->getId();
132		$teamMembers = rtrim($memberList, ",");
133
134		$teamArray[] = array(
135		// API v1 legacy support
136		"team" => $teamName,
137		"members" => $teamMembers,
138
139		// API v2+
140		"team_id" => $teamID,
141		"team_name" => $teamName,
142		"team_members" => $teamMembers
143		);
144		}
145
146		return new JsonResponse(array(
147		// API v1 legacy support
148		"teamDump" => &$teamArray,
149
150		// API v2+
151		"team_list" => &$teamArray
152		));
153		}
154
155	22	public function matchReportAction(Logger $log, Request $request)
156		{
157	22	$log->addNotice("Match data received from " . $request->getClientIp());
158	22	$log->addDebug("Debug match data query: " . http_build_query($this->params->all()));
159
160	22	$matchType = $this->params->get('matchType', Match::OFFICIAL);
161
162	22	$teamOnePlayers = $this->bzidsToIdArray('teamOnePlayers');
163	22	$teamTwoPlayers = $this->bzidsToIdArray('teamTwoPlayers');
164
165	22	$teamOne = $teamTwo = null;
166	22
167		if (Match::OFFICIAL === $matchType) {
168	22	$teamOne = $this->getTeam($teamOnePlayers);
169		$teamTwo = $this->getTeam($teamTwoPlayers);
170	22
171	21	if ($teamOne->isValid() && $teamTwo->isValid() && $teamOne->isSameAs($teamTwo)) {
		0 ignored issues – show Bug introduced 2017-08-21 01:41 UTC by Report Bug Copy Issue Report It seems like `$teamTwo` defined by `$this->getTeam($teamTwoPlayers)` on line `169` can be `null`; however, `Model::isSameAs()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
172	21	$log->addNotice("The '" . $teamOne->getName() . "' team played against each other in an official match. Match invalidated.");
173		throw new ForbiddenException("Holy sanity check, Batman! The same team can't play against each other in an official match.");
174	21	}
175	1	} elseif (Match::FUN === $matchType) {
176	21	if (count($teamOnePlayers) < 2 \|\| count($teamTwoPlayers) < 2) {
177		throw new ForbiddenException("You are not allowed to report a match with less than 2 players per team.");
178	1	}
179	1	}
180
181		$map = Map::fetchFromAlias($this->params->get('mapPlayed'));
182		$server = Server::fetchFromAddress($this->params->get('server'));
183
184	21	$match = Match::enterMatch(
185		($teamOne !== null) ? $teamOne->getId() : null,
186	21	($teamTwo !== null) ? $teamTwo->getId() : null,
187	21	$this->params->get('teamOneWins'),
188	21	$this->params->get('teamTwoWins'),
189	21	$this->params->get('duration'),
190	21	null,
191	21	$this->params->get('matchTime'),
192	21	$teamOnePlayers,
193	21	$teamTwoPlayers,
194	21	$this->params->get('server'),
195	21	$this->params->get('replayFile'),
196	21	$map->getId(),
197	21	$this->params->get('matchType'),
198	21	$this->params->get('teamOneColor'),
199	21	$this->params->get('teamTwoColor'),
200	21	$this->explodeQueryParam('teamOneIPs'),
201	21	$this->explodeQueryParam('teamTwoIPs'),
202		$this->explodeQueryParam('teamOneCallsigns'),
203		$this->explodeQueryParam('teamTwoCallsigns')
204	21	);
205
206	21	if ($server->isValid()) {
207	21	$match->setServer($server->getId());
208		}
209
210	21	$log->addNotice("Match reported automatically", array(
211	21	'winner' => array(
212		'name' => $match->getWinner()->getName(),
213	21	'score' => $match->getScore($match->getWinner()),
214	21	),
215	21	'loser' => array(
216		'name' => $match->getLoser()->getName(),
217		'score' => $match->getScore($match->getLoser())
218	21	),
219		'eloDiff' => $match->getEloDiff(),
220	21	'type' => $matchType,
221	13	'map' => $map->getName()
222	4	));
223	9
224	4	$bzfsAnnouncement = $match->getName();
225	4
226		if (!$match->getWinner()->supportsMatchCount() \|\| !$match->getLoser()->supportsMatchCount()) {
227		if ($match->getWinner()->supportsMatchCount()) {
228		$bzfsAnnouncement .= sprintf("\n %s: +%d", $match->getWinner()->getName(), $match->getEloDiff());
229	21	} elseif ($match->getLoser()->supportsMatchCount()) {
230		$diff = -$match->getEloDiff();
231	20	$bzfsAnnouncement .= sprintf("\n %s: %d", $match->getLoser()->getName(), $diff);
232	20	}
233		}
234
235	20	if ($match->isOfficial()) {
236	20	$inverseSymbol = (
237		$match->getTeamMatchType() === Match::TEAM_V_TEAM && $match->isDraw() &&
238		($match->getWinner()->isSameAs($match->getTeamB()) \|\| $match->getPlayerEloDiff(false) < 0)
239		);
240	21
241		$symbol = ($inverseSymbol) ? '-/+' : '+/-';
242		$bzfsAnnouncement .= sprintf("\n player elo: %s %d", $symbol, $match->getPlayerEloDiff());
243		}
244
245		// Output the match stats that will be sent back to BZFS
246		return $bzfsAnnouncement;
247		}
248
249		/**
250	22	* Split a query parameter with a delimiter.
251		*
252	22	* @param string $parameter The query parameter to get and split
253		* @param string $delimiter A delimiter for splitting the string
254	22	*
255	22	* @return array
256		*/
257		private function explodeQueryParam($parameter, $delimiter = ',')
258	22	{
259		$split = explode($delimiter, $this->params->get($parameter, ''));
260
261		if (!is_array($split)) {
262		return [];
263		}
264
265		return $split;
266		}
267	21
268		/**
269	21	* Convert a comma-separated list of bzids to player IDs so we can pass them to Match::enterMatch()
270		*
271	21	* @param string $queryParam The query parameter storing a comma-separated list of BZIDs
272	21	*
273		* @return int[] A list of Player IDs
274	21	*/
275	12	private function bzidsToIdArray($queryParam)
276	21	{
277	21	$players = $this->explodeQueryParam($queryParam);
278	17
279		foreach ($players as &$player) {
280	21	$player = Player::getFromBZID($player)->getId();
281		}
282
283		return $players;
284	17	}
285
286		/**
287		* Queries the database to get the team which a conversation of players belong to
288		*
289		* @param int[] $players The IDs of players
290	22	* @return Team The team
291		*/
292	22	private function getTeam($players)
293		{
294		$team = null;
295
296		foreach ($players as $id) {
297		$player = Player::get($id);
298
299		if ($player->isTeamless()) {
300		return Team::invalid();
301		} elseif ($team == null) {
302		$team = $player->getTeam();
303		} elseif ($team->getId() != $player->getTeam()->getId()) {
304		// This player is on a different team from the previous player!
305		return Team::invalid();
306		}
307		}
308
309		return $team;
310		}
311
312		/**
313		* {@inheritdoc}
314		*/
315		protected static function getLogChannel()
316		{
317		return 'api';
318		}
319		}
320

allejo / bzion

LeagueOverseerHookController::matchReportAction() F last analyzed 2018-04-10 03:56 UTC

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

LeagueOverseerHookController::matchReportAction() F
last analyzed 2018-04-10 03:56 UTC