Padosoft\AffiliateNetwork\Networks\Zanox

Complexity

Total Complexity

37

Size/Duplication

Total Lines	231
Duplicated Lines	6.93 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	login()	0	21	4
A	checkLogin()	0	4	1
A	getMerchants()	16	16	3
D	getDeals()	0	75	13
F	getSales()	0	40	14
A	getStats()	0	17	1

<?php

namespace Padosoft\AffiliateNetwork\Networks;

use Padosoft\AffiliateNetwork\DealsResultset;
use Padosoft\AffiliateNetwork\Transaction;
use Padosoft\AffiliateNetwork\Merchant;
use Padosoft\AffiliateNetwork\Stat;
use Padosoft\AffiliateNetwork\Deal;
use Padosoft\AffiliateNetwork\AbstractNetwork;
use Padosoft\AffiliateNetwork\NetworkInterface;

// require "../vendor/fubralimited/php-oara/Oara/Network/Publisher/Zanox/Zapi/ApiClient.php";

/**
 * Class Zanox
 * @package Padosoft\AffiliateNetwork\Networks
 */
class Zanox extends AbstractNetwork implements NetworkInterface
{
    /**
     * @var object
     */
    private $_network   = null;
    protected $_apiClient = null;
    private $_username  = '';
    private $_password  = '';
    private $_logged    = false;


    /**
     * @method __construct
     */
    public function __construct(string $username, string $password)
    {
        $this->_network = new ZanoxEx;
        $this->_username = $username;
        $this->_password = $password;
        $this->login( $this->_username, $this->_password );

    }

    public function login(string $username, string $password): bool
    {
        $this->_logged = false;
        if (isNullOrEmpty( $username ) || isNullOrEmpty( $password )) {

            return false;
        }
        $this->_username = $username;
        $this->_password = $password;
        $credentials = array();
        $credentials["connectid"] = $this->_username;
        $credentials["secretkey"] = $this->_password;
        $this->_network->login( $credentials );
        $this->_apiClient = $this->_network->getApiClient();
        if ($this->_network->checkConnection()) {
            $this->_logged = true;

        }

        return $this->_logged;
    }

    /**
     * @return bool
     */
    public function checkLogin(): bool
    {
        return $this->_logged;
    }

    /**
     * @return array of Merchants
     */
    public function getMerchants(): array

This method seems to be duplicated in your project.

    {
        if (!$this->checkLogin()) {
            return array();
        }
        $arrResult = array();
        $merchantList = $this->_network->getMerchantList();
        foreach ($merchantList as $merchant) {
            $Merchant = Merchant::createInstance();
            $Merchant->merchant_ID = $merchant['cid'];
            $Merchant->name = $merchant['name'];
            $arrResult[] = $Merchant;
        }

        return $arrResult;
    }

    /**
     * @param int|null $merchantID
     * @param int $page
     * @param int $items_per_page
     *
     * @return DealsResultset
     */
    public function getDeals($merchantID=NULL,int $page=0,int $items_per_page=10 ): DealsResultset
    {
        if (!isIntegerPositive($items_per_page)){
            $items_per_page=10;
        }
        $result=DealsResultset::createInstance();
        if (!$this->checkLogin()) {
            return $result;
        }
        /*$this->_apiClient->setConnectId( $this->_username );

60% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->_apiClient->setSecretKey( $this->_password );*/
        $adSpaces=$this->_apiClient->getAdspaces(0,100);
        if (!is_object($adSpaces)){
            $adSpaces=json_encode($adSpaces);
        }
        if ($adSpaces->items<1){
            return $result;
        }

        $adSpaceId=$adSpaces->adspaceItems->adspaceItem[0]->id;
        if (!isIntegerPositive($merchantID)){
            $merchantID=NULL;
        }

        $Response = $this->_apiClient->searchIncentives($merchantID,$adSpaceId,'coupons',NULL,$page,$items_per_page);
        
        if (!is_object($Response)){
            $Response=json_decode($Response);
        }
        $result->page=$Response->page;
        $result->items=$Response->items;
        $result->total=$Response->total;
        ($Response->total>0)?$result->num_pages=(int)ceil($Response->total/$items_per_page):$result->num_pages=0;

The property num_pages does not seem to exist in Padosoft\AffiliateNetwork\DealsResultset.

        $arrAdmediumItems = $Response->incentiveItems->incentiveItem;

        foreach ($arrAdmediumItems as $admediumItems) {
            $Deal = Deal::createInstance();
            $Deal->id = (int)$admediumItems->id;

The property id does not seem to exist in Padosoft\AffiliateNetwork\Deal.

            $Deal->created_at =$admediumItems->createDate;

The property created_at does not seem to exist in Padosoft\AffiliateNetwork\Deal.

            $Deal->startDate = $admediumItems->startDate;
            isset($admediumItems->endDate)?$Deal->endDate = $admediumItems->endDate:$Deal->endDate = '';

It seems like '' of type string is incompatible with the declared type object<DateTime> of property $endDate.

            $Deal->name = $admediumItems->name;
            $Deal->code = $admediumItems->couponCode;
            $Deal->description = $admediumItems->info4customer;
            $Deal->note = $admediumItems->info4publisher.' '.$admediumItems->restrictions;

The property note does not seem to exist in Padosoft\AffiliateNetwork\Deal.

            $Deal->is_percent = 0;

The property is_percent does not seem to exist. Did you mean is_percentage?

            $Deal->value=0;

The property value does not seem to exist in Padosoft\AffiliateNetwork\Deal.

            $Deal->currency='';

The property currency does not seem to exist. Did you mean currency_initial?

            //dd($admediumItems->percentage);

72% of this comment could be valid code. Did you maybe forget this after debugging?

            if (isset($admediumItems->percentage) && isIntegerPositive($admediumItems->percentage)){
                $Deal->is_percent = 1;

The property is_percent does not seem to exist. Did you mean is_percentage?

                $Deal->value=$admediumItems->percentage;
            }elseif (isset($admediumItems->total)){

                $Deal->value=$admediumItems->total;
                $Deal->currency=$admediumItems->currency;

The property currency does not seem to exist. Did you mean currency_initial?

            }
            //$Deal->deal_type = $admediumItems['admediumType'];

64% of this comment could be valid code. Did you maybe forget this after debugging?

            $Deal->merchant_ID = (int)$admediumItems->program->id;
            
            $Deal->merchant_name = $admediumItems->program->_;
            $Deal->ppv = $admediumItems->admedia->admediumItem[0]->trackingLinks->trackingLink[0]->ppv;
            $Deal->ppc = $admediumItems->admedia->admediumItem[0]->trackingLinks->trackingLink[0]->ppc;
            $result->deals[]=$Deal;
            /*if ($merchantID > 0) {

58% of this comment could be valid code. Did you maybe forget this after debugging?

                if ($merchantID == $admediumItems['program']['@id']) {
                    $arrResult[] = $Deal;
                }
            } else {
                $arrResult[] = $Deal;
            }*/
        }
        //dd($result);
        return $result;
    }


