eXpansion\Bundle\Maps\Plugins\Maps

Complexity

Total Complexity

9

Size/Duplication

Total Lines	98
Duplicated Lines	13.27 %

Coupling/Cohesion

Components	2
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	13	13	1
A	setStatus()	0	6	2
A	getMaps()	0	4	1
A	onMapListModified()	0	4	1
A	onExpansionMapChange()	0	4	1
A	onExpansionNextMapChange()	0	4	1
A	onBeginMap()	0	4	1
A	onEndMap()	0	4	1

<?php

namespace eXpansion\Bundle\Maps\Plugins;

use eXpansion\Framework\AdminGroups\Helpers\AdminGroups;
use eXpansion\Framework\Core\DataProviders\Listener\MapDataListenerInterface;
use eXpansion\Framework\Core\DataProviders\Listener\MatchDataListenerInterface;
use eXpansion\Framework\Core\Helpers\ChatNotification;
use eXpansion\Framework\Core\Plugins\StatusAwarePluginInterface;
use eXpansion\Framework\Core\Services\Console;
use eXpansion\Framework\Core\Storage\MapStorage;
use Maniaplanet\DedicatedServer\Connection;
use Maniaplanet\DedicatedServer\Structures\Map;


class Maps implements MatchDataListenerInterface, MapDataListenerInterface, StatusAwarePluginInterface
{
    /** @var Connection */
    protected $connection;

    /** @var Console */
    protected $console;

    /** @var AdminGroups */
    protected $adminGroups;

    /** @var bool */
    protected $enabled = true;

    /** @var MapStorage */
    protected $mapStorage;

    /** @var ChatNotification */
    protected $chatNotification;

    function __construct(

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

        Connection $connection,
        Console $console,
        AdminGroups $adminGroups,
        MapStorage $mapStorage,
        ChatNotification $chatNotification
    ) {
        $this->connection = $connection;
        $this->console = $console;
        $this->adminGroups = $adminGroups;
        $this->mapStorage = $mapStorage;
        $this->chatNotification = $chatNotification;
    }

    /**
     * Set the status of the plugin
     *
     * @param boolean $status
     *
     * @return void
     */
    public function setStatus($status)
    {
        if (!$status) {

This if statement is empty and can be removed.

        }
    }

    /**
     * @return array
     */
    public function getMaps()
    {
        return $this->mapStorage->getMaps();
    }

    /**
     * @param \Maniaplanet\DedicatedServer\Structures\Map[] $oldMaps
     * @param string $currentMapUid
     * @param string $nextMapUid
     * @param bool $isListModified
     * @return mixed|void
     */
    public function onMapListModified($oldMaps, $currentMapUid, $nextMapUid, $isListModified)
    {

    }

    public function onExpansionMapChange($currentMap, $previousMap)
    {
        // TODO: Implement onExpansionMapChange() method.
    }

    public function onExpansionNextMapChange($nextMap, $previousNextMap)
    {
        // TODO: Implement onExpansionNextMapChange() method.
    }

    /**
     * @param Map $map
     *
     * @return mixed
     */
    public function onBeginMap(Map $map)
    {
        $this->chatNotification->sendMessage('expansion_maps.chat.onbeginmap', null, ['%name%' => $map->name, '%author%' => $map->author]);
    }

    /**
     * @param Map $map
     *
     * @return mixed
     */
    public function onEndMap(Map $map)
    {
        // TODO: Implement onEndMap() method.
    }
}