Redirectが期待した動作とならない

viewのフォームに入力した値をControllerで判定し特定の条件に一致する場合に、
元のview(入力したフォーム)にインプットデータ付きかつ、エラーメッセージ($message)付きで返そうとしています。
ところが、以下の３パターンのいずれでもtopページへのリダイレクトとなります。
試しにwith、withInputを削っても同様です。
意図せず、リダイレクト先をオーバーライドするような書き方をしたのかと疑いもしましたが、見当もつかず・・・

どうすべきか、何が問題なのか教えていただきたいです。

　　　パターン１
        $previous=url()->previous();
        return redirect($previous)->withInput()->with(['message'=>$message]);

　　　パターン２
        return redirect()->action('UpdateController@show')->withInput()->with(['message'=>$message]);
　　　
　　　パターン３
        return back()->WithInput()->with(['message'=>$message]);

当然ながら、$previousをddで見ても、期待したURLとなっていますが、実際にはtopへとリダイレクトされてしまう状況です。

なお、条件は入力値が検索してhitしない場合のみエラーとなるように書いています。

Routingは以下の通りです。
元はpostのみだったのですが、Redirectするにあたり、getとなってしまうため、以下のように変えました。

Route::match(['get','post'],'User/show', 'UpdateController@show');

クリックジャッキング対策として、以下middlewareを追加しています。
その他に、通常のMVC関連以外のファイルは作成していません。

class AddXFrameOptions
{
    public function handle($request, Closure $next)
    {
        $response = $next($request);
        $response->headers->set('X-Frame-Options', 'deny');
        return $response;
    }
}

退会済みユーザー

2020/09/24 21:45

何かしらの条件を満たしたときにtopへリダイレクトするようになってて、そこを通っちゃってる・・・なんてことはないですよね。

azot2

2020/09/25 00:00

Controller等、自分で触る範囲にそういった記述はないです。デバッグでもパターン１～３のあとは、framework側の処理しか動いていないことは確認しています。 framework側の処理も目で追っては見たんですが、知識不足から原因がわからない状態です。

kgm

2020/09/27 14:31

もしかして、ミドルウェアで飛び先変えているのでは？

azot2

2020/09/28 00:52

middlewareについて、追記しました。これでRedirectに影響するとは思えないんですが、確かにリクエストヘッダをいじっているので、そのせいなんでしょうか？

mix-peach

2020/09/29 01:22

Route::match(['get','post'],'User/show', 'UpdateController@show'); がリダイレクトで戻りたい「入力画面」ですよね？　この中身も提示していただけませんか？

azot2

2020/09/29 04:40 編集

ありがとうございます。おっしゃる通り、UpdateController@show内に指定パラメータが存在しない場合に、topへリダイレクトさせる処理を組み込んでいました。頭の中で勝手に思い込んでいたんですが、以下の動きとなるんですね。　誤：back()系の処理withInput付きで行うと、アクションが起こる前のViewを一発で表示させる　正：back()系の処理withInput付きで行っても、通常のController→Viewの流れで処理されるこれで原因はわかったのですが、↑の理解が正しければControllerにwithInputからデータを取り出す前提のコードを書かないといけないような気がしています。もっとスマートに書けないものでしょうか？ (誤)の理解のように動かすためにview('index')->WithInput()->with(['message'=>$message]); と書ければよかったんですが、そうもいかないようで新たな壁にぶつかっています。お手数ですが、「追記・修正」でなく、改めて回答していただければ後ほどベストアンサーに選ばせていただきます。

mix-peach

2020/09/29 05:25 編集

問題の原因には、ご自分で気づかれてますので、自己解決が良いかと思います＾＾ withInputで渡したデータ、ですが。これは、bladeファイルに　old()　で指定して簡単に使うことが出来ます。リファレンスの withInput() の近辺にも記載されていますので、読んでみてください。https://readouble.com/laravel/7.x/ja/requests.html URLからの推測で、ユーザーさんの情報が出ると仮定しサンプルを書くなら <input type="text" name="name" value="{{old('name', $user->name)}}"> こんな感じ。。。old('name', $user->name)　の第2引数（省略もできますが、今回のケースでは必要そうですね）は、前回の入力値がない場合にデフォルト表示する値を指定できます。これで、UpdateController@show　の中では【withInputで渡されるかもしれないデータ】を意識しなくても、view側だけで解決できる・・という仕組みです。

azot2

2020/10/09 10:53 編集

なかなか返信できずに、申し訳ありませんでした。リダイレクトされた2周目の処理と、１周目の処理がバッティングして意図しない処理とならないように、Controlerに都度if文を追加する形にはなりましたが、無事oldを使用して期待した値を取り出すことが出来ました。ありがとうございました。