Larrock\Core\Helpers\MessageLarrock

Complexity

Total Complexity

12

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Importance

Changes	4
Bugs	1	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	success()	0	7	2
A	danger()	0	9	3
A	notice()	0	7	2
A	warning()	0	7	2
A	manual()	0	9	3

<?php

namespace Larrock\Core\Helpers;

use Larrock\Core\Events\MessageLarrockEvent;

/**
 * Вывод уведомлений в интерфейс, запись событий в лог
 * Class MessageLarrock.
 */
class MessageLarrock
{
    /**
     * @param string $message
     * @param null|bool $logWrite
     */
    public static function success(string $message, $logWrite = null)
    {
        \Session::push('message.success', $message);
        if ($logWrite) {
            \Log::info($message);
        }
        event(new MessageLarrockEvent('success', $message));
    }

    /**
     * @param string $message
     * @param null|bool $logWrite
     * @param null $exception

Are you sure the doc-type for parameter $exception is correct as it would always require null to be passed?

     * @throws \Exception
     */
    public static function danger(string $message, $logWrite = null, $exception = null)
    {
        \Session::push('message.danger', $message);
        if ($logWrite) {
            \Log::error($message);
        }
        event(new MessageLarrockEvent('danger', $message));
        if ($exception) {

$exception is of type null, thus it always evaluated to false.

            throw new \Exception($message, 500);
        }
    }

    /**
     * @param string $message
     * @param null|bool $logWrite
     */
    public static function warning(string $message, $logWrite = null)
    {
        \Session::push('message.warning', $message);
        if ($logWrite) {
            \Log::info($message);
        }
        event(new MessageLarrockEvent('warning', $message));
    }

    /**
     * @param string $message
     * @param null|bool $logWrite
     */
    public static function notice(string $message, $logWrite = null)
    {
        \Session::push('message.notice', $message);
        if ($logWrite) {
            \Log::notice($message);
        }
        event(new MessageLarrockEvent('notice', $message));
    }

    /**
     * @param string $type - тип ошибки
     * @param string $message
     * @param null|bool $logWrite
     * @param null $exception

Are you sure the doc-type for parameter $exception is correct as it would always require null to be passed?

     * @throws \Exception
     */
    public static function manual(string $type, string $message, $logWrite = null, $exception = null)
    {
        \Session::push('message.'.$type, $message);
        if ($logWrite) {
            \Log::notice($message);
        }
        event(new MessageLarrockEvent($type, $message));
        if ($exception) {

$exception is of type null, thus it always evaluated to false.

            throw new \Exception($message, 500);
        }
    }
}