Cog\Laravel\Ban\Services\BanService

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	ban()	0	3	1
A	unban()	0	4	1
A	deleteExpiredBans()	0	8	1

<?php

/*
 * This file is part of Laravel Ban.
 *
 * (c) Anton Komarev <anton@komarev.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Cog\Laravel\Ban\Services;

use Cog\Contracts\Ban\Ban as BanContract;
use Cog\Contracts\Ban\Bannable as BannableContract;
use Cog\Contracts\Ban\BanService as BanServiceContract;
use Cog\Laravel\Ban\Models\Ban;
use Illuminate\Support\Carbon;

class BanService implements BanServiceContract
{
    /**
     * Ban entity.
     *
     * @param \Cog\Contracts\Ban\Bannable $bannable
     * @param array $attributes
     * @return \Cog\Contracts\Ban\Ban
     */
    public function ban(BannableContract $bannable, array $attributes = []): BanContract
    {
        return $bannable->bans()->create($attributes);

The method bans() does not exist on Cog\Contracts\Ban\Bannable. Did you maybe mean ban()?

    }

    /**
     * Unban entity.
     *
     * @param \Cog\Contracts\Ban\Bannable $bannable
     * @return void
     */
    public function unban(BannableContract $bannable): void
    {
        $bannable->bans->each(function ($ban) {

Accessing bans on the interface Cog\Contracts\Ban\Bannable suggest that you code against a concrete implementation. How about adding an instanceof check?

            $ban->delete();
        });
    }

    /**
     * Delete all expired Ban models.
     *
     * @return void
     */
    public function deleteExpiredBans(): void
    {
        $bans = Ban::query()
            ->where('expired_at', '<=', Carbon::now()->format('Y-m-d H:i:s'))
            ->get();

        $bans->each(function ($ban) {
            $ban->delete();
        });
    }
}