Linkeys\UrlSigner\Middleware\CheckLinkValid::handle()

Complexity

Conditions	8
Paths	5

Size

Total Lines	22
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	8

Importance

Changes

0

<?php
        public function handle(Request $request, Closure $next)
    {

        $link = $request->get('link');

        if($link->clickLimitReached()) {
            throw new LinkClickLimitReachedException;
        }
        if($link->group && $link->group->clickLimitReached()) {
            throw new LinkGroupClickLimitReachedException;
        }
        if($link->expired()) {
            throw new LinkExpiredException;
        }

        if($link->group && $link->group->expired() && $link->expiry === null) {
            throw new LinkGroupExpiredException;
        }

        Event::dispatch(new LinkClicked($link));

It seems like $link can also be of type null; however, parameter $link of Linkeys\UrlSigner\Events\LinkClicked::__construct() does only seem to accept Linkeys\UrlSigner\Contracts\Models\Link, maybe add an additional type check?

        return $next($request);
    }
}