GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.

Collection::getAllEmails()   A
last analyzed

Complexity

Conditions 1
Paths 1

Size

Total Lines 10
Code Lines 7

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 7
nc 1
nop 0
dl 0
loc 10
rs 10
c 0
b 0
f 0
1
<?php
2
declare(strict_types=1);
3
/**
4
 */
5
6
namespace CommerceLeague\ActiveCampaign\Model\ResourceModel\Subscriber;
7
8
use CommerceLeague\ActiveCampaign\Setup\SchemaInterface;
9
use Magento\Framework\DB\Select;
10
use Magento\Newsletter\Model\ResourceModel\Subscriber\Collection as ExtendSubscriberCollection;
11
12
/**
13
 * Class Collection
14
 * @codeCoverageIgnore
15
 */
16
class Collection extends ExtendSubscriberCollection
17
{
18
    /**
19
     * @inheritDoc
20
     */
21
    protected function _initSelect()
22
    {
23
        parent::_initSelect();
24
25
        $this->getSelect()->joinLeft(
26
            ['ac_contact' => $this->_resource->getTable(SchemaInterface::CONTACT_TABLE)],
0 ignored issues
show
Unused Code introduced by
The call to Magento\Framework\DB\Select::joinLeft() has too many arguments starting with array('ac_contact' => $t...erface::CONTACT_TABLE)). ( Ignorable by Annotation )

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

26
        $this->getSelect()->/** @scrutinizer ignore-call */ joinLeft(

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
27
            'ac_contact.email = main_table.subscriber_email',
28
            ['ac_contact.activecampaign_id']
29
        );
30
31
        return $this;
32
    }
33
34
    /**
35
     * @return Collection
36
     */
37
    public function excludeCustomers(): self
38
    {
39
        $this->getSelect()->where('main_table.customer_id = 0');
40
        return $this;
41
    }
42
43
    /**
44
     * @param string $email
45
     * @return Collection
46
     */
47
    public function addEmailFilter(string $email): self
48
    {
49
        $this->getSelect()->where('main_table.subscriber_email = ?', $email);
50
        return $this;
51
    }
52
53
    /**
54
     * @return Collection
55
     */
56
    public function addContactOmittedFilter(): self
57
    {
58
        $this->getSelect()->where('ac_contact.activecampaign_id IS NULL');
59
        return $this;
60
    }
61
62
    /**
63
     * @return array
64
     */
65
    public function getAllEmails(): array
66
    {
67
        $emailsSelect = clone $this->getSelect();
68
        $emailsSelect->reset(Select::ORDER);
69
        $emailsSelect->reset(Select::LIMIT_COUNT);
70
        $emailsSelect->reset(Select::LIMIT_OFFSET);
71
        $emailsSelect->reset(Select::COLUMNS);
72
        $emailsSelect->columns('subscriber_email', 'main_table');
0 ignored issues
show
Unused Code introduced by
The call to Magento\Framework\DB\Select::columns() has too many arguments starting with 'subscriber_email'. ( Ignorable by Annotation )

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

72
        $emailsSelect->/** @scrutinizer ignore-call */ 
73
                       columns('subscriber_email', 'main_table');

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
73
74
        return $this->getConnection()->fetchCol($emailsSelect, $this->_bindParams);
75
    }
76
}
77