PulseColumnTimelineValue::updateValue() - Code Metrics - Inspection of "Implement new Timeline column type" - allejo/PhpPulse - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c07f69...57c551 )

by Vladimir

created 2017-01-09 17:30 UTC

PulseColumnTimelineValue::updateValue() A

↳ Parent: PulseColumnTimelineValue

Complexity

Conditions	1
Paths	1

Size

Total Lines	16
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	16
rs	9.4285
cc	1
eloc	10
nc	1
nop	2

<?php

namespace allejo\DaPulse\Objects;

/**
 * Class PulseColumnTextValue
 *
 * @package allejo\DaPulse\Objects
 * @since   0.2.1
 */
class PulseColumnTimelineValue extends PulseColumnValue
{
    /**
     * Get a timeline column's content
     *
     * @api
     *
     * @since  0.2.1
     *
     * @return \DateTime[] The timeline's begin and end dates
     *
     *     array(
     *       'from' => \DateTime
     *       'to'   => \DateTime
     *     )
     */
    public function getValue ()
    {
        if (!isset($this->column_value))
        {
            $this->column_value = array(
                'from' => new \DateTime($this->jsonResponse["value"]["from"]),
                'to'   => new \DateTime($this->jsonResponse["value"]["to"])
            );
        }

        return $this->column_value;
    }

    /**
     * Update the values of a timeline column
     *
     * @api
     *
     * @param \DateTime $from
     * @param \DateTime $to
     *
     * @since 0.2.1
     */
    public function updateValue ($from, $to)

    {
        $url        = sprintf("%s/%d/columns/%s/timeline.json", self::apiEndpoint(), $this->board_id, $this->column_id);
        $postParams = array(
            "pulse_id" => $this->pulse_id,
            "from"     => $from->format('Y-m-d'),
            "to"       => $to->format('Y-m-d')
        );

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

        $this->column_value = array(
            'from' => $from,
            'to'   => $to
        );
    }
}

1			<?php
2
3			namespace allejo\DaPulse\Objects;
4
5			/**
6			* Class PulseColumnTextValue
7			*
8			* @package allejo\DaPulse\Objects
9			* @since 0.2.1
10			*/
11			class PulseColumnTimelineValue extends PulseColumnValue
12			{
13			/**
14			* Get a timeline column's content
15			*
16			* @api
17			*
18			* @since 0.2.1
19			*
20			* @return \DateTime[] The timeline's begin and end dates
21			*
22			* array(
23			* 'from' => \DateTime
24			* 'to' => \DateTime
25			* )
26			*/
27			public function getValue ()
28			{
29			if (!isset($this->column_value))
30			{
31			$this->column_value = array(
32			'from' => new \DateTime($this->jsonResponse["value"]["from"]),
33			'to' => new \DateTime($this->jsonResponse["value"]["to"])
34			);
35			}
36
37			return $this->column_value;
38			}
39
40			/**
41			* Update the values of a timeline column
42			*
43			* @api
44			*
45			* @param \DateTime $from
46			* @param \DateTime $to
47			*
48			* @since 0.2.1
49			*/
50			public function updateValue ($from, $to)
			0 ignored issues – show Comprehensibility introduced 2017-01-09 17:35 UTC by Report Bug Copy Issue Report Avoid variables with short names like $to. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
51			{
52			$url = sprintf("%s/%d/columns/%s/timeline.json", self::apiEndpoint(), $this->board_id, $this->column_id);
53			$postParams = array(
54			"pulse_id" => $this->pulse_id,
55			"from" => $from->format('Y-m-d'),
56			"to" => $to->format('Y-m-d')
57			);
58
59			self::sendPut($url, $postParams);
60
61			$this->column_value = array(
62			'from' => $from,
63			'to' => $to
64			);
65			}
66			}

allejo / PhpPulse

Push — master ( c07f69...57c551 )

PulseColumnTimelineValue::updateValue() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like