Omnipay\Pesapal\OAuth\OAuthConsumer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	__toString()	0	3	1

<?php

namespace Omnipay\Pesapal\OAuth;

class OAuthConsumer
{
    /**
     * @var string
     */
    public $key;

    /**
     * @var string
     */
    public $secret;

    /**
     * @param $key
     * @param $secret
     * @param null $callback_url

Are you sure the doc-type for parameter $callback_url is correct as it would always require null to be passed?

     */
    public function __construct(
        $key,
        $secret,
        $callback_url = null
    ) {
        $this->key = $key;
        $this->secret = $secret;
        $this->callback_url = $callback_url;

The property callback_url does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
     * @return string
     */
    public function __toString(): string
    {
        return "OAuthConsumer[key=$this->key,secret=$this->secret]";
    }
}