Omnipay\MoMo\Message\Concerns\RequestHash

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	generateHash()	18	18	3
	getHashParameters()	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\RSAEncrypt;

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

This class seems to be duplicated in your project.

{
    /**
     * Trả về dữ liệu mã hóa theo [[getHashParameters()]].
     *
     * @return string
     */
    protected function generateHash(): string
    {
        $data = [];
        $rsa = new RSAEncrypt(
            $this->getPublicKey()

It seems like getPublicKey() 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->getHashParameters() as $pos => $parameter) {
            if (! is_string($pos)) {
                $pos = $parameter;
            }

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

        return $rsa->encrypt($data);
    }

    /**
     * Trả về danh sách parameters dùng để mã hóa gửi đến MoMo.
     *
     * @return array
     */
    abstract protected function getHashParameters(): array;
}