tinymeng\spreadsheet\Connector\Gateway::verifyFile()

Complexity

Conditions	9
Paths	8

Size

Total Lines	27
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function verifyFile($path, $verifyFile = true, $zip = null){
        if ($verifyFile && preg_match('~^data:image/[a-z]+;base64,~', $path) !== 1) {
            // Check if a URL has been passed. https://stackoverflow.com/a/2058596/1252979
            if (filter_var($path, FILTER_VALIDATE_URL)) {
                $this->path = $path;

The property path does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                // Implicit that it is a URL, rather store info than running check above on value in other places.
                $this->isUrl = true;

The property isUrl does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                $imageContents = file_get_contents($path);
                $filePath = tempnam(sys_get_temp_dir(), 'Drawing');
                if ($filePath) {
                    file_put_contents($filePath, $imageContents);
                    if (file_exists($filePath)) {
                        return $filePath;
                    }
                }
            } elseif (file_exists($path)) {
                return $path;
            } elseif ($zip instanceof ZipArchive) {

The type tinymeng\spreadsheet\Connector\ZipArchive was not found. Did you mean ZipArchive? If so, make sure to prefix the type with \.

                $zipPath = explode('#', $path)[1];
                if ($zip->locateName($zipPath) !== false) {
                    return $path;
                }
            } else {
                throw new PhpSpreadsheetException("File $path not found!");
            }
        }
        return $path;
    }