BPT\api\telegram

Complexity

Total Complexity

6

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	downloadFile()	0	8	2
A	__call()	0	6	4

<?php

namespace BPT\api;
use BPT\settings;
use BPT\tools;

/**
 * telegram class , Adding normal method call to request class and a simple name for being easy to call
 */
class telegram extends request {
    public function __call (string $name, array $arguments) {
        if (!isset($arguments[1]) && isset($arguments[0]) && is_array($arguments[0])) {
            return request::$name(...$arguments[0]);
        }
        else {
            return request::$name(...$arguments);
        }
    }

    /**
     * download telegram file with file_id to destination location
     *
     * It has 20MB download limit(same as telegram)
     *
     * e.g. => tools::downloadFile('test.mp4');
     *
     * e.g. => tools::downloadFile('test.mp4','file_id_asdadadadadadad);
     *
     * @param string|null $destination destination for save the file
     * @param string|null $file_id     file_id for download, if not set, will generate by request::catchFields method
     *
     * @return bool
     */
    public static function downloadFile(string|null $destination = null, string|null $file_id = null): bool {
        $file = telegram::getFile();
        if (isset($file->file_path)) {
            $file_path = settings::$down_url.'bot'.settings::$token.'/'.$file->file_path;
            return tools::downloadFile($file_path,$destination);

It seems like $destination can also be of type null; however, parameter $path of BPT\tools::downloadFile() does only seem to accept string, maybe add an additional type check?

        }
        else {
            return false;
        }
    }
}