ClientRemote::deleteUser() - Code Metrics - Inspection of "WIP: add uploading file" - directus/directus-sdk-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — d64 ( f52daa...138eb5 )

by Welling

created 2016-11-18 16:16 UTC

ClientRemote::deleteUser() A

↳ Parent: ClientRemote

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	4
ccs	0
cts	0
cp	0
crap	2
rs	10
c	0
b	0
f	0

<?php

/**
 * Directus – <http://getdirectus.com>
 *
 * @link      The canonical repository – <https://github.com/directus/directus>
 * @copyright Copyright 2006-2016 RANGER Studio, LLC – <http://rangerstudio.com>
 * @license   GNU General Public License (v3) – <http://www.gnu.org/copyleft/gpl.html>
 */

namespace Directus\SDK;
use Directus\Util\ArrayUtils;
use Directus\Util\StringUtils;

/**
 * Client Remote
 *
 * @author Welling Guzmán <[email protected]>
 */
class ClientRemote extends BaseClientRemote
{
    protected $baseEndpoint = 'http://localhost/api';
    protected $hostedBaseEndpointFormat = 'https://%s.directus.io/api';

    /**
     * @inheritdoc
     */
    public function getTables(array $params = [])
    {
        return $this->performRequest('GET', static::TABLE_LIST_ENDPOINT);

    }

    /**
     * @inheritdoc
     */
    public function getTable($tableName)
    {
        return $this->performRequest('GET', static::TABLE_INFORMATION_ENDPOINT, $tableName);

    }

    /**
     * @inheritdoc
     */
    public function getColumns($tableName, array $params = [])
    {
        return $this->performRequest('GET', static::COLUMN_LIST_ENDPOINT, $tableName);

    }

    /**
     * @inheritdoc
     */
    public function getColumn($tableName, $columnName)
    {
        return $this->performRequest('GET', static::COLUMN_INFORMATION_ENDPOINT, [$tableName, $columnName]);

    }

