MobileHash - Code Metrics - Inspection of "F smart" - dokobit/ws-api-sdk-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#9)

unknown

created 2018-03-12 14:57 UTC

MobileHash A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	117
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

Metric	Value
wmc	6
lcom	1
cbo	6
dl	0
loc	117
ccs	40
cts	40
cp	1
rs	10
c	0
b	0
f	0

6 Methods

Rating	Name	Size	Complexity
A	__construct()	17	1
A	getFields()	12	1
A	createResult()	4	1
A	getAction()	4	1
A	getMethod()	4	1
A	getValidationConstraints()	18	1

<?php
namespace Isign\Sign;

use Isign\DocumentTypeProvider;
use Isign\FileFieldsTrait;
use Isign\QueryInterface;
use Isign\Validator\Constraints\Code;
use Isign\Validator\Constraints\Phone;
use Symfony\Component\Validator\Constraints as Assert;

/**
 * Sign document using mobile ID hash.
 */
class MobileHash implements QueryInterface
{
    /** @var string user phone number */
    private $phone;

    /** @var string user personal code */
    private $code;
    
    /** @var string Hash to sign */
    private $hash;
    
    /** @var string Hash algorithm */
    private $hashAlgorithm;
    
    /** @var string Signature encryption type. */
    private $encryption;
    
    /** @var string Message displayed on the phone screen. Attention: UTF-8 symbols are not allowed */
    private $message;

    /** @var string Language for messages displayed on the phone screen */
    private $language;

    /**
     * @param string $phone
     * @param string $code
     * @param string $hash
     * @param string $hashAlgorithm
     * @param string $encryption
     * @param string $message
     * @param string $language
     */
    public function __construct(
        $phone,
        $code,
        $hash,
        $hashAlgorithm,
        $encryption,
        $message,
        $language
    ) {
        $this->phone = $phone;
        $this->code = $code;
        $this->hash = $hash;
        $this->hashAlgorithm = $hashAlgorithm;
        $this->encryption = $encryption;
        $this->message = $message;
        $this->language = $language;
    }

    /**
     * Field and values association used in query
     * @return array
     */
    public function getFields()
    {
        return [
            'phone' => $this->phone,
            'code' => $this->code,
            'hash' => $this->hash,
            'hash_algorithm' => $this->hashAlgorithm,
            'encryption' => $this->encryption,
            'message' => $this->message,
            'language' => $this->language,
        ];
    }

    /**
     * Validation constraints for request data validation
     * @return Assert\Collection
     */
    public function getValidationConstraints()
    {
        return new Assert\Collection([
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());
}
            'phone' => new Assert\Required([
                new Assert\NotBlank(),
                new Phone(),
            ]),
            'code' => new Assert\Required([
                new Assert\NotBlank(),
                new Code()
            ]),
            'hash' => new Assert\NotBlank(),
            'hash_algorithm' => new Assert\NotBlank(),
            'encryption' => new Assert\NotBlank(),
            'message' => new Assert\NotBlank(),
            'language' => new Assert\NotBlank(),
        ]);
    }

    /**
     * Result object for this query result
     * @return MobileHashResult
     */
    public function createResult()
    {
        return new MobileHashResult();
    }

    /**
     * API action name, part of full API request url
     * @return string
     */
    public function getAction()
    {
        return 'mobile/sign/hash';
    }

    /**
     * HTTP method to use
     * @return string
     */
    public function getMethod()
    {
        return QueryInterface::POST;
    }
}


