Laravel5で別々のメールを2人に送る方法を教えてください

Question

現在ユーザーの入力した情報をメールでユーザーと管理者それぞれに送る機能を実装しようとしています。

ユーザーにはサンクスメールを、管理者には入力した内容一式をメールで送信します。
こちらどちらか片方だけの時には正しく送信できるのですが、両方一度にとなるとうまく動作しません。
以下は記述しているコードです。

app/Http/Controllers/TestController.php

class TestController extends Controller
{
    public function index(){
        return view('test.index');
    }

    public function store(Request $request){
        $validator = Validator::make($request->all(), [
            'first_name' => 'required',
            'email' => 'required|email|confirmed',
            'body' => 'required',
        ]);

        if ($validator->fails()) {
            return redirect()
                        ->back()
                        ->withErrors($validator)
                        ->withInput();
        }
        else {
            $data = $request -> all();
            $request -> session() -> put($data);

            return view('test.confirm', compact("data"));
        }
    }

    public function confirm(Request $request){
        $data = $request -> all();
        $request -> session() -> put($data);

        return view('test.confirm', compact("data"));
    }

    public function complete(){
        $data = session() -> all();

        $this -> sendMailToUser();
        $this -> sendMailToAdmin();

        return view('test.complete');
    }

    public function sendMailToUser(Request $request){
        $data = session() -> all();

        Mail::send(['text' => 'test.mailtouser'], $data, function($message) use($data){
            $message -> to($data['email']) -> subject("お問い合わせを受け付けました");
        });
    }

    public function sendMailToAdmin(Request $request){
        $data = session() -> all();

        Mail::send(['text' => 'test.mailtoadmin'], $data, function($message) use($data){
            $message -> to('[e-mail]') -> subject("お問い合わせ受付");
        });
    }
}

route.php ( 当方 Laravel5.3 のため、実際は routes/web.php )

Route::resource('/test', 'TestController', ['only' => ['index', 'store']]);

Route::get('/test/confirm', function () {
    return view('test.confirm');
});

Route::get('/test/complete', function () {
    return view('test.complete');
});

Route::post('/test/complete','TestController@sendMailToUser');
Route::post('/test/complete','TestController@sendMailToAdmin');

ユーザーの入力内容を一旦セッションに記憶した後、
確認画面で内容を表示、そこで送信ボタンを押すと
セッションの内容を mailtoadmin.blade.php でまとめてメールで管理者に送信、
その後にユーザーに mailtouser.blade.php に書いた平文メールを送った後、
完了ページに遷移するという流れです。

このうち、ユーザーにメールを送る、という部分が実行できておりません。管理者には送信できています。
どの部分を修正すると二人に送信できますでしょうか。
ご教授よろしくお願いいたします。

追記

次のようにすることで二人に送ることができるようにはなりましたが、記述として正しくないような気がします。。
もっと良い書き方はございますでしょうか。

    public function sendMail(Request $request){
        $data = session() -> all();

        Mail::send(['text' => 'test.mailtouser'], $data, function($message) use($data){
            $message -> to($data['email']) -> subject("お問い合わせを受け付けました");
        });

        Mail::send(['text' => 'test.mailtoadmin'], $data, function($message) use($data){
            $message -> to('[e-mail]') -> subject("お問い合わせ受付");
        });
    }

Answer 1

少し気になったのはルートで/test/completeに対して2つコントローラーが設定されている点ですかね。この設定自体無理だった気がします。ルートは下に書いたものが優先で行われるので、結果として管理者側に送信されたのかと。一つのコントローラー内にまとめてみたらどうでしょうか？

【追記】
追記読みました。その書き方で特におかしい点は見当たらないので大丈夫だと思いますよ。むしろコントローラー分けてるほうが不自然に感じます。

Comments

[qwe001]

返信ありがとうございます。コントローラを２つに分けていたことに特に深い理由はありません。強いて言えば私の勉強不足が原因です。。記載のやり方で問題ないとのこと、確認ありがとうございます。安心しました。また何かありましたらよろしくお願いいたします。