prgTW\BaseCRM\Service\Contacts

Complexity

Total Complexity

3

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	6
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getEndpoint()	0	4	1
A	all()	0	4	1
B	create()	0	29	1

<?php

namespace prgTW\BaseCRM\Service;

use prgTW\BaseCRM\Resource\DetachedResource;
use prgTW\BaseCRM\Resource\Page;
use prgTW\BaseCRM\Resource\PaginatedListResource;

class Contacts extends PaginatedListResource
{
	/** {@inheritdoc} */
	protected function getEndpoint()
	{
		return self::ENDPOINT_SALES;
	}

	/**
	 * @param int $page
	 *
	 * @return Page

Should the return type not be \prgTW\BaseCRM\Resource\ResourceCollection?

	 */
	public function all($page = 1)
	{
		return parent::getPage($page);

It seems like you call parent on a different method (getPage() instead of all()). Are you sure this is correct? If so, you might want to change this to $this->getPage().

	}

	/** {@inheritdoc} */
	public function create(DetachedResource $resource, array $fieldNames = [], $useKey = true)
	{
		$fieldNames = array_intersect($fieldNames, [
			'name',
			'last_name',
			'first_name',
			'is_organisation',
			'contact_id',
			'email',
			'phone',
			'mobile',
			'twitter',
			'skype',
			'facebook',
			'linkedin',
			'address',
			'city',
			'country',
			'title',
			'description',
			'industry',
			'website',
			'fax',
			'tag_list',
			'private',
		]);

		return parent::create($resource, $fieldNames, $useKey);
	}
}