Как исправить Array to string conversion?

Question

[kaxa3201]

$arrayFiles = [];
        foreach ($files as $file) {
            $arrayFiles[] = (StorageFactory::make('minio'))->store($file->getClientOriginalName(),
                file_get_contents($file));
        }

        return $arrayFiles;

Получаю ошибку
Array to string conversion

public function run()
    {
        $matcher = new UrlMatcher($this->routes, $this->requestContext);
        try {
            $this->request->attributes->add($matcher->match($this->request->getPathInfo()));

            $this->controller = $this->getController();

            $this->arguments = $this->getArguments($this->controller);
            $response = $this->executeAction();
        } catch (ResourceNotFoundException $exception) {
            $response = new Response('Not Found', 404);
        } catch (Exception $exception) {
            $response = new Response('An error occurred', 500);
        }

        if (!empty($response) && is_object($response)) {
            $response->send();
        } else {
            echo $response;
        }
    }

указывает на эту строку
else {
echo $response;
}

Comments

[irishmann]

// Ванга-мод ON

[1210mk2]

php 7? в самом методе возвращаемый тип string?

[kaxa3201]

1210mk2, php 7.2

$arrayFiles = [];
        foreach ($files as $file) {
            $arrayFiles[] = (StorageFactory::make('minio'))->store($file->getClientOriginalName(),
                file_get_contents($file));
        }

        var_dump($arrayFiles);

array (size=2)
  0 => string 'http://172.19.0.7:9000/downloads/%D0%92%D1%8B%D0%B4%D0%B5%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5_002.png' (length=95)
  1 => string 'http://172.19.0.7:9000/downloads/%D0%92%D1%8B%D0%B4%D0%B5%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5_001.png' (length=95)

[1210mk2]

kaxa3201, это норм, саму шапку метода дайте.
типа public function get_something() {

[kaxa3201]

1210mk2,

public function upload($files)
    {
        $arrayFiles = [];
        foreach ($files as $file) {
            $arrayFiles[] = (StorageFactory::make('minio'))->store($file->getClientOriginalName(),
                file_get_contents($file));
        }

        var_dump($arrayFiles);
    }

public function store($name, $content)
    {
        try {
            $result = $this->s3->putObject([
                'Bucket' => 'downloads',
                'Key'    => $name,
                'Body'   => $content,
            ]);
            return $result->get('ObjectURL');
        } catch (\Exception $e) {
            Log::debug('StorageService exception: ' . $e->getMessage());

            return null;
        }
    }

[1210mk2]

kaxa3201, это значит не то, о чем я подумал. как написали уже в ответах - echo $response; не прокатывает, если это массив. преобразовывайте.

Answer 1

[Григорий Васильков]

Во первых ты пытаешься file_get_contents() от объекта $file. Эта функция на вход хочет строку.
Возможно у твоего обьекта file есть какой-нибудь arrayAccess или __toString() почему-то возвращающий массив

$arrayFiles[] = (StorageFactory::make('minio'))->store($file->getClientOriginalName(), file_get_contents($file));

Может как-то так
file_get_contents($file->getClientFilePath())

функцию getClientFilePath() я с потолка взял. Она не имеет смысла кстати. Путь на клиенте тебе не доступен для чтения. Если это файл отправляемый пользователем там будет что-то $file->getTmpPath(), куда он временно закачался для твоего скрипта и откуда исчезнет если ничего не сделать. И там будет не file_get_contents() а какой-нибудь copy($file->getTmpPath(), $new_location);

Второй код ничего не говорит и даже не вызывает твой $arrayFiles никак не привязан

Что до $response, то если в нем $arrayFiles; то сделать "echo Array()" нельзя без такой ошибки. Сначала Array() нужно конвертировать в строку с помощью json_encode() или там serialize() или другим способом

Comments

[kaxa3201]

Григорий Васильков "И там будет не file_get_contents() а какой-нибудь copy($file->getTmpPath(), $new_location);"
у меня будет записывать в minio поэтому copy() же не нужно
что то типа такого

$url = (StorageFactory::make(StorageServiceContract::STORAGE_MINIO))->store($file->getClientOriginalName(),
            file_get_contents($file->getPathName()));

[Григорий Васильков]

kaxa3201, ну да, вот, точно