PresenceService::togglePresence() - Code Metrics - Inspection of "Merge branch feature/members-presence-at-sessions" - lagdo/tontine - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — main (#55)

by Thierry

created 2024-01-15 01:52 UTC

PresenceService::togglePresence() A

↳ Parent: PresenceService

Complexity

Conditions	2
Paths	2

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	3
nc	2
nop	2
dl	0
loc	5
rs	10
c	1
b	0
f	0

<?php

namespace Siak\Tontine\Service\Meeting;

use Illuminate\Support\Collection;
use Siak\Tontine\Model\Member;
use Siak\Tontine\Model\Round;
use Siak\Tontine\Model\Session;
use Siak\Tontine\Service\TenantService;
use Siak\Tontine\Service\Tontine\MemberService;

class PresenceService
{
    /**
     * @param TenantService $tenantService
     */
    public function __construct(private TenantService $tenantService,
        private MemberService $memberService, private SessionService $sessionService)
    {}

    /**
     * @param Round $round
     *
     * @return Collection
     */
    public function getRoundAbsences(Round $round): Collection
    {
        return Member::select('members.id', 'members.name', 'absences.session_id')

            ->join('absences', 'absences.member_id', '=', 'members.id')
            ->join('sessions', 'absences.session_id', '=', 'sessions.id')
            ->where('sessions.round_id', $round->id)
            ->get()
            // Group the data (member name) by session id and member id.
            ->groupBy('session_id')
            ->map(fn($members) => $members->groupBy('id')
                ->map(fn($_members) => $_members->first()->name));
    }

    /**
     * @param Session $session
     *
     * @return Collection
     */
    public function getSessionAbsences(Session $session): Collection
    {
        return $session->absents()->pluck('name', 'id');
    }

    /**
     * @param Member $member
     *
     * @return Collection
     */
    public function getMemberAbsences(Member $member): Collection
    {
        $round = $this->tenantService->round();
        return $member->absences()->where('round_id', $round->id)->pluck('title', 'id');
    }

    /**
     * @param Session $session
     * @param Member $member
     *
     * @return void
     */
    public function togglePresence(Session $session, Member $member)
    {
        !$session->absents()->find($member->id) ?
            $session->absents()->attach($member->id) :
            $session->absents()->detach($member->id);
    }

    /**
     * Get the number of sessions in the selected round.
     *
     * @return int
     */
    public function getSessionCount(): int
    {
        return $this->sessionService->active()->getSessionCount();
    }

    /**
     * Get a paginated list of sessions in the selected round.
     *
     * @param int $page
     *
     * @return Collection
     */
    public function getSessions(int $page = 0): Collection
    {
        return $this->sessionService->active()
            ->withCount(['absents'])
            ->getSessions($page, true);
    }

    /**
     * Find a session.
     *
     * @param int $sessionId
     *
     * @return Session|null
     */
    public function getSession(int $sessionId): ?Session
    {
        return $this->sessionService->active()
            ->withCount(['absents'])
            ->getSession($sessionId);
    }

    /**
     * Get the number of members.
     *
     * @param string $search
     *
     * @return int
     */
    public function getMemberCount(string $search = ''): int
    {
        return $this->memberService->active()->getMemberCount($search);
    }

    /**
     * Get a paginated list of members.
     *
     * @param string $search
     * @param int $page
     *
     * @return Collection
     */
    public function getMembers(string $search, int $page = 0): Collection
    {
        $round = $this->tenantService->round();
        return $this->memberService->active()
            ->withCount(['absences' => fn($query) => $query->where('round_id', $round->id)])
            ->getMembers($search, $page);
    }

    /**
     * Find a member.
     *
     * @param int $memberId
     *
     * @return Member|null
     */
    public function getMember(int $memberId): ?Member
    {
        $round = $this->tenantService->round();
        return $this->memberService->active()
            ->withCount(['absences' => fn($query) => $query->where('round_id', $round->id)])
            ->getMember($memberId);
    }
}


1			<?php
2
3			namespace Siak\Tontine\Service\Meeting;
4
5			use Illuminate\Support\Collection;
6			use Siak\Tontine\Model\Member;
7			use Siak\Tontine\Model\Round;
8			use Siak\Tontine\Model\Session;
9			use Siak\Tontine\Service\TenantService;
10			use Siak\Tontine\Service\Tontine\MemberService;
11
12			class PresenceService
13			{
14			/**
15			* @param TenantService $tenantService
16			*/
17			public function __construct(private TenantService $tenantService,
18			private MemberService $memberService, private SessionService $sessionService)
19			{}
20
21			/**
22			* @param Round $round
23			*
24			* @return Collection
25			*/
26			public function getRoundAbsences(Round $round): Collection
27			{
28			return Member::select('members.id', 'members.name', 'absences.session_id')
			0 ignored issues – show Bug Best Practice introduced 2024-01-15 01:57 UTC by Report Bug Copy Issue Report The expression `return Siak\Tontine\Mode...ion(...) { /* ... */ })` could return the type `Illuminate\Database\Eloq...Relations\HasOneThrough` which is incompatible with the type-hinted return `Illuminate\Support\Collection`. Consider adding an additional type-check to rule them out. Loading history...
29			->join('absences', 'absences.member_id', '=', 'members.id')
30			->join('sessions', 'absences.session_id', '=', 'sessions.id')
31			->where('sessions.round_id', $round->id)
32			->get()
33			// Group the data (member name) by session id and member id.
34			->groupBy('session_id')
35			->map(fn($members) => $members->groupBy('id')
36			->map(fn($_members) => $_members->first()->name));
37			}
38
39			/**
40			* @param Session $session
41			*
42			* @return Collection
43			*/
44			public function getSessionAbsences(Session $session): Collection
45			{
46			return $session->absents()->pluck('name', 'id');
47			}
48
49			/**
50			* @param Member $member
51			*
52			* @return Collection
53			*/
54			public function getMemberAbsences(Member $member): Collection
55			{
56			$round = $this->tenantService->round();
57			return $member->absences()->where('round_id', $round->id)->pluck('title', 'id');
58			}
59
60			/**
61			* @param Session $session
62			* @param Member $member
63			*
64			* @return void
65			*/
66			public function togglePresence(Session $session, Member $member)
67			{
68			!$session->absents()->find($member->id) ?
69			$session->absents()->attach($member->id) :
70			$session->absents()->detach($member->id);
71			}
72
73			/**
74			* Get the number of sessions in the selected round.
75			*
76			* @return int
77			*/
78			public function getSessionCount(): int
79			{
80			return $this->sessionService->active()->getSessionCount();
81			}
82
83			/**
84			* Get a paginated list of sessions in the selected round.
85			*
86			* @param int $page
87			*
88			* @return Collection
89			*/
90			public function getSessions(int $page = 0): Collection
91			{
92			return $this->sessionService->active()
93			->withCount(['absents'])
94			->getSessions($page, true);
95			}
96
97			/**
98			* Find a session.
99			*
100			* @param int $sessionId
101			*
102			* @return Session\|null
103			*/
104			public function getSession(int $sessionId): ?Session
105			{
106			return $this->sessionService->active()
107			->withCount(['absents'])
108			->getSession($sessionId);
109			}
110
111			/**
112			* Get the number of members.
113			*
114			* @param string $search
115			*
116			* @return int
117			*/
118			public function getMemberCount(string $search = ''): int
119			{
120			return $this->memberService->active()->getMemberCount($search);
121			}
122
123			/**
124			* Get a paginated list of members.
125			*
126			* @param string $search
127			* @param int $page
128			*
129			* @return Collection
130			*/
131			public function getMembers(string $search, int $page = 0): Collection
132			{
133			$round = $this->tenantService->round();
134			return $this->memberService->active()
135			->withCount(['absences' => fn($query) => $query->where('round_id', $round->id)])
136			->getMembers($search, $page);
137			}
138
139			/**
140			* Find a member.
141			*
142			* @param int $memberId
143			*
144			* @return Member\|null
145			*/
146			public function getMember(int $memberId): ?Member
147			{
148			$round = $this->tenantService->round();
149			return $this->memberService->active()
150			->withCount(['absences' => fn($query) => $query->where('round_id', $round->id)])
151			->getMember($memberId);
152			}
153			}
154

lagdo / tontine

Pull Request — main (#55)

PresenceService::togglePresence() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like