Mailxpert\Request\CustomFieldsChoicesRequest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	6	1
A	post()	0	10	2

<?php
/**
 * Date: 21/08/15
 */

namespace Mailxpert\Request;

use Mailxpert\Exceptions\MailxpertSDKException;
use Mailxpert\Mailxpert;

/**
 * Class CustomFieldsChoicesRequest
 * @package Mailxpert\Request
 */
class CustomFieldsChoicesRequest
{
    /**
     * @param Mailxpert $mailxpert
     * @param string    $customFieldId
     * @param array     $params
     *
     * @return \Mailxpert\MailxpertResponse
     */
    public static function get(Mailxpert $mailxpert, $customFieldId, array $params = [])
    {
        $response = $mailxpert->sendRequest('GET', sprintf('custom_fields/%s/choices', $customFieldId), $params);

        return $response;
    }

    /**
     * @param Mailxpert $mailxpert
     * @param string    $customFieldId
     * @param array     $params
     *
     * @return \Mailxpert\MailxpertResponse
     * @throws MailxpertSDKException
     */
    public static function post(Mailxpert $mailxpert, $customFieldId, array $params)
    {
        $response = $mailxpert->sendRequest('POST', sprintf('custom_fields/%s/choices', $customFieldId), [], null, json_encode($params));

        if (!$response->getHeader('Location')) {

The expression $response->getHeader('Location') of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            throw new MailxpertSDKException('An error occured during the Contactfield Choice creation.');
        }

        return $response;
    }
}