allejo\DaPulse\Objects\PulseColumnNumericValue

Complexity

Total Complexity

4

Size/Duplication

Total Lines	51
Duplicated Lines	31.37 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	updateValue()	16	16	2
A	getValue()	0	4	1
A	setValue()	0	4	1

<?php

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

namespace allejo\DaPulse\Objects;

/**
 * Class PulseColumnTextValue
 *
 * @package allejo\DaPulse\Objects
 * @since   0.2.0
 */
class PulseColumnNumericValue extends PulseColumnValue
{
    /**
     * Get a numeric column's content
     *
     * @api
     *
     * @since  0.3.0 Docs correctly specify a numeric return type
     * @since  0.2.0
     *
     * @return int|double|null Null is returned if there is no value set for this column
     */
    public function getValue ()
    {
        return parent::getValue();
    }

    /**
     * Update the value of a numeric column
     *
     * @api
     *
     * @param int|double $number
     *
     * @since 0.3.0 \InvalidArgumentException is now thrown
     * @since 0.2.0
     *
     * @throws \InvalidArgumentException if $number is not a numeric value
     */
    public function updateValue ($number)

This method seems to be duplicated in your project.

    {
        if (!is_numeric($number))
        {
            throw new \InvalidArgumentException('$number is expected to be a numeric type');
        }

        $url        = sprintf("%s/%d/columns/%s/numeric.json", self::apiEndpoint(), $this->board_id, $this->column_id);
        $postParams = [
            "pulse_id" => $this->pulse_id,
            "value"    => $number
        ];

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

    protected function setValue ($response)
    {
        $this->column_value = $response["value"];
    }
}