    /**
     * @inheritdoc
     */
    public function getEntries($tableName, array $options = [])
    {
        return $this->performRequest('GET', static::TABLE_ENTRIES_ENDPOINT, $tableName, null, $options);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @inheritdoc
     */
    public function getEntry($tableName, $id, array $options = [])
    {
        return $this->performRequest('GET', static::TABLE_ENTRY_ENDPOINT, [$tableName, $id]);

    }

    /**
     * @inheritdoc
     */
    public function getUsers(array $params = [])
    {
        return $this->getEntries('directus_users', $params);
    }

    /**
     * @inheritdoc
     */
    public function getUser($id, array $params = [])
    {
        return $this->getEntry($id, 'directus_users', $params);
    }

    /**
     * @inheritdoc
     */
    public function getGroups()
    {
        return $this->performRequest('GET', static::GROUP_LIST_ENDPOINT);

    }

    /**
     * @inheritdoc
     */
    public function getGroup($groupID)
    {
        return $this->performRequest('GET', static::GROUP_INFORMATION_ENDPOINT, $groupID);

    }

    /**
     * @inheritdoc
     */
    public function getGroupPrivileges($groupID)
    {
        return $this->performRequest('GET', static::GROUP_PRIVILEGES_ENDPOINT, $groupID);

    }

    /**
     * @inheritdoc
     */
    public function getFiles()
    {
        return $this->performRequest('GET', static::FILE_LIST_ENDPOINT);

    }

    /**
     * @inheritdoc
     */
    public function getFile($fileID)
    {
        return $this->performRequest('GET', static::FILE_INFORMATION_ENDPOINT, $fileID);

    }

    /**
     * @inheritdoc
     */
    public function getSettings()
    {
        return $this->performRequest('GET', static::SETTING_LIST_ENDPOINT);
    }

    /**
     * @inheritdoc
     */
    public function getSettingsByCollection($collectionName)
    {
        return $this->performRequest('GET', static::SETTING_COLLECTION_ENDPOINT, $collectionName);

    }

    /**
     * @inheritdoc
     */
    public function getMessages($userId)
    {
        return $this->performRequest('GET', static::MESSAGES_USER_ENDPOINT, $userId);

    }

    /**
     * @inheritdoc
     */
    public function createEntry($tableName, array $data)
    {
        return $this->performRequest('POST', static::TABLE_ENTRY_CREATE_ENDPOINT, $tableName, $data);

    }

    /**
     * @inheritdoc
     */
    public function updateEntry($tableName, $id, array $data)
    {
        return $this->performRequest('PUT', static::TABLE_ENTRY_UPDATE_ENDPOINT, [$tableName, $id], $data);
    }

    /**
     * @inheritdoc
     */
    public function deleteEntry($tableName, $ids)
    {
        return $this->performRequest('DELETE', static::TABLE_ENTRY_DELETE_ENDPOINT, [$tableName, $ids]);
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
    }

    /**
     * @inheritdoc
     */
    public function createUser(array $data)

    {
        // @TODO: Add hooks
        if (ArrayUtils::has($data, 'password')) {
            // @NOTE: Use Directus password hash
            $data['password'] = password_hash($data['password'], PASSWORD_DEFAULT, ['cost' => 12]);
        }

        return $this->createEntry('directus_users', $data);
    }

    /**
     * @inheritdoc
     */
    public function updateUser($id, array $data)

    {
        // @TODO: Add hooks
        if (ArrayUtils::has($data, 'password')) {
            // @NOTE: Use Directus password hash
            $data['password'] = password_hash($data['password'], PASSWORD_DEFAULT, ['cost' => 12]);
        }

        return $this->updateEntry('directus_users', $id, $data);
    }

    /**
     * @inheritdoc
     */
    public function deleteUser($ids)
    {
        return $this->deleteEntry('directus_users', $ids);
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
    }

    /**
     * @inheritdoc
     */
    public function createFile(array $data)
    {
        $attributes = [];
        if (ArrayUtils::has($data, 'file')) {
            $path = $data['file'];
            $ext = pathinfo($path, PATHINFO_EXTENSION);
            $mimeType = mime_content_type($path);
            $attributes['name'] = pathinfo($path, PATHINFO_FILENAME) . '.' . $ext;
            $attributes['type'] = $mimeType;
            $content = file_get_contents($path);
            $base64 = 'data:' . $mimeType . ';base64,' . base64_encode($content);
            $attributes['data'] = $base64;
            unset($data['file']);
        } else if (ArrayUtils::has($data, 'data')) {
            $finfo = new \finfo(FILEINFO_MIME);
            list($mimeType, $charset) = explode('; charset=', $finfo->buffer($data['data']));
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
            $base64 = 'data:' . $mimeType . ';base64,' . base64_encode($data['data']);
            list($type, $subtype) = explode('/', $mimeType);
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
            $attributes['data'] = $base64;
            $attributes['type'] = $mimeType;
            $attributes['name'] = StringUtils::randomString() . '.' . $subtype;
            unset($data['data']);
        } else {
            throw new \Exception('Missing "file" or "data" attribute.');
        }

        $data = array_merge($data, $attributes);

        return $this->performRequest('POST', static::FILE_CREATE_ENDPOINT, null, $data);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @inheritdoc
     */
    public function updateFile($id, array $data)
    {
        return $this->updateEntry('directus_files', $id, $data);
    }

    /**
     * @inheritdoc
     */
    public function deleteFile($ids)
    {
        return $this->deleteEntry('directus_files', $ids);
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
    }
}


1		<?php
2
3		/**
4		* Directus – <http://getdirectus.com>
5		*
6		* @link The canonical repository – <https://github.com/directus/directus>
7		* @copyright Copyright 2006-2016 RANGER Studio, LLC – <http://rangerstudio.com>
8		* @license GNU General Public License (v3) – <http://www.gnu.org/copyleft/gpl.html>
9		*/
10
11		namespace Directus\SDK;
12		use Directus\Util\ArrayUtils;
13		use Directus\Util\StringUtils;
14
15		/**
16		* Client Remote
17		*
18		* @author Welling Guzmán <[email protected]>
19		*/
20		class ClientRemote extends BaseClientRemote
21		{
22		protected $baseEndpoint = 'http://localhost/api';
23	2	protected $hostedBaseEndpointFormat = 'https://%s.directus.io/api';
24
25	2	/**
26		* @inheritdoc
27		*/
28	4	public function getTables(array $params = [])
29		{
30	4	return $this->performRequest('GET', static::TABLE_LIST_ENDPOINT);
		1 ignored issue – show Bug Compatibility introduced 2016-11-16 18:40 UTC by Report Bug Copy Issue Report The expression `$this->performRequest('G...::TABLE_LIST_ENDPOINT);` of type `Directus\SDK\Response\En...ctus\SDK\Response\Entry` adds the type `Directus\SDK\Response\Entry` to the return on line 30 which is incompatible with the return type declared by the interface `Directus\SDK\RequestsInterface::getTables` of type `Directus\SDK\Response\EntryCollection`. Loading history...
31		}
32
33	2	/**
34		* @inheritdoc
35	2	*/
36		public function getTable($tableName)
37		{
38	2	return $this->performRequest('GET', static::TABLE_INFORMATION_ENDPOINT, $tableName);
		0 ignored issues – show Bug Compatibility introduced 2016-11-16 18:40 UTC by Report Bug Copy Issue Report The expression `$this->performRequest('G..._ENDPOINT, $tableName);` of type `Directus\SDK\Response\En...ctus\SDK\Response\Entry` adds the type `Directus\SDK\Response\EntryCollection` to the return on line 38 which is incompatible with the return type declared by the interface `Directus\SDK\RequestsInterface::getTable` of type `Directus\SDK\Response\Entry`. Loading history...

directus / directus-sdk-php

Push — d64 ( f52daa...138eb5 )

ClientRemote::deleteUser() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like