Omnipay\MoMo\Message\Concerns\RequestSignature

Complexity

Total Complexity

3

Size/Duplication

Total Lines	33
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	generateSignature()	18	18	3
	getSignatureParameters()	1	1	?

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

namespace Omnipay\MoMo\Message\Concerns;

use Omnipay\MoMo\Support\Arr;
use Omnipay\MoMo\Support\Signature;

/**
 * @author Vuong Minh <vuongxuongminh@gmail.com>
 * @since 1.0.0
 */
trait RequestSignature

This class seems to be duplicated in your project.

{
    /**
     * Trả về chữ ký điện tử gửi đến MoMo dựa theo [[getSignatureParameters()]].
     *
     * @return string
     */
    protected function generateSignature(): string
    {
        $data = [];
        $signature = new Signature(
            $this->getSecretKey()

It seems like getSecretKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        );
        $parameters = $this->getParameters();

It seems like getParameters() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        foreach ($this->getSignatureParameters() as $pos => $parameter) {
            if (! is_string($pos)) {
                $pos = $parameter;
            }

            $data[$pos] = Arr::getValue($parameter, $parameters);
        }

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

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