ManagesContacts - Code Metrics - Inspection of "Adds method `updateContactById`" - testmonitor/activecampaign - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#16)

by Pablo

created 2019-08-12 12:16 UTC

ManagesContacts A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	181
Duplicated Lines	15.47 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

Metric	Value
wmc	14
lcom	0
cbo	3
dl	28
loc	181
rs	10
c	0
b	0
f	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	contacts()	0	8	1
A	findContact()	0	10	1
A	createContact()	0	9	1
A	findOrCreateContact()	0	10	2
A	contactAutomations()	0	8	1
A	contactTags()	0	8	1
A	removeAutomationFromContact()	14	14	2
A	removeAllAutomationsFromContact()	0	8	2
A	removeTagFromContact()	14	14	2
A	updateContactById()	0	6	1

How to fix Duplicated Code

<?php

namespace TestMonitor\ActiveCampaign\Actions;

use TestMonitor\ActiveCampaign\Resources\Tag;
use TestMonitor\ActiveCampaign\Resources\Contact;
use TestMonitor\ActiveCampaign\Resources\Automation;
use TestMonitor\ActiveCampaign\Resources\ContactTag;
use TestMonitor\ActiveCampaign\Resources\ContactAutomation;

trait ManagesContacts
{
    /**
     * Get all contacts.
     *
     * @return array
     */
    public function contacts()
    {
        return $this->transformCollection(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->get('contacts'),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            Contact::class,
            'contacts'
        );
    }

    /**
     * Find contact by email.
     *
     * @param string $email
     *
     * @return Contact|null
     */
    public function findContact($email)
    {
        $contacts = $this->transformCollection(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->get('contacts', ['query' => ['email' => $email]]),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            Contact::class,
            'contacts'
        );

        return array_shift($contacts);
    }

    /**
     * Create new contact.
     *
     * @param string $email
     * @param string $firstName
     * @param string $lastName
     * @param int|null $orgid
     *
     * @return Contact|null
     */
    public function createContact($email, $firstName, $lastName, $orgid = null)
    {
        $contacts = $this->transformCollection(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->post('contacts', ['json' => ['contact' => compact('email', 'firstName', 'lastName', 'orgid')]]),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            Contact::class
        );

        return array_shift($contacts);
    }

    /**
     * Find or create a contact.
     *
     * @param string $email
     * @param string $firstName
     * @param string $lastName
     * @param int|null $orgid
     *
     * @return Contact
     */
    public function findOrCreateContact($email, $firstName, $lastName, $orgid = null)
    {
        $contact = $this->findContact($email);

        if ($contact instanceof Contact) {
            return $contact;
        }

        return $this->createContact($email, $firstName, $lastName, $orgid);
    }

    /**
     * Updates a contact by its ActiveCampaign ID.
     *
     * @param      $id
     * @param      $email
     * @param      $firstName
     * @param      $lastName
     * @param null $orgid
     *
     * @return Contact|null
     */
    public function updateContactById($id, $email, $firstName, $lastName, $orgid = null)
    {
        $this->put('contacts/'.$id, ['json' => ['contact' => compact('email', 'firstName', 'lastName', 'orgid')]]);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        return $this->findContactById($id);

    }

    /**
     * Get all automations of a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     *
     * @return array
     */
    public function contactAutomations(Contact $contact)
    {
        return $this->transformCollection(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->get("contacts/{$contact->id}/contactAutomations"),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ContactAutomation::class,
            'contactAutomations'
        );
    }

    /**
     * Get all tags of a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     *
     * @return array
     */
    public function contactTags(Contact $contact)
    {
        return $this->transformCollection(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->get("contacts/{$contact->id}/contactTags"),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ContactTag::class,
            'contactTags'
        );
    }

    /**
     * Removing a automation from a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     * @param \TestMonitor\ActiveCampaign\Resources\Automation $automation
     */
    public function removeAutomationFromContact(Contact $contact, Automation $automation)

