Laravel 画像更新

Question

バージョン

laravel 6.20.43
php 7.4.21
homestead

ご教授の程、よろしくお願い致します。

Comments

[m.ts10806]

なにをもって「保存されていない」を確認していますか？

[lala]

phpMyAdminのUser Tableのimageカラムの更新、public/imageフォルダの画像格納、ブラウザ表示変更と確認しております。

[m.ts10806]

それらすべてが変わってないのか、変わってるけど画面表示上変わってないのかどちらでしょうか。
あと、もともとの画像はどのようにアップロードされましたか？

[lala]

確認しているところは全て変更されていません。
元の画像はテーブルを作成する際にデフォルトで指定の画像が登録する様にしています。

[m.ts10806]

では次
・images以外は更新されていますか？
・UsersController.update()で$requestを参照していますが、入力した情報はすべて来ていますか？
・$this->validate()の実装はどうなっていますか？そして何かしら返却されても受け取ってないのでそのまま次に流れるようになってますが、バリデーションは想定の動作はしていますか？

[lala]

・ブラウザ及びphpMyAdminでも確認出来ており、images以外は更新出来ています。
・入力情報は全て出来ています。
・バリデーションは想定の動作していると思います。

[m.ts10806]

>入力情報は全て出来ています。
上記をどのように確認したのかを知りたいのですが、
例えば、
dd($images);
で何が出ますか？

記憶間違いだったら申し訳ないのですが、Laravelでファイルアップロードの際に参照するのって
$request->file('images')
のような形だったと思います。巣のPHPだと$_FILESですよね。
調べた感じでも $request->input()で参照している記事は見ません。

もちろん、「DBに何を保存して」「表示の際にどこからの何を表示させるか」は
仕様なのでそれによっても違うと思いますが。

[lala]

Undefined variable: images とエラーが出たため、上手く定義できていないことがわかりました。
確認方法が未熟でした
$request->input()を使わず、$request->file('images')などを使用して実装を試してみます。

[m.ts10806]

>Undefined variable: images とエラーが出たため

dd書く場所間違えてるのでは。

$images = $request->input('images');
dd($images);

たぶんNULLが出そうな気がします。

[m.ts10806]

>$request->input()を使わず、$request->file('images')などを使用して実装を試してみます。

そもそも$request->input()では送信したファイルは取得できないものと。
文字データではなくファイル実態ですから。

[lala]

>$images = $request->input('images');
>dd($images);
>たぶんNULLが出そうな気がします。

各場所誤っており上記の通り行ったら、nullと出ました
そのため修正します

[m.ts10806]

一応、一つの方向性として回答にしておきます。

[m.ts10806]

> https://teratail.com/help/avoid-asking
> 意図的に内容が抹消された質問

解決してなくても補足するわけでもなくほぼ抹消もやってることは同じ。
そのつもりはなくても「解決するつもりはなかったんだな」という印象を受けます。

Answer 1

ファイルアップロードの内容取得はRequest::input()ではなくRequest::file()を使います。
https://readouble.com/laravel/9.x/ja/filesystem.html#file-uploads

※巣のPHPでも$_POST/$_GETではなく$_FILESを利用しますので同じです。

Comments

[lala]

Request::input()ではなくRequest::file()に変更しました。
Call to a member function store() on nullとエラーが出ます。解決方法が分からないので教えて頂けますでしょうか？

[m.ts10806]

type=fileのnameはimagesですよね。

[lala]

ご指摘のとおりです。imagesに修正しましたが同じエラーのままです

[m.ts10806]

では結局1つずつ確認するしかありません。
dd($request->file('images'));

あと現状のupdate()の実装だと$imagesは未定義の変数になるので
そのあたりの考慮は必要だと思います。

[lala]

$user->images = $request->file('images')->store('public/');
$images = $request->file('images');
dd($images);
で行いましたが　Call to a member function store() on nullと出ます

[m.ts10806]

nullが起きる場所以降でdd()書いてもそこまで通りません。
追記修正依頼のコメントでやりとりしてるときと同じミスをしています。

[lala]

回答返信漏れており失礼しました。どの場所でdd($images);を記載しても状況が変わらなかったです

[m.ts10806]

dd()するとそこでdieするので（確か dump and dieの略）、何も変わらないはずはないです。

dd( $request->file('images'));
$user->images = $request->file('images')->store('public/');

こうするとstore()通りません。

[lala]

回答ありがとうございます。
試してみたところ
419 Page Expiredのエラーが出ました。

[m.ts10806]

あの、それはいいのですが、そこから自分で調べたりされないのでしょうか？
Laravelのバージョンも分かっているのなら、調べはすぐつくと思うのですが。

[lala]

調べても解決しない為新規で質問作成します。ご協力ありがとうございました

[m.ts10806]

本件未解決のまま放置のままとされるわけですね。
何をどう調べて解決しないか提示しないことには結局丸投げになると思いますし、
申し訳ないですが、質問者さんにはまだLaravel手を出すのは早すぎると感じました。
私自身Laravelは4か5で止まってる状態ですが、Webの基本的な知識があれば理解できて
解決できることも多いので、まずは問題の切り分け方や調べ方から学ぶのを強くすすめます。

今回で言えば
Laravel ファイルアップロード
から
Laravel 419
で、バージョンに合わせた情報を調べれば前に進むものです。
6と7って結構大きな仕様変更あったみたいですよ。
互換性ゼロではないでしょうけど。バージョンが違う情報参照してるのでしたら解決しないのは当然とも言えます。

Answer 2

自己解決とさせていただき、閉じます

Comments

[m.ts10806]

自己解決としても回答が何も具体的でないので低評価しています。

> https://teratail.com/help#resolve-myself
> また後から読む人のためにも、具体的な解決手順を記載してください。