Pnz\MattermostClient\Api\Files

Complexity

Total Complexity

5

Size/Duplication

Total Lines	65
Duplicated Lines	15.38 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	sendFile()	0	33	3
A	getFileMetadata()	10	10	2

<?php

declare(strict_types=1);

namespace Pnz\MattermostClient\Api;

use Http\Message\MultipartStream\MultipartStreamBuilder;
use Pnz\MattermostClient\Exception\InvalidArgumentException;
use Pnz\MattermostClient\Model\Channel\Channel;
use Pnz\MattermostClient\Model\Channel\ChannelMember;
use Pnz\MattermostClient\Model\Channel\ChannelMembers;
use Pnz\MattermostClient\Model\Channel\ChannelStats;
use Pnz\MattermostClient\Model\File\FileMetadata;
use Pnz\MattermostClient\Model\Post\Posts;

This use statement conflicts with another class in this namespace, Pnz\MattermostClient\Api\Posts.

use Pnz\MattermostClient\Model\Status;
use Psr\Http\Message\ResponseInterface;

final class Files extends HttpApi
{
    /**
     * Upload a File to a channel.
     *
     *
     * @see: https://api.mattermost.com/v4/#tag/files%2Fpaths%2F~1files%2Fpost
     *
     * @return FileInfo|ResponseInterface
     */
    public function sendFile(string $fileContents, string $channelId, string $clientId = null)

The parameter $clientId is not used and could be removed.

    {
        if (empty($fileContents)) {
            throw new InvalidArgumentException('File contents can not be empty');
        }

        if (empty($channelId)) {
            throw new InvalidArgumentException('ChannelID can not be empty');
        }


        $headers = [];
        $multipartStreamBuilder = new MultipartStreamBuilder();

        // Add channelID
        $multipartStreamBuilder->addResource('channel_id', $channelId);
        // Add file contents
        $multipartStreamBuilder->addResource('files', base64_encode(substr($fileContents, 0, 100)));
        // Add client id
        $multipartStreamBuilder->addResource('client_ids', 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa');

        $multipartStream = $multipartStreamBuilder->build();
        $headers['Content-Type'] = 'multipart/form-data; boundary='.$multipartStreamBuilder->getBoundary();
        $multipartStreamBuilder->reset();

        //var_dump($fileContents, $multipartStream->getContents(), $headers);

74% of this comment could be valid code. Did you maybe forget this after debugging?

        $response = $this->httpPostRaw(sprintf('/files', $channelId), $multipartStream, $headers);

        var_dump($response->getBody()->getContents());

var_dump($response->getBody()->getContents()); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

        // return $this->handleResponse($response, Posts::class);

60% of this comment could be valid code. Did you maybe forget this after debugging?

    }


    /**
     * Get a file metadata.
     *
     * @param string $fileId
     *
     * @see https://api.mattermost.com/v4/#tag/files%2Fpaths%2F~1files~1%7Bfile_id%7D~1info%2Fget
     *
     * @return FileMetadata|ResponseInterface
     */
    public function getFileMetadata(string $fileId)

This method seems to be duplicated in your project.

    {
        if (empty($fileId)) {
            throw new InvalidArgumentException('FileID can not be empty');
        }

        $response = $this->httpGet(sprintf('/files/%s', $fileId));

        return $this->handleResponse($response, FileMetadata::class);
    }
}