1		<?php
2		namespace Isign\Sign;
3
4		use Isign\DocumentTypeProvider;
5		use Isign\FileFieldsTrait;
6		use Isign\QueryInterface;
7		use Isign\Validator\Constraints\Code;
8		use Isign\Validator\Constraints\Phone;
9		use Symfony\Component\Validator\Constraints as Assert;
10
11		/**
12		* Sign document using mobile ID hash.
13		*/
14		class MobileHash implements QueryInterface
15		{
16		/** @var string user phone number */
17		private $phone;
18
19		/** @var string user personal code */
20		private $code;
21
22		/** @var string Hash to sign */
23		private $hash;
24
25		/** @var string Hash algorithm */
26		private $hashAlgorithm;
27
28		/** @var string Signature encryption type. */
29		private $encryption;
30
31		/** @var string Message displayed on the phone screen. Attention: UTF-8 symbols are not allowed */
32		private $message;
33
34		/** @var string Language for messages displayed on the phone screen */
35		private $language;
36
37		/**
38		* @param string $phone
39		* @param string $code
40		* @param string $hash
41		* @param string $hashAlgorithm
42		* @param string $encryption
43		* @param string $message
44		* @param string $language
45		*/
46	5	public function __construct(
47		$phone,
48		$code,
49		$hash,
50		$hashAlgorithm,
51		$encryption,
52		$message,
53		$language
54		) {
55	5	$this->phone = $phone;
56	5	$this->code = $code;
57	5	$this->hash = $hash;
58	5	$this->hashAlgorithm = $hashAlgorithm;
59	5	$this->encryption = $encryption;
60	5	$this->message = $message;
61	5	$this->language = $language;
62	5	}
63
64		/**
65		* Field and values association used in query
66		* @return array
67		*/
68	1	public function getFields()
69		{
70		return [
71	1	'phone' => $this->phone,
72	1	'code' => $this->code,
73	1	'hash' => $this->hash,
74	1	'hash_algorithm' => $this->hashAlgorithm,
75	1	'encryption' => $this->encryption,
76	1	'message' => $this->message,
77	1	'language' => $this->language,
78	1	];
79		}
80
81		/**
82		* Validation constraints for request data validation
83		* @return Assert\Collection
84		*/
85	1	public function getValidationConstraints()
86		{
87	1	return new Assert\Collection([
		0 ignored issues – show Bug Best Practice introduced 2017-04-13 14:55 UTC by Report Bug Copy Issue Report The return type of `return new \Symfony\Comp...nstraints\NotBlank()));` (`Symfony\Component\Validator\Constraints\Collection`) is incompatible with the return type declared by the interface `Isign\QueryInterface::getValidationConstraints` of type `Isign\Symfony\Component\...\Constraints\Collection`. If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design. Let’s take a look at an example: 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()); } Our function `my_function` expects a `Post` object, and outputs the author of the post. The base class `Post` returns a simple string and outputting a simple string will work just fine. However, the child class `BlogPost` which is a sub-type of `Post` instead decided to return an `object`, and is therefore violating the SOLID principles. If a `BlogPost` were passed to `my_function`, PHP would not complain, but ultimately fail when executing the `strtoupper` call in its body. Loading history...
88	1	'phone' => new Assert\Required([
89	1	new Assert\NotBlank(),
90	1	new Phone(),
91	1	]),
92	1	'code' => new Assert\Required([
93	1	new Assert\NotBlank(),
94	1	new Code()
95	1	]),
96	1	'hash' => new Assert\NotBlank(),
97	1	'hash_algorithm' => new Assert\NotBlank(),
98	1	'encryption' => new Assert\NotBlank(),
99	1	'message' => new Assert\NotBlank(),
100	1	'language' => new Assert\NotBlank(),
101	1	]);
102		}
103
104		/**
105		* Result object for this query result
106		* @return MobileHashResult
107		*/
108	1	public function createResult()
109		{
110	1	return new MobileHashResult();
111		}
112
113		/**
114		* API action name, part of full API request url
115		* @return string
116		*/
117	1	public function getAction()
118		{
119	1	return 'mobile/sign/hash';
120		}
121
122		/**
123		* HTTP method to use
124		* @return string
125		*/
126	1	public function getMethod()
127		{
128	1	return QueryInterface::POST;
129		}
130		}
131

dokobit / ws-api-sdk-php

Pull Request — develop (#9)

MobileHash A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

6 Methods

Duplication Side-by-Side

Filter issues like