Daaner\NovaPoshta\Models\Orders

Complexity

Total Complexity

2

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	printDocument()	0	10	2

<?php

namespace Daaner\NovaPoshta\Models;

use Daaner\NovaPoshta\NovaPoshta;

class Orders extends NovaPoshta
{
    protected $model = 'orders';
    protected $calledMethod;
    protected $methodProperties = null;

	/**
	 * printDocument method of InternetDocument model.
	 *
	 * @param array|string $DocumentRefs Array of Documents IDs
	 * @param string       $type         (pdf|html)
	 *
	 * @return mixed
	 */
	public function printDocument($DocumentRefs, $type = 'html')
	{
		if (is_array($DocumentRefs) === false) {
			$DocumentRefs = explode(', ', /** @scrutinizer ignore-type */ $DocumentRefs);
		}

		$orders = "orders[]/" . implode("/orders[]/", $DocumentRefs);

It seems like $DocumentRefs can also be of type string; however, parameter $pieces of implode() does only seem to accept array, maybe add an additional type check?

		$this->calledMethod = 'printDocument/' . $orders . '/type/' . $type . '/apiKey/' . $this->getApi();

		return $this->getResponse($this->model, $this->calledMethod, $this->methodProperties);
	}
}