Passed
Push — master ( 58ac10...e3c27b )
by Adrien
05:30
created

BookableAvailable   A

Complexity

Total Complexity 12

Size/Duplication

Total Lines 66
Duplicated Lines 0 %

Test Coverage

Coverage 76%

Importance

Changes 0
Metric Value
wmc 12
eloc 25
c 0
b 0
f 0
dl 0
loc 66
ccs 19
cts 25
cp 0.76
rs 10

2 Methods

Rating   Name   Duplication   Size   Complexity  
B assert() 0 49 11
A getName() 0 3 1
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Application\Acl\Assertion;
6
7
use Application\Model\Booking;
8
use Application\Model\User;
9
use Ecodev\Felix\Acl\Assertion\NamedAssertion;
10
use Laminas\Permissions\Acl\Acl;
11
use Laminas\Permissions\Acl\Resource\ResourceInterface;
12
use Laminas\Permissions\Acl\Role\RoleInterface;
13
14
class BookableAvailable implements NamedAssertion
15
{
16
    public function getName(): string
17
    {
18
        return 'le réservable est est disponible';
19
    }
20
21
    /**
22
     * Assert that the bookable of the given booking can be rented by the current user.
23
     *
24
     * @param \Application\Acl\Acl $acl
25
     * @param RoleInterface $role
26
     * @param ResourceInterface $resource
27
     * @param string $privilege
28
     *
29
     * @return bool
30
     */
31 3
    public function assert(Acl $acl, ?RoleInterface $role = null, ?ResourceInterface $resource = null, $privilege = null)
32
    {
33
        /** @var null|Booking $booking */
34 3
        $booking = $resource->getInstance();
0 ignored issues
show
Bug introduced by
The method getInstance() does not exist on Laminas\Permissions\Acl\Resource\ResourceInterface. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

34
        /** @scrutinizer ignore-call */ 
35
        $booking = $resource->getInstance();

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
Bug introduced by
The method getInstance() does not exist on null. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

34
        /** @scrutinizer ignore-call */ 
35
        $booking = $resource->getInstance();

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
35
36 3
        if (!$booking) {
37
            return $acl->reject('the booking does not exist');
0 ignored issues
show
Bug introduced by
The method reject() does not exist on Laminas\Permissions\Acl\Acl. It seems like you code against a sub-type of Laminas\Permissions\Acl\Acl such as Ecodev\Felix\Acl\Acl. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

37
            return $acl->/** @scrutinizer ignore-call */ reject('the booking does not exist');
Loading history...
38
        }
39
40 3
        if (!User::getCurrent()) {
41
            return $acl->reject('the user is not logged in');
42
        }
43
44 3
        $bookable = $booking->getBookable();
45
46 3
        if (!$bookable) {
47
            // Booking using user's own equipment is always allowed
48 1
            return true;
49
        }
50
51 2
        if (!$bookable->isActive()) {
52
            return $acl->reject('the bookable is not active');
53
        }
54
55
        // Check that the user has ALL required licenses for the bookable
56 2
        if (!$bookable->getLicenses()->isEmpty() && User::getCurrent()->getRole() !== User::ROLE_BOOKING_ONLY) {
57 1
            $userLicenses = User::getCurrent()->getLicenses();
58
59 1
            foreach ($bookable->getLicenses() as $requiredLicense) {
60 1
                if (!$userLicenses->contains($requiredLicense)) {
61 1
                    return $acl->reject('the user does not have the required license: ' . $requiredLicense->getName());
62
                }
63
            }
64
        }
65
66 1
        if ($bookable->getSimultaneousBookingMaximum() > 0) {
67
            // Check that the bookable has no more running bookings than its maximum
68 1
            $runningBookings = _em()->getRepository(Booking::class)->findBy([
69 1
                'bookable' => $bookable,
70
                'endDate' => null,
71
            ]);
72
73 1
            $count = count($runningBookings);
74 1
            if ($count >= $bookable->getSimultaneousBookingMaximum()) {
75
                return $acl->reject('the bookable limit of simultaneous bookings has been reached: ' . $count . '/' . $bookable->getSimultaneousBookingMaximum());
76
            }
77
        }
78
79 1
        return true;
80
    }
81
}
82