    {
        $contactAutomations = $this->contactAutomations($contact);

        $contactAutomation = current(array_filter($contactAutomations, function ($contactAutomation) use ($automation) {
            return $contactAutomation->automation == $automation->id;
        }));

        if (empty($contactAutomation)) {
            return;
        }

        $this->delete("contactAutomations/{$contactAutomation->id}");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * Removing all automations from a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     */
    public function removeAllAutomationsFromContact(Contact $contact)
    {
        $contactAutomations = $this->contactAutomations($contact);

        foreach ($contactAutomations as $contactAutomation) {
            $this->delete("contactAutomations/{$contactAutomation->id}");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }
    }

    /**
     * Removing a tag from a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     * @param \TestMonitor\ActiveCampaign\Resources\Tag $tag
     */
    public function removeTagFromContact(Contact $contact, Tag $tag)

    {
        $contactTags = $this->contactTags($contact);

        $contactTag = current(array_filter($contactTags, function ($contactTag) use ($tag) {
            return $contactTag->tag == $tag->id;
        }));

        if (empty($contactTag)) {
            return;
        }

        $this->delete("contactTags/{$contactTag->id}");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1		<?php
2
3		namespace TestMonitor\ActiveCampaign\Actions;
4
5		use TestMonitor\ActiveCampaign\Resources\Tag;
6		use TestMonitor\ActiveCampaign\Resources\Contact;
7		use TestMonitor\ActiveCampaign\Resources\Automation;
8		use TestMonitor\ActiveCampaign\Resources\ContactTag;
9		use TestMonitor\ActiveCampaign\Resources\ContactAutomation;
10
11		trait ManagesContacts
12		{
13		/**
14		* Get all contacts.
15		*
16		* @return array
17		*/
18		public function contacts()
19		{
20		return $this->transformCollection(
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `transformCollection()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
21		$this->get('contacts'),
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
22		Contact::class,
23		'contacts'
24		);
25		}
26
27		/**
28		* Find contact by email.
29		*
30		* @param string $email
31		*
32		* @return Contact\|null
33		*/
34		public function findContact($email)
35		{
36		$contacts = $this->transformCollection(
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `transformCollection()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
37		$this->get('contacts', ['query' => ['email' => $email]]),
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
38		Contact::class,
39		'contacts'
40		);
41
42		return array_shift($contacts);
43		}
44
45		/**
46		* Create new contact.
47		*
48		* @param string $email
49		* @param string $firstName
50		* @param string $lastName
51		* @param int\|null $orgid
52		*
53		* @return Contact\|null
54		*/
55		public function createContact($email, $firstName, $lastName, $orgid = null)
56		{
57		$contacts = $this->transformCollection(
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `transformCollection()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
58		$this->post('contacts', ['json' => ['contact' => compact('email', 'firstName', 'lastName', 'orgid')]]),
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `post()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
59		Contact::class
60		);
61
62		return array_shift($contacts);
63		}
64
65		/**
66		* Find or create a contact.
67		*
68		* @param string $email
69		* @param string $firstName
70		* @param string $lastName
71		* @param int\|null $orgid
72		*
73		* @return Contact
74		*/
75		public function findOrCreateContact($email, $firstName, $lastName, $orgid = null)
76		{
77		$contact = $this->findContact($email);
78
79		if ($contact instanceof Contact) {
80		return $contact;
81		}
82
83		return $this->createContact($email, $firstName, $lastName, $orgid);
84		}
85
86		/**
87		* Updates a contact by its ActiveCampaign ID.
88		*
89		* @param $id
90		* @param $email
91		* @param $firstName
92		* @param $lastName
93		* @param null $orgid
94		*
95		* @return Contact\|null
96		*/
97		public function updateContactById($id, $email, $firstName, $lastName, $orgid = null)
98		{
99		$this->put('contacts/'.$id, ['json' => ['contact' => compact('email', 'firstName', 'lastName', 'orgid')]]);
		0 ignored issues – show Bug introduced 2019-08-12 12:16 UTC by Report Bug Copy Issue Report It seems like `put()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
100
101		return $this->findContactById($id);
		0 ignored issues – show Bug introduced 2019-08-12 12:16 UTC by Report Bug Copy Issue Report The method `findContactById()` does not exist on `TestMonitor\ActiveCampaign\Actions\ManagesContacts`. Did you maybe mean `findContact()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
102		}
103
104		/**
105		* Get all automations of a contact.
106		*
107		* @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
108		*
109		* @return array
110		*/
111		public function contactAutomations(Contact $contact)
112		{
113		return $this->transformCollection(
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `transformCollection()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
114		$this->get("contacts/{$contact->id}/contactAutomations"),
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
115		ContactAutomation::class,
116		'contactAutomations'
117		);
118		}
119
120		/**
121		* Get all tags of a contact.
122		*
123		* @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
124		*
125		* @return array
126		*/
127		public function contactTags(Contact $contact)
128		{
129		return $this->transformCollection(
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `transformCollection()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
130		$this->get("contacts/{$contact->id}/contactTags"),
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `get()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
131		ContactTag::class,
132		'contactTags'
133		);
134		}
135
136		/**
137		* Removing a automation from a contact.
138		*
139		* @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
140		* @param \TestMonitor\ActiveCampaign\Resources\Automation $automation
141		*/
142	View Code Duplication	public function removeAutomationFromContact(Contact $contact, Automation $automation)
		0 ignored issues – show Duplication introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
143		{
144		$contactAutomations = $this->contactAutomations($contact);
145
146		$contactAutomation = current(array_filter($contactAutomations, function ($contactAutomation) use ($automation) {
147		return $contactAutomation->automation == $automation->id;
148		}));
149
150		if (empty($contactAutomation)) {
151		return;
152		}
153
154		$this->delete("contactAutomations/{$contactAutomation->id}");
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `delete()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
155		}
156
157		/**
158		* Removing all automations from a contact.
159		*
160		* @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
161		*/
162		public function removeAllAutomationsFromContact(Contact $contact)
163		{
164		$contactAutomations = $this->contactAutomations($contact);
165
166		foreach ($contactAutomations as $contactAutomation) {
167		$this->delete("contactAutomations/{$contactAutomation->id}");
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `delete()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
168		}
169		}
170
171		/**
172		* Removing a tag from a contact.
173		*
174		* @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
175		* @param \TestMonitor\ActiveCampaign\Resources\Tag $tag
176		*/
177	View Code Duplication	public function removeTagFromContact(Contact $contact, Tag $tag)
		0 ignored issues – show Duplication introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
178		{
179		$contactTags = $this->contactTags($contact);
180
181		$contactTag = current(array_filter($contactTags, function ($contactTag) use ($tag) {
182		return $contactTag->tag == $tag->id;
183		}));
184
185		if (empty($contactTag)) {
186		return;
187		}
188
189		$this->delete("contactTags/{$contactTag->id}");
		0 ignored issues – show Bug introduced 2019-07-15 14:12 UTC by Report Bug Copy Issue Report It seems like `delete()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
190		}
191		}
192

testmonitor / activecampaign

Pull Request — master (#16)

ManagesContacts A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

10 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like