allejo\DaPulse\PulseBoard

Complexity

Total Complexity

28

Size/Duplication

Total Lines	473
Duplicated Lines	5.71 %

Coupling/Cohesion

Components	2
Dependencies	5

Importance

Changes	19
Bugs	1	Features	7

20 Methods

Rating	Name	Duplication	Size	Complexity
A	getUrl()	0	4	1
A	getId()	0	4	1
A	getName()	0	4	1
A	getDescription()	0	4	1
A	getColumns()	0	9	1
A	getCreatedAt()	0	6	1
A	getUpdatedAt()	0	6	1
A	getSubscribers()	0	6	1
A	addSubscriber()	0	15	2
A	removeSubscriber()	11	11	2
B	createColumn()	0	25	6
A	getGroups()	0	9	1
A	createGroup()	0	11	1
A	deleteGroup()	0	6	1
A	getPulses()	0	15	2
A	createPulse()	0	17	1
A	pulseInjection()	7	7	1
A	archiveBoard()	9	9	1
A	createBoard()	0	14	1
A	getBoards()	0	6	1

<?php

/**
 * This file contains the PulseBoard class
 *
 * @copyright 2015 Vladimir Jimenez
 * @license   https://github.com/allejo/PhpPulse/blob/master/LICENSE.md MIT
 */

namespace allejo\DaPulse;

use allejo\DaPulse\Exceptions\ArgumentMismatchException;
use allejo\DaPulse\Exceptions\InvalidArraySizeException;
use allejo\DaPulse\Objects\ApiObject;

/**
 * This class contains all of the respective functionality for working a board on DaPulse
 *
 * @api
 * @package allejo\DaPulse
 * @since   0.1.0
 */
class PulseBoard extends ApiObject

The property $created_at is not named in camelCase.

The property $updated_at is not named in camelCase.

