Padosoft\AffiliateNetwork\EffiliationEx

Complexity

Total Complexity

8

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	2
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
C	getTransactionList()	0	71	8

<?php
/**
 * Copyright (c) Padosoft.com 2017.
 */
namespace Padosoft\AffiliateNetwork;
use Oara\Network\Publisher\Effiliation as EffiliationOara;

class EffiliationEx extends EffiliationOara{
    /**
     * @param null $merchantList
     * @param \DateTime|null $dStartDate
     * @param \DateTime|null $dEndDate
     * @return array
     */
    public function getTransactionList($merchantList = null, \DateTime $dStartDate = null, \DateTime $dEndDate = null)
    {
        try {

            $totalTransactions = array();

            $merchantIdList = \Oara\Utilities::getMerchantIdMapFromMerchantList($merchantList);

$merchantIdList is not used, you could remove the assignment.

            $url = 'http://api.effiliation.com/apiv2/transaction.csv?key=' . $this->_credentials["apiPassword"] . '&start=' . $dStartDate->format("d/m/Y") . '&end=' . $dEndDate->format("d/m/Y") . '&type=date&all=yes';

It seems like $dStartDate is not always an object, but can also be of type null. Maybe add an additional type check?

It seems like $dEndDate is not always an object, but can also be of type null. Maybe add an additional type check?

            // Set timeout to 300 secs. due to api delays - <PN> 2017-06-20
            $ctx = stream_context_create(array(
                    'http' => array(
                        'timeout' => 300
                    )
                )
            );
            // Log Debug info
            echo (New \DateTime())->format("d/m/Y H:i:s") . " - EffiliationEx getTransactionList from " . $dStartDate->format("d/m/Y") . " to " . $dEndDate->format("d/m/Y"),PHP_EOL;

            $content = \utf8_encode(\file_get_contents($url, 0, $ctx));
            $exportData = \str_getcsv($content, "\n");
            $num = \count($exportData);

            for ($i = 1; $i < $num; $i++) {
                $transactionExportArray = \str_getcsv($exportData[$i], "|");
                // We don't need to check whether the merchant id is valid or not ...
                // ... for old transactions may be expired and not included in current merchant list
                // <PN> 2017-06-22
                // if (isset($merchantIdList[(int)$transactionExportArray[2]])) {

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

                    $transaction = Array();
                    $merchantId = (int)$transactionExportArray[2];
                    $transaction['merchantId'] = $merchantId;
                    $transaction['date'] = $transactionExportArray[10];
                    $transaction['unique_id'] = $transactionExportArray[0];
                    $transaction['custom_id'] = '';
                    $transaction['status'] = \Oara\Utilities::STATUS_PENDING;
                    if ($transactionExportArray[4] != null) {
                        $transaction['custom_id'] = $transactionExportArray[4];
                    }

                    switch ($transactionExportArray[9]) {
                        case 'Valide':
                            $transaction['status'] = \Oara\Utilities::STATUS_CONFIRMED;
                            break;
                        case 'Attente':
                            $transaction['status'] = \Oara\Utilities::STATUS_PENDING;
                            break;
                        case 'Refusé':
                        case 'Refuse':
                            // Handle both variations - <PN> - 2017-06-21
                            $transaction['status'] = \Oara\Utilities::STATUS_DECLINED;
                            break;
                        default:
                            // Invalid status
                            echo PHP_EOL."EffiliationEx - Invalid transaction status: " . $transactionExportArray[9] . " (transaction id = " . $transactionExportArray[0] . ")";
                            break;
                    }
                    $transaction['amount'] = \Oara\Utilities::parseDouble($transactionExportArray[7]);
                    $transaction['commission'] = \Oara\Utilities::parseDouble($transactionExportArray[8]);
                    $totalTransactions[] = $transaction;
                // }
            }
        } catch (\Exception $e) {
            // Avoid lost of transactions if one date failed - <PN> - 2017-06-20
            echo PHP_EOL."EffiliationEx - getTransactionList err: ".$e->getMessage().PHP_EOL;
            throw new \Exception($e);
        }
        echo (New \DateTime())->format("d/m/Y H:i:s") . " - EffiliationEx getTransactionList - return " . count($totalTransactions) . " transactions",PHP_EOL;
        return $totalTransactions;
    }
}