allejo\DaPulse\Objects\PulseColumnPersonValue

Complexity

Total Complexity

9

Size/Duplication

Total Lines	69
Duplicated Lines	20.29 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getValue()	14	14	3
A	updateValue()	0	13	2
A	isNullValue()	0	11	4

<?php

namespace allejo\DaPulse\Objects;

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
     *
     * **Warning** This function may return a null value so ensure the returned value is not null before calling any
     * functions that belong to a PulseUser object.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return PulseUser|null Null is returned when no person is listed in this person column
     */
    public function getValue ()

This method seems to be duplicated in your project.

    {
        if ($this->isNullValue())
        {
            return null;
        }

        if (!isset($this->column_value))
        {
            $this->column_value = new PulseUser($this->jsonResponse["value"]["id"]);
        }

        return $this->column_value;
    }

    /**
     * Update the person in a person column
     *
     * **Warning** Currently, there is no way of removing a person from a column; only replacing them.
     *
     * @api
     *
     * @param int|PulseUser $user The new user that will be assigned to the person column
     *
     * @since 0.1.0
     */
    public function updateValue ($user)
    {
        $user       = ($user instanceof PulseUser) ? $user->getId() : $user;
        $url        = sprintf("%s/%d/columns/%s/person.json", self::apiEndpoint(), $this->board_id, $this->column_id);
        $postParams = array(
            "pulse_id" => $this->pulse_id,
            "user_id"  => $user
        );

        self::sendPut($url, $postParams);

        $this->column_value = new PulseUser($user);
    }

    /**
     * {@inheritdoc}
     */
    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

        $isNullUser = array_key_exists('value', $this->jsonResponse) &&
                      array_key_exists('id', $this->jsonResponse['value']) &&
                      $this->jsonResponse['value']['id'] === 0;

        return (parent::isNullValue() || $isNullUser);
    }
}