{
    /**
     * The suffix that is appended to the URL to access functionality for certain objects
     *
     * @internal
     */
    const API_PREFIX = "boards";

    // =================================================================================================================
    //   Instance Variables
    // =================================================================================================================

    /**
     * The resource's URL.
     *
     * @var string
     */
    protected $url;

    /**
     * The board's unique identifier.
     *
     * @var int
     */
    protected $id;

    /**
     * The board's name.
     *
     * @var string
     */
    protected $name;

    /**
     * The board's description.
     *
     * @var string
     */
    protected $description;

    /**
     * The board's visible columns.
     *
     * @var array
     */
    protected $columns;

    /**
     * Creation time.
     *
     * @var \DateTime
     */
    protected $created_at;

    /**
     * Last update time.
     *
     * @var \DateTime
     */
    protected $updated_at;

    // =================================================================================================================
    //   Getter functions
    // =================================================================================================================

    /**
     * The resource's URL.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return string
     */
    public function getUrl()
    {
        return $this->url;
    }

    /**
     * The board's unique identifier.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return int
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * The board's name.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return string
     */
    public function getName()
    {
        return $this->name;
    }

    /**
     * The board's description.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return string
     */
    public function getDescription()
    {
        return $this->description;
    }

    /**
     * The board's visible columns.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return PulseColumn[]
     */
    public function getColumns()
    {
        self::lazyInject($this->columns, array(
            "board_id" => $this->getId()
        ));
        self::lazyArray($this->columns, "PulseColumn");

        return $this->columns;
    }

    /**
     * Creation time.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return \DateTime
     */
    public function getCreatedAt()
    {
        self::lazyLoad($this->created_at, "DateTime");

        return $this->created_at;
    }

    /**
     * Last update time.
     *
     * @api
     *
     * @since  0.1.0
     *
     * @return \DateTime
     */
    public function getUpdatedAt()
    {
        self::lazyLoad($this->updated_at, "DateTime");

        return $this->updated_at;
    }

    // =================================================================================================================
    //   Subscriber functions
    // =================================================================================================================

    /**
     * Get the users who are subscribed to this board
     *
     * @api
     *
     * @param  array $params
     *
     * @since  0.1.0
     *
     * @return PulseUser[]
     */
    public function getSubscribers ($params = array())
    {
        $url = sprintf("%s/%d/subscribers.json", $this::apiEndpoint(), $this->getId());

        return self::fetchJsonArrayToObjectArray($url, "PulseUser", $params);
    }

    /**
     * Subscriber a user to a board
     *
     * @api
     *
     * @param int|PulseUser $user_id  The user that will be subscribed to the board
     * @param bool|null     $as_admin Set to true if the user will be an admin of the board
     *
     * @since 0.1.0
     */
    public function addSubscriber ($user_id, $as_admin = NULL)

The parameter $user_id is not named in camelCase.

The parameter $as_admin is not named in camelCase.

    {
        if ($user_id instanceof PulseUser)
        {
            $user_id = $user_id->getId();
        }

        $url = sprintf("%s/%d/subscribers.json", self::apiEndpoint(), $this->getId());
        $params = array(
            "user_id" => $user_id
        );

        self::setIfNotNullOrEmpty($params, "as_admin", $as_admin);

$as_admin is of type boolean|null, but the function expects a string.

        self::sendPut($url, $params);
    }

    /**
     * Unsubscribe a user from this board
     *
     * @api
     *
     * @param int|PulseUser $user_id The user that will be unsubscribed from the board
     *
     * @since 0.1.0
     */
    public function removeSubscriber ($user_id)

This method seems to be duplicated in your project.

The parameter $user_id is not named in camelCase.

    {
        if ($user_id instanceof PulseUser)
        {
            $user_id = $user_id->getId();
        }

        $url = sprintf("%s/%d/subscribers/%d.json", self::apiEndpoint(), $this->getId(), $user_id);

        self::sendDelete($url);
    }

    // =================================================================================================================
    //   Columns functions
    // =================================================================================================================

    /**
     * Create a new column for the current board.
     *
     * If you are creating a status column, use the constants available in the **PulseColumnColorValue** class to match
     * the colors. Keep in mind this array cannot have a key higher than 11 nor can it be an associative array. Here's
     * an example of how to match statuses with specific colors.
     *
     * ```php
     * $labels = array(
     *     PulseColumnColorValue::Orange  => "Working on it",
     *     PulseColumnColorValue::L_Green => "Done",
     *     PulseColumnColorValue::Red     => "Delayed"
     * );
     * ```
     *
     * @api
     *
     * @param string $title  The title of the column. This title will automatically be "slugified" and become the ID
     *                       of the column.
     * @param string $type   The type of value that this column will use. Either use the available constants in the
     *                       PulseColumn class or use the following strings: "date", "person", "status", "text".
     * @param array  $labels If the column type will be "status," then this array will be the values for each of the
     *                       colors.
     *
     * @see PulseColumn::Date   PulseColumn::Date
     * @see PulseColumn::Person PulseColumn::Person
     * @see PulseColumn::Status PulseColumn::Status
     * @see PulseColumn::Text   PulseColumn::Text
     * @see PulseColumnColorValue::Orange  PulseColumnColorValue::Orange
     * @see PulseColumnColorValue::L_Green PulseColumnColorValue::L_Green
     * @see PulseColumnColorValue::Red     PulseColumnColorValue::Red
     * @see PulseColumnColorValue::Blue    PulseColumnColorValue::Blue
     * @see PulseColumnColorValue::Purple  PulseColumnColorValue::Purple
     * @see PulseColumnColorValue::Grey    PulseColumnColorValue::Grey
     * @see PulseColumnColorValue::Green   PulseColumnColorValue::Green
     * @see PulseColumnColorValue::L_Blue  PulseColumnColorValue::L_Blue
     * @see PulseColumnColorValue::Gold    PulseColumnColorValue::Gold
     * @see PulseColumnColorValue::Yellow  PulseColumnColorValue::Yellow
     * @see PulseColumnColorValue::Black   PulseColumnColorValue::Black
     *
     * @since 0.1.0
     *
     * @throws ArgumentMismatchException Status definitions were defined yet the type of the column was not a status
     *                                   type column
     * @throws InvalidArraySizeException The array containing the value of statuses has a key larger than the
     *                                   supported 10 indices
     *
     * @return $this This instance will be updated to have updated information to reflect the new column that was
     *               created
     */
    public function createColumn ($title, $type, $labels = array())
    {
        if ($type !== PulseColumn::Status && !empty($labels))
        {
            throw new ArgumentMismatchException("No color definitions are required for a non-color column.");
        }

        if ($type === PulseColumn::Status && count($labels) > 0 && max(array_keys($labels)) > 10)
        {
            throw new InvalidArraySizeException("The range of status can only be from 0-10.");
        }

        $url        = sprintf("%s/%d/columns.json", self::apiEndpoint(), $this->getId());
        $postParams = array(
            "title" => $title,
            "type"  => $type
        );

        self::setIfNotNullOrEmpty($postParams, "labels", $labels);

$labels is of type array, but the function expects a string.

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

It seems like self::sendPost($url, $postParams) of type * is incompatible with the declared type array of property $jsonResponse.

        $this->assignResults();

        return $this;
    }

    // =================================================================================================================
    //   Group functions
    // =================================================================================================================

    /**
     * Get all of the groups belonging to a board.
     *
     * A group is defined as the colorful headers that split up pulses into categories.
     *
     * @api
     *
     * @param bool $show_archived Set to true if you would like to get archived groups in a board as well

Should the type for parameter $show_archived not be boolean|null?

     *
     * @since 0.1.0
     *
     * @return PulseGroup[]
     */
    public function getGroups ($show_archived = NULL)

The parameter $show_archived is not named in camelCase.

    {
        $url = sprintf("%s/%d/groups.json", self::apiEndpoint(), $this->getId());
        $params = array();

        self::setIfNotNullOrEmpty($params, "show_archived", $show_archived);

$show_archived is of type boolean|null, but the function expects a string.

        return self::fetchJsonArrayToObjectArray($url, "PulseGroup", $params);
    }

    /**
     * Create a new group in a board
     *
     * @api
     *
     * @param  string $title The title of the board
     *
     * @since  0.1.0
     *
     * @return PulseGroup
     */
    public function createGroup ($title)
    {
        $url        = sprintf("%s/%s/groups.json", self::apiEndpoint(), $this->getId());
        $postParams = array("title" => $title);

        // The API doesn't return the board ID, so since we have access to it here: set it manually
        $groupResult             = self::sendPost($url, $postParams);
        $groupResult["board_id"] = $this->getId();

        return (new PulseGroup($groupResult));
    }

    /**
     * Delete a group from a board
     *
     * @api
     *
     * @param string $group_id The group ID to be deleted
     *
     * @since 0.1.0
     */
    public function deleteGroup ($group_id)

The parameter $group_id is not named in camelCase.

    {
        $url = sprintf("%s/%d/groups/%s.json", self::apiEndpoint(), $this->getId(), $group_id);

        self::sendDelete($url);
    }

    // =================================================================================================================
    //   Pulse functions
    // =================================================================================================================

    /**
     * @return Pulse[]
     */
    public function getPulses ()
    {
        $url = sprintf("%s/%d/pulses.json", self::apiEndpoint(), $this->getId());
        $data = self::sendGet($url);
        $pulses = array();

        foreach ($data as $entry)
        {
            $this->pulseInjection($entry);

            $pulses[] = new Pulse($entry["pulse"]);
        }

        return $pulses;
    }

    public function createPulse ($name, $owner, $group_id = null)

The parameter $group_id is not named in camelCase.

    {
        $url = sprintf("%s/%d/pulses.json", self::apiEndpoint(), $this->getId());
        $postParams = array(
            "user_id" => $owner,
            "pulse" => array(
                "name" => $name
            )
        );

        self::setIfNotNullOrEmpty($postParams, "group_id", $group_id);

        $result = self::sendPost($url, $postParams);
        $this->pulseInjection($result);

        return (new Pulse($result["pulse"]));
    }

    private function pulseInjection (&$result)

This method seems to be duplicated in your project.

    {
        // Inject some information so a Pulse object can survive on its own
        $result["pulse"]["group_id"] = $result["board_meta"]["group_id"];
        $result["pulse"]["column_structure"] = $this->getColumns();
        $result["pulse"]["raw_column_values"] = $result["column_values"];
    }

    // =================================================================================================================
    //   Board functions
    // =================================================================================================================

    public function archiveBoard ()

This method seems to be duplicated in your project.

    {
        $this->checkInvalid();

        $url = sprintf("%s/%s.json", self::apiEndpoint(), $this->getId());
        self::sendDelete($url);

        $this->deletedObject = true;
    }

    public static function createBoard ($name, $user_id, $description = NULL)

The parameter $user_id is not named in camelCase.

    {
        $url        = sprintf("%s.json", self::apiEndpoint());
        $postParams = array(
            "user_id" => $user_id,
            "name"    => $name
        );

        self::setIfNotNullOrEmpty($postParams, "description", $description);

        $boardResult = self::sendPost($url, $postParams);

        return (new PulseBoard($boardResult));
    }

    public static function getBoards ($params = array())
    {
        $url = sprintf("%s.json", self::apiEndpoint());

        return self::fetchJsonArrayToObjectArray($url, "PulseBoard", $params);
    }
}