ActivateUserByForce::__construct() - Code Metrics - anomalylabs/users-module - Measure and Improve Code Quality continuously with Scrutinizer

ActivateUserByForce::__construct() A
last analyzed 2017-05-19 08:34 UTC

↳ Parent: ActivateUserByForce

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php namespace Anomaly\UsersModule\User\Command;

use Anomaly\UsersModule\User\Contract\UserInterface;
use Anomaly\UsersModule\User\Contract\UserRepositoryInterface;


/**
 * Class ActivateUserByForce
 *
 * @link          http://pyrocms.com/
 * @author        PyroCMS, Inc. <[email protected]>
 * @author        Ryan Thompson <[email protected]>
 */
class ActivateUserByForce
{

    /**
     * The user instance.
     *
     * @var UserInterface
     */
    protected $user;

    /**
     * Create a new ActivateUserByForce instance.
     *
     * @param UserInterface $user
     */
    function __construct(UserInterface $user)

    {
        $this->user = $user;
    }

    /**
     * Handle the command.
     *
     * @param  UserRepositoryInterface $users
     * @return bool
     */
    public function handle(UserRepositoryInterface $users)
    {
        $this->user->activated       = true;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        $this->user->activation_code = null;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}

        $users->save($this->user);

        return true;
    }
}


1			<?php namespace Anomaly\UsersModule\User\Command;
2
3			use Anomaly\UsersModule\User\Contract\UserInterface;
4			use Anomaly\UsersModule\User\Contract\UserRepositoryInterface;
5
6
7			/**
8			* Class ActivateUserByForce
9			*
10			* @link http://pyrocms.com/
11			* @author PyroCMS, Inc. <[email protected]>
12			* @author Ryan Thompson <[email protected]>
13			*/
14			class ActivateUserByForce
15			{
16
17			/**
18			* The user instance.
19			*
20			* @var UserInterface
21			*/
22			protected $user;
23
24			/**
25			* Create a new ActivateUserByForce instance.
26			*
27			* @param UserInterface $user
28			*/
29			function __construct(UserInterface $user)
			0 ignored issues – show Best Practice introduced 2015-12-02 14:19 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
30			{
31			$this->user = $user;
32			}
33
34			/**
35			* Handle the command.
36			*
37			* @param UserRepositoryInterface $users
38			* @return bool
39			*/
40			public function handle(UserRepositoryInterface $users)
41			{
42			$this->user->activated = true;
			0 ignored issues – show Bug introduced 2015-12-02 14:19 UTC by Report Bug Copy Issue Report Accessing `activated` on the interface `Anomaly\UsersModule\User\Contract\UserInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
43			$this->user->activation_code = null;
			0 ignored issues – show Bug introduced 2015-12-02 14:19 UTC by Report Bug Copy Issue Report Accessing `activation_code` on the interface `Anomaly\UsersModule\User\Contract\UserInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
44
45			$users->save($this->user);
46
47			return true;
48			}
49			}
50

anomalylabs / users-module

ActivateUserByForce::__construct() A last analyzed 2017-05-19 08:34 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

ActivateUserByForce::__construct() A
last analyzed 2017-05-19 08:34 UTC