Ticaje\BookingApi\Domain\Policies\Calendar\Disabling\CQRS\Validator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	throwException()	0	3	1
B	validate()	0	20	8

<?php
declare(strict_types=1);
/**
 * Domain Policy Class
 * @package Ticaje_BookingApi
 * @author  Hector Luis Barrientos <ticaje@filetea.me>
 */

namespace Ticaje\BookingApi\Domain\Policies\Calendar\Disabling\CQRS;

use Exception;

/**
 * Trait Validator
 * @package Ticaje\BookingApi\Domain\Policies\Calendar\Disabling\CQRS
 */
trait Validator
{
    /**
     * @param $decodedRule
     * @param $type
     *
     * @return bool
     * @throws Exception
     */
    private function validate($decodedRule, $type): bool
    {
        switch ($type) {
            case 'single':
                if (isset($decodedRule['date'])) {
                    return $decodedRule;
                }
                break;
            case 'period':
                if (isset($decodedRule['from']) && isset($decodedRule['to'])) {
                    return true;
                }
                break;
            case 'recurrent_day':
                if (isset($decodedRule['dayOfWeek'])) {
                    return true;
                }
                break;
        }
        $this->throwException();

In this branch, the function will implicitly return null which is incompatible with the type-hinted return boolean. Consider adding a return statement or allowing null as return value.

    }

    /**
     * @throws Exception
     */
    private function throwException()
    {
        throw new Exception(self::VALIDATION_ERROR);

The constant Ticaje\BookingApi\Domain\Policies\Calendar\Disabling\CQRS\Validator::VALIDATION_ERROR was not found. Maybe you did not declare it correctly or list all dependencies?

    }
}