BiffBangPow\SSMonitor\Server\Client\ClientCommon

Complexity

Total Complexity

6

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getWarnings()	0	5	2
A	getClientName()	0	6	2
A	getClientTitle()	0	6	2

<?php

namespace BiffBangPow\SSMonitor\Server\Client;

use SilverStripe\Core\Extensible;

trait ClientCommon
{
    use Extensible;

    /**
     * Get the client name/identifier for this client module
     * @return string
     * @throws \Exception
     */
    public function getClientName(): string
    {
        if ($this->clientName === '') {
            throw new \Exception("No client name defined for monitoring client");
        }
        return $this->clientName;
    }

    /**
     * Get the friendly name for this client module
     * @return string
     * @throws \Exception
     */
    public function getClientTitle(): string
    {
        if ($this->config()->get('client_title')) {

The method config() does not exist on BiffBangPow\SSMonitor\Server\Client\ClientCommon. Since you implemented __call, consider adding a @method annotation.

            return $this->config()->get('client_title');
        }
        return $this->getClientName();
    }

    /**
     * Get an array of warning messages
     * @param $data
     * @return array|false
     */
    public function getWarnings($data)
    {
        $allWarnings = [];
        $this->extend('updateWarnings', $allWarnings, $data);
        return (count($allWarnings) > 0) ? $allWarnings : false;
    }
}