RequestSignature::generateSignature() - Code Metrics - phpviet/omnipay-vnpay - Measure and Improve Code Quality continuously with Scrutinizer

RequestSignature::generateSignature() A
last analyzed 2020-07-09 07:48 UTC

↳ Parent: RequestSignature

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	14
rs	9.7998
c	0
b	0
f	0
cc	2
nc	2
nop	1

<?php
/**
 * @link https://github.com/phpviet/omnipay-vnpay
 * @copyright (c) PHP Viet
 * @license [MIT](http://www.opensource.org/licenses/MIT)
 */

namespace Omnipay\VNPay\Message\Concerns;

use Omnipay\VNPay\Support\Signature;

/**
 * @author Vuong Minh <[email protected]>
 * @since 1.0.0
 */
trait RequestSignature
{
    /**
     * Trả về chữ ký điện tử gửi đến VNPay dựa theo [[getSignatureParameters()]].
     *
     * @param  string  $hashType
     * @return string
     */
    protected function generateSignature(string $hashType = 'sha256'): string
    {
        $data = [];
        $signature = new Signature(
            $this->getVnpHashSecret(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $hashType
        );

        foreach ($this->getSignatureParameters() as $parameter) {
            $data[$parameter] = $this->getParameter($parameter);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        return $signature->generate($data);
    }

    /**
     * Trả về danh sách parameters dùng để tạo chữ ký số.
     *
     * @return array
     */
    abstract protected function getSignatureParameters(): array;
}


1			<?php
2			/**
3			* @link https://github.com/phpviet/omnipay-vnpay
4			* @copyright (c) PHP Viet
5			* @license [MIT](http://www.opensource.org/licenses/MIT)
6			*/
7
8			namespace Omnipay\VNPay\Message\Concerns;
9
10			use Omnipay\VNPay\Support\Signature;
11
12			/**
13			* @author Vuong Minh <[email protected]>
14			* @since 1.0.0
15			*/
16			trait RequestSignature
17			{
18			/**
19			* Trả về chữ ký điện tử gửi đến VNPay dựa theo [[getSignatureParameters()]].
20			*
21			* @param string $hashType
22			* @return string
23			*/
24			protected function generateSignature(string $hashType = 'sha256'): string
25			{
26			$data = [];
27			$signature = new Signature(
28			$this->getVnpHashSecret(),
			0 ignored issues – show Bug introduced 2019-07-05 10:23 UTC by Report Bug Copy Issue Report It seems like `getVnpHashSecret()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
29			$hashType
30			);
31
32			foreach ($this->getSignatureParameters() as $parameter) {
33			$data[$parameter] = $this->getParameter($parameter);
			0 ignored issues – show Bug introduced 2019-07-05 10:23 UTC by Report Bug Copy Issue Report It seems like `getParameter()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
34			}
35
36			return $signature->generate($data);
37			}
38
39			/**
40			* Trả về danh sách parameters dùng để tạo chữ ký số.
41			*
42			* @return array
43			*/
44			abstract protected function getSignatureParameters(): array;
45			}
46

phpviet / omnipay-vnpay

RequestSignature::generateSignature() A last analyzed 2020-07-09 07:48 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

RequestSignature::generateSignature() A
last analyzed 2020-07-09 07:48 UTC