    /**
     * @param \DateTime $dateFrom
     * @param \DateTime $dateTo
     * @param int $merchantID

There is no parameter named $merchantID. Did you maybe mean $arrMerchantID?

     *
     * @return array of Transaction
     */
    public function getSales(\DateTime $dateFrom, \DateTime $dateTo, array $arrMerchantID = array()): array
    {
        if (!$this->checkLogin()) {
            return array();
        }
        $arrResult = array();
        if (count( $arrMerchantID ) < 1) {
            $merchants = $this->getMerchants();
            foreach ($merchants as $merchant) {
                $arrMerchantID[$merchant->merchant_ID] = ['cid' => $merchant->merchant_ID, 'name' => $merchant->name];
            }
        }
        $transcationList = $this->_network->getTransactionList( $arrMerchantID, $dateTo, $dateFrom );
        foreach ($transcationList as $transaction) {
            $Transaction = Transaction::createInstance();
            array_key_exists_safe( $transaction,
                'currency' ) ? $Transaction->currency = $transaction['currency'] : $Transaction->currency = '';
            array_key_exists_safe( $transaction,
                'status' ) ? $Transaction->status = $transaction['status'] : $Transaction->status = '';
            array_key_exists_safe( $transaction,
                'amount' ) ? $Transaction->amount = $transaction['amount'] : $Transaction->amount = '';

The property $amount was declared of type double, but '' is of type string. Maybe add a type cast?

            array_key_exists_safe( $transaction,
                'custom_id' ) ? $Transaction->custom_ID = $transaction['custom_id'] : $Transaction->custom_ID = '';
            array_key_exists_safe( $transaction,
                'title' ) ? $Transaction->title = $transaction['title'] : $Transaction->title = '';
            array_key_exists_safe( $transaction,
                'unique_id' ) ? $Transaction->unique_ID = $transaction['unique_id'] : $Transaction->unique_ID = '';
            array_key_exists_safe( $transaction,
                'commission' ) ? $Transaction->commission = $transaction['commission'] : $Transaction->commission = '';

The property $commission was declared of type double, but '' is of type string. Maybe add a type cast?

            $date = new \DateTime( $transaction['date'] );
            $Transaction->date = $date; // $date->format('Y-m-d H:i:s');

It seems like $date of type object<DateTime> is incompatible with the declared type string of property $date.

75% of this comment could be valid code. Did you maybe forget this after debugging?

            array_key_exists_safe( $transaction,
                'merchantId' ) ? $Transaction->merchant_ID = $transaction['merchantId'] : $Transaction->merchant_ID = '';

The property $merchant_ID was declared of type integer, but '' is of type string. Maybe add a type cast?

            array_key_exists_safe( $transaction,
                'approved' ) ? $Transaction->approved = $transaction['approved'] : $Transaction->approved = '';

The property $approved was declared of type boolean, but '' is of type string. Maybe add a type cast?

            $arrResult[] = $Transaction;
        }

        return $arrResult;
    }

    /**
     * @param \DateTime $dateFrom
     * @param \DateTime $dateTo
     * @param int $merchantID
     *
     * @return array of Stat
     */
    public function getStats(\DateTime $dateFrom, \DateTime $dateTo, int $merchantID = 0): array
    {
        return array();
        /*

59% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->_apiClient->setConnectId($this->_username);
        $this->_apiClient->setSecretKey($this->_password);
        $dateFromIsoEngFormat = $dateFrom->format('Y-m-d');
        $dateToIsoEngFormat = $dateTo->format('Y-m-d');
        $response = $this->_apiClient->getReportBasic($dateFromIsoEngFormat, $dateToIsoEngFormat);
        $arrResponse = json_decode($response, true);
        $reportItems = $arrResponse['reportItems'];
        $Stat = Stat::createInstance();
        $Stat->reportItems = $reportItems;

        return array($Stat);
        */
    }
}