PermissionRepository::update() - Code Metrics - laraflock/dashboard - Measure and Improve Code Quality continuously with Scrutinizer

PermissionRepository::update() B
last analyzed 2015-12-23 16:32 UTC

↳ Parent: PermissionRepository

Complexity

Conditions	4
Paths	5

Size

Total Lines	25
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	16
CRAP Score	4

Metric	Value
dl	0
loc	25
ccs	16
cts	16
cp	1
rs	8.5806
cc	4
eloc	14
nc	5
nop	3
crap	4

<?php

/**
 * @package     Dashboard
 * @author      Ian Olson <[email protected]>
 * @license     MIT
 * @copyright   2015, Laraflock
 * @link        https://github.com/laraflock
 */

namespace Laraflock\Dashboard\Repositories\Permission;

use Illuminate\Database\QueryException;
use Laraflock\Dashboard\Exceptions\PermissionsException;
use Laraflock\Dashboard\Models\Permission;
use Laraflock\Dashboard\Repositories\Base\BaseRepository;

class PermissionRepository extends BaseRepository implements PermissionRepositoryInterface
{
    /**
     * Permission instance.
     *
     * @var \Laraflock\Dashboard\Models\Permission
     */
    protected $permission;

    /**
     * The constructor.
     *
     * @param \Laraflock\Dashboard\Models\Permission $permission
     */
    public function __construct(Permission $permission)
    {
        $this->permission = $permission;
    }

    /**
     * {@inheritDoc}
     */
    public function getAll()
    {
        return $this->permission->all();
    }

    /**
     * {@inheritDoc}
     */
    public function getById($id)
    {
        return $this->permission->find($id);
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    /**
     * {@inheritDoc}
     */
    public function create(array $data, $validate = true)
    {
        $this->rules = [
          'name' => 'required',
          'slug' => 'required|unique:permissions',
        ];

        if ($validate) {
            $this->validate($data);
        }

        try {
            $permission = $this->permission->create($data);
        } catch (QueryException $e) {
            throw new PermissionsException(trans('dashboard::dashboard.errors.permission.create'));
        }

        return $permission;
    }

    /**
     * {@inheritDoc}
     */
    public function update(array $data, $id, $validate = true)
    {
        if (!$permission = $this->getById($id)) {
            throw new PermissionsException(trans('dashboard::dashboard.errors.permission.found'));
        }

        $this->rules = [
          'name' => 'required',
          'slug' => 'required|alpha_dash',
        ];

        if ($permission->slug != $data['slug']) {
            $this->rules['slug'] = 'required|alpha_dash|unique:permissions';
        }

        if ($validate) {
            $this->validate($data);
        }

        $permission->name = $data['name'];
        $permission->slug = $data['slug'];
        $permission->save();

        return $permission;
    }

    /**
     * {@inheritDoc}
     */
    public function delete($id)
    {
        if (!$permission = $this->getById($id)) {
            throw new PermissionsException(trans('dashboard::dashboard.errors.permission.found'));
        }

        $permission->delete();

        return true;
    }
}

1		<?php
2
3		/**
4		* @package Dashboard
5		* @author Ian Olson <[email protected]>
6		* @license MIT
7		* @copyright 2015, Laraflock
8		* @link https://github.com/laraflock
9		*/
10
11		namespace Laraflock\Dashboard\Repositories\Permission;
12
13		use Illuminate\Database\QueryException;
14		use Laraflock\Dashboard\Exceptions\PermissionsException;
15		use Laraflock\Dashboard\Models\Permission;
16		use Laraflock\Dashboard\Repositories\Base\BaseRepository;
17
18		class PermissionRepository extends BaseRepository implements PermissionRepositoryInterface
19		{
20		/**
21		* Permission instance.
22		*
23		* @var \Laraflock\Dashboard\Models\Permission
24		*/
25		protected $permission;
26
27		/**
28		* The constructor.
29		*
30		* @param \Laraflock\Dashboard\Models\Permission $permission
31		*/
32	110	public function __construct(Permission $permission)
33		{
34	110	$this->permission = $permission;
35	110	}
36
37		/**
38		* {@inheritDoc}
39		*/
40	4	public function getAll()
41		{
42	4	return $this->permission->all();
43		}
44
45		/**
46		* {@inheritDoc}
47		*/
48	14	public function getById($id)
49		{
50	14	return $this->permission->find($id);
		0 ignored issues – show Documentation Bug introduced 2015-12-22 17:56 UTC by Report Bug Copy Issue Report The method `find` does not exist on `object<Laraflock\Dashboard\Models\Permission>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
51		}
52
53		/**
54		* {@inheritDoc}
55		*/
56	24	public function create(array $data, $validate = true)
57		{
58	24	$this->rules = [
59	24	'name' => 'required',
60	24	'slug' => 'required\|unique:permissions',
61		];
62
63	24	if ($validate) {
64	13	$this->validate($data);
65	12	}
66
67		try {
68	24	$permission = $this->permission->create($data);
69	24	} catch (QueryException $e) {
70	1	throw new PermissionsException(trans('dashboard::dashboard.errors.permission.create'));
71		}
72
73	24	return $permission;
74		}
75
76		/**
77		* {@inheritDoc}
78		*/
79	6	public function update(array $data, $id, $validate = true)
80		{
81	6	if (!$permission = $this->getById($id)) {
82	2	throw new PermissionsException(trans('dashboard::dashboard.errors.permission.found'));
83		}
84
85	4	$this->rules = [
86	4	'name' => 'required',
87	4	'slug' => 'required\|alpha_dash',
88		];
89
90	4	if ($permission->slug != $data['slug']) {
91	3	$this->rules['slug'] = 'required\|alpha_dash\|unique:permissions';
92	3	}
93
94	4	if ($validate) {
95	3	$this->validate($data);
96	1	}
97
98	2	$permission->name = $data['name'];
99	2	$permission->slug = $data['slug'];
100	2	$permission->save();
101
102	2	return $permission;
103		}
104
105		/**
106		* {@inheritDoc}
107		*/
108	4	public function delete($id)
109		{
110	4	if (!$permission = $this->getById($id)) {
111	2	throw new PermissionsException(trans('dashboard::dashboard.errors.permission.found'));
112		}
113
114	2	$permission->delete();
115
116	2	return true;
117		}
118		}

laraflock / dashboard

PermissionRepository::update() B last analyzed 2015-12-23 16:32 UTC

Complexity

Size

Duplication

Code Coverage

Duplication Side-by-Side

Filter issues like

PermissionRepository::update() B
last analyzed 2015-12-23 16:32 UTC