yiiviet\payment\momo\SignatureValidatorTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

72.72%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	signatureValidator()	0	13	2
A	getDataSignAttributes()	0	4	1

<?php
/**
 * @link https://github.com/yiiviet/yii2-payment
 * @copyright Copyright (c) 2017 Yii Viet
 * @license [New BSD License](http://www.opensource.org/licenses/bsd-license.php)
 */

namespace yiiviet\payment\momo;

use yii\base\InvalidCallException;

/**
 * Trait SignatureValidatorTrait cung cấp phương thức xác minh chữ ký dữ liệu từ MOMO.
 *
 * @author Vuong Minh <vuongxuongminh@gmail.com>
 * @since 1.0.3
 */
trait SignatureValidatorTrait
{
    /**
     * Phương thức kiểm tra chữ ký dữ liệu có hợp lệ hay không từ MOMO gửi sang.
     *
     * @param string $attribute có giá trị là chữ ký cần kiểm tra.
     * @param array $params thiết lập từ rule
     * @param \yii\validators\InlineValidator $validator
     * @throws \yii\base\InvalidConfigException|\yii\base\NotSupportedException|InvalidCallException
     */
    public function signatureValidator($attribute, $params, \yii\validators\InlineValidator $validator)

The parameter $params is not used and could be removed.

    {
        $dataSignAttributes = $this->getDataSignAttributes();
        $data = array_merge(array_fill_keys($dataSignAttributes, ''), $this->get(false));

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

        $dataSign = array_intersect_key($data, array_flip($dataSignAttributes));
        $dataSign = urldecode(http_build_query($dataSign));

        $client = $this->getClient();

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

        if (!$client->validateSignature($dataSign, $this->$attribute ?? '')) {
            $validator->addError($this, $attribute, $validator->message);
        }
    }

    /**
     * Phương thức hồ trợ cung cấp các attributes dùng để xác minh tính hợp lệ của chữ ký dữ liệu phản hồi từ MOMO.
     *
     * @return array attributes dùng để xác minh tính hợp lệ.
     */
    protected function getDataSignAttributes(): array
    {
        throw new InvalidCallException(__METHOD__ . ' must be override by class used it!');
    }
}