Stratadox\CardGame\EventHandler\BringerOfBadNews

Complexity

Total Complexity

6

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	handle()	0	24	5

<?php declare(strict_types=1);

Class found in ".php" file; use ".inc" extension instead

This file is missing a doc comment.

The PHP open tag does not have a corresponding PHP close tag

Filename "BringerOfBadNews.php" doesn't match the expected filename "bringerofbadnews.php"

The PHP open tag must be followed by exactly one blank line

Expected 1 space before "="; 0 found

Expected 1 space after "="; 0 found

namespace Stratadox\CardGame\EventHandler;

use Stratadox\CardGame\Account\TriedOpeningAccountForUnknownEntity;
use Stratadox\CardGame\DomainEvent;
use Stratadox\CardGame\Match\TriedStartingMatchForPendingProposal;
use Stratadox\CardGame\Proposal\TriedAcceptingExpiredProposal;
use Stratadox\CardGame\Proposal\TriedAcceptingUnknownProposal;
use Stratadox\CardGame\ReadModel\Refusals;

final class BringerOfBadNews implements EventHandler

Missing doc comment for class BringerOfBadNews

Missing class doc comment

{

Opening brace should be on the same line as the declaration for class BringerOfBadNews

Opening brace should be on the same line as the declaration

    private $refusals;

Expected 1 blank line(s) before first member var; 0 found

Private member variable "refusals" must contain a leading underscore

Missing member variable doc comment

Private member variable "refusals" must be prefixed with an underscore

    public function __construct(Refusals $refusals)

Missing doc comment for function __construct()

Missing function doc comment

    {

Opening brace should be on the same line as the declaration

        $this->refusals = $refusals;
    }

Expected 1 blank line before closing function brace; 0 found

    public function handle(DomainEvent $event): void

Missing doc comment for function handle()

Missing function doc comment

    {

Opening brace should be on the same line as the declaration

        if ($event instanceof TriedOpeningAccountForUnknownEntity) {
            $this->refusals->addFor(
                $event->aggregateId(),
                'Cannot open account for unknown entity'
            );
        }

No blank line found after control structure

        if ($event instanceof TriedStartingMatchForPendingProposal) {
            $this->refusals->addFor(
                $event->aggregateId(),
                'The proposal is still pending!'
            );
        }

No blank line found after control structure

        if ($event instanceof TriedAcceptingExpiredProposal) {
            $this->refusals->addFor(
                $event->aggregateId(),
                'The proposal has already expired!'
            );
        }

No blank line found after control structure

        if ($event instanceof TriedAcceptingUnknownProposal) {
            $this->refusals->addFor(
                $event->aggregateId(),
                'Proposal not found'
            );
        }
    }

Expected 1 blank line after function; 0 found

Expected 1 blank line before closing function brace; 0 found

}

As per coding style, files should not end with a newline character.