allejo\DaPulse\PulseColumn

Complexity

Total Complexity

12

Size/Duplication

Total Lines	91
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

12.12%

Importance

Changes	4
Bugs	0	Features	0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	getId()	0	4	1
A	getTitle()	0	4	1
A	getEmptyText()	0	4	1
A	getLabels()	0	4	1
A	getBoardId()	0	4	1
A	editTitle()	0	4	1
A	editLabels()	0	4	1
A	deleteColumn()	0	8	1
A	editField()	0	12	1
A	getColumnsUrl()	0	4	1
A	getType()	0	10	2

<?php

namespace allejo\DaPulse;

use allejo\DaPulse\Objects\ApiObject;

class PulseColumn extends ApiObject
{
    const API_PREFIX = "boards";

    const Date     = "date";

This class constant is not uppercase (expected DATE).

    const Numeric  = "numeric";

This class constant is not uppercase (expected NUMERIC).

    const Person   = "person";

This class constant is not uppercase (expected PERSON).

    const Status   = "status";

This class constant is not uppercase (expected STATUS).

    const Text     = "text";

This class constant is not uppercase (expected TEXT).

    const Timeline = "timerange";

This class constant is not uppercase (expected TIMELINE).

    protected $id;
    protected $title;
    protected $type;
    protected $empty_text;
    protected $labels;
    protected $board_id;

    public function getId ()
    {
        return $this->id;
    }

    public function getTitle ()
    {
        return $this->title;
    }

    public function getType ()
    {
        // @todo Workaround due to a bug in DaPulse's API see: https://github.com/allejo/PhpPulse/issues/5

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

        if ($this->type === "color")
        {
            $this->type = self::Status;
        }

        return $this->type;
    }

    public function getEmptyText ()
    {
        return $this->empty_text;
    }

    public function getLabels ()
    {
        return $this->labels;
    }

    public function getBoardId ()
    {
        return $this->board_id;
    }

    public function editTitle ($title)
    {
        $this->editField("title", $title);
    }

    public function editLabels ($labels)
    {
        $this->editField("labels", $labels);
    }

    public function deleteColumn ()
    {
        $this->checkInvalid();

        self::sendDelete($this->getColumnsUrl());

        $this->deletedObject = true;
    }

    private function editField ($field, $value)
    {
        $this->checkInvalid();

        $postParams = array(
            $field => $value
        );

        self::sendPut($this->getColumnsUrl(), $postParams);

        $this->$field = $value;
    }

    private function getColumnsUrl ()
    {
        return sprintf("%s/%d/columns/%s.json", parent::apiEndpoint(), $this->getBoardId(), $this->getId());

It seems like you call parent on a different method (apiEndpoint() instead of getColumnsUrl()). Are you sure this is correct? If so, you might want to change this to $this->apiEndpoint().

    }
}