Nip\MailModule\Models\LinksTable\LinkTrait::incrementQueryFromSendGrid()

Complexity

Conditions	2
Paths	2

Size

Total Lines	18
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function incrementQueryFromSendGrid($notification)
    {
        $cols = ['hash', 'id_email', 'email', 'smtp-id', 'category', 'url', 'ip', 'useragent', 'clicks'];
        /** @var Insert $query */
        $query = $this->newQuery('insert');

It seems like newQuery() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $query->setCols($cols);

        $notification['hash'] = sha1($notification['id_email'] . $notification['email'] . $notification['url']);
        $values = [];
        foreach ($cols as $col) {
            $values[$col] = $notification[$col];
        }
        $values['clicks'] = 1;
        $query->data($values);
        $onDuplicate = ['clicks' => ['`clicks`+1', false]];
        $query->onDuplicate($onDuplicate);

        return $query;
    }