SilverStripe\Auditor\AuditHookMemberGroupSet::removeByID()

Complexity

Conditions	8
Paths	5

Size

Total Lines	28
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function removeByID($itemID)
    {
        parent::removeByID($itemID);

        if ($this->getJoinTable() === 'Group_Members') {
            $currentMember = Security::getCurrentUser();
            if (!$currentMember || !$currentMember->exists()) {

$currentMember is of type SilverStripe\Security\Member, thus it always evaluated to true.

                return;
            }

            $group = Group::get()->byId($itemID);
            $member = Member::get()->byId($this->getForeignID());

$this->getForeignID() of type string is incompatible with the type integer expected by parameter $id of SilverStripe\ORM\DataList::byID().

            if (!$group) {

$group is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

                return;
            }
            if (!$member) {

$member is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

                return;
            }

            $this->getAuditLogger()->info(sprintf(
                '"%s" (ID: %s) removed Member "%s" (ID: %s) from Group "%s" (ID: %s)',
                $currentMember->Email ?: $currentMember->Title,
                $currentMember->ID,
                $member->Email ?: $member->Title,
                $member->ID,
                $group->Title,
                $group->ID
            ));