Polidog\Chatwork\Api\Rooms\Files

Complexity

Total Complexity

4

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes	4
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	show()	0	13	2
A	detail()	0	7	1

<?php

declare(strict_types=1);

namespace Polidog\Chatwork\Api\Rooms;

use Polidog\Chatwork\Client\ClientInterface;
use Polidog\Chatwork\Entity\Collection\CollectionInterface;
use Polidog\Chatwork\Entity\Factory\FileFactory;
use Polidog\Chatwork\Entity\File;

/**
 * Class Files.
 */
class Files
{
    /**
     * @var ClientInterface
     */
    private $client;

    /**
     * @var FileFactory
     */
    private $factory;

    /**
     * @var int
     */
    private $roomId;

    /**
     * Files constructor.
     *
     * @param ClientInterface $client
     * @param FileFactory     $factory
     * @param int             $roomId
     */
    public function __construct(ClientInterface $client, FileFactory $factory, int $roomId)
    {
        $this->client = $client;
        $this->factory = $factory;
        $this->roomId = $roomId;
    }

    /**
     * @param null $accountId

Are you sure the doc-type for parameter $accountId is correct as it would always require null to be passed?

     *
     * @return CollectionInterface
     */
    public function show($accountId = null)
    {
        $options = [
        ];

        if (!is_null($accountId)) {

The condition is_null($accountId) is always true.

            $options['account_id'] = $accountId;
        }

        return $this->factory->collection(
            $this->client->get(
                "rooms/{$this->roomId}/files",
                $options
            )
        );
    }

    /**
     * @param $id
     * @param $downloadLink
     *
     * @return File
     */
    public function detail($id, $downloadLink = false)
    {
        return $this->factory->entity(
            $this->client->get(
                "rooms/{$this->roomId}/files/{$id}",
                [
                    'create_download_url' => (int) $downloadLink,
                ]
            )
        );
    }
}