allejo\DaPulse\Objects\PulseColumnPersonValue

Complexity

Total Complexity

10

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

90.91%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getValue()	0	4	1
A	updateValue()	0	13	1
A	isNullValue()	0	10	6
A	setValue()	0	10	2

<?php

/**
 * @copyright 2017 Vladimir Jimenez
 * @license   https://github.com/allejo/PhpPulse/blob/master/LICENSE.md MIT
 */

namespace allejo\DaPulse\Objects;

use allejo\DaPulse\Exceptions\ColumnNotFoundException;
use allejo\DaPulse\PulseUser;

/**
 * Class PulseColumnTextValue
 *
 * @package allejo\DaPulse\Objects
 * @since   0.1.0
 */
class PulseColumnPersonValue extends PulseColumnValue
{
    /**
     * Get the person assigned listed in the person column
     *
     * @api
     *
     * @since  0.4.0 ColumnNotFoundException is now thrown
     * @since  0.1.0
     *
     * @throws ColumnNotFoundException The specified column ID does not exist for the parent Pulse
     *
     * @return PulseUser|null Null is returned when no person is listed in this person column
     */
    public function getValue ()
    {
        return parent::getValue();
    }

    /**
     * Update the person in a person column
     *
     * @api
     *
     * @param int|PulseUser $user The new user that will be assigned to the person column
     *
     * @since 0.3.0 \InvalidArgumentException is now thrown
     * @since 0.1.0
     *
     * @throws \InvalidArgumentException if $user is not an integer, is not positive, or is not a PulseUser object
     */
    public function updateValue ($user)
    {
        $user       = PulseUser::_castToInt($user);
        $url        = sprintf("%s/%d/columns/%s/person.json", self::apiEndpoint(), $this->board_id, $this->column_id);
        $postParams = [
            "pulse_id" => $this->pulse_id,
            "user_id"  => $user
        ];

        $result = self::sendPut($url, $postParams);
        $this->jsonResponse = $result;

It seems like $result of type * is incompatible with the declared type array of property $jsonResponse.

        $this->setValue($result);
    }

    protected function isNullValue ()
    {
        // Thank you very much, DaPulse. Changing the return type of an API endpoint clearly does not break any existing
        // code. Check to see if an invalid user is returned to be able to return null

        return parent::isNullValue() ||
               (is_array($this->jsonResponse) &&
                isset($this->jsonResponse['value']) && is_array($this->jsonResponse['value']) &&
                isset($this->jsonResponse['value']['id']) && $this->jsonResponse['value']['id'] === 0);
    }

    protected function setValue ($response)
    {
        if (!isset($response['value']['id']))
        {
            $this->column_value = null;
            return;
        }

        $this->column_value = new PulseUser($response["value"]["id"]);
    }
}