【Laravel】確認画面から入力画面に戻った時のcheckbox値を保持する方法

Question

CRUDの流れを作っている際に、以下の件でハマりました。

入力 → 確認 → 完了　という流れで
確認から入力に戻った際、input type="text"の場合は「{{ old('ClientName') }}」でデータを保持しておけると思いますが
input type="checkbox"の場合、どのようにしてデータを保持するのか教えて頂きたいです。

現在、以下のようにコーディングしています。

■controllerファイル

    //追加ページ
    public function create() {
      $month = Client::$month;
      return view('client.create', compact('month'));
    }
    //追加の確認
    public function confirm(ClientRequest $request) {
      $data = $request->all();
      if (isset($request->month)) {
          $data['month'] = implode(', ',$request->month);
      }
      return view('client.confirm')->with($data);
    }

    //追加の処理
    public function store(ClientRequest $request) {
      $action = $request->get('action');// name=action の value名を取得
      $input = $request->except('action');// 入力内容を取得

      if ($action === "確定") {
        $clients = new Client();
        $clients->name = $request->ClientName;
        $clients->month = $request->ClientMonth;
        $clients->body = $request->ClientBody;
        $clients->save();
        return redirect('/client/list');
      } else {
        return redirect('/client/create')->withInput($input);
      }

    }

■viewファイル(入力画面)

<form class="" action="{{ url('/client/confirm') }}" method="post">
  {{ csrf_field() }}

  <p>
    <input type="text" name="ClientName" placeholder="名前" value="{{ old('ClientName') }}">
    @if($errors->has('ClientName'))
      <span class="error">{{ $errors->first('ClientName') }}</span>
    @endif
  </p>

  <p>
    @foreach($month as $key => $value)
      <label class="checkbox-inline">
        {!! Form::checkbox('month[]', $value) !!}
        {{ $value }}
      </label>
    @endforeach
  </p>

  <p>
    <textarea name="ClientBody" rows="8" cols="80" placeholder="本文">{{ old('ClientBody') }}</textarea>
    @if($errors->has('ClientBody'))
      <span class="error">{{ $errors->first('ClientBody') }}</span>
    @endif
  </p>

  <input type="submit" value="登録">
</form>

■viewファイル(確認画面)

<form class="" action="{{ url('/client') }}" method="post">

  <input type="hidden" name="ClientName" value="{{ $ClientName }}">
  <input type="hidden" name="ClientMonth" value="{{ $month }}">
  <input type="hidden" name="ClientBody" value="{{ $ClientBody }}">
  {{ csrf_field() }}

  この内容で登録します。

  <ul>
    <li>{{ $ClientName }}</li>
    <li>{{ $month }}</li>
    <li>{!! nl2br(e($ClientBody)) !!}</li>
  </ul>

  <input type="submit" name="action" value="戻る">
  <input type="submit" name="action" value="確定">

</form>

Answer 1

    // 追加ページ
    public function create()
    {
        // 配列が入っているなら Client::$month も Client::$months の複数形にしたほうが混乱が少ないかも
        return view('client.create', ['months' => Client::$month]);
    }

    // 追加の確認
    public function confirm(ClientRequest $request)
    {
        return view('client.confirm')
            ->with(compact('request'));
    }

    //追加の処理
    public function store(ClientRequest $request)
    {
        if ($request->action === '確定') {
            Client::make($request->only('name', 'body'))
                ->fill(['month' => implode(', ', $request->months ?? [])])
                ->save();
            return redirect('/client/list');
        }

        return redirect('/client/create')
            ->withInput($request->except('action'));
    }

<form class="" action="{{ url('/client/confirm') }}" method="post">
  {{ csrf_field() }}

  <p>
    {!! Form::text('name', old('name'), ['placeholder' => '名前']) !!}
    @if ($errors->has('name'))
      <span class="error">{{ $errors->first('name') }}</span>
    @endif
  </p>

  <p>
    @foreach ($months as $key => $value)
      <label class="checkbox-inline">
        {!! Form::checkbox("months[$key]", $value, old("months.$key")) !!}
        {{ $value }}
      </label>
    @endforeach
  </p>

  <p>
    {!! Form::textarea('body', old('body'), [
      'rows' => 8, 'cols' => 80, 'placeholder' => '本文',
    ]) !!}
    @if ($errors->has('body'))
      <span class="error">{{ $errors->first('body') }}</span>
    @endif
  </p>

  <input type="submit" value="登録">
</form>

<form class="" action="{{ url('/client') }}" method="post">
  {{ csrf_field() }}

  {!! Form::hidden('name', $request->name) !!}
  @foreach ($request->months ?? [] as $key => $value)
    {!! Form::hidden("months[$key]", $value) !!} <!-- ← ここで配列にしていなかったのが根本的な原因 -->
  @endforeach
  {!! Form::hidden('body', $request->body) !!}

  この内容で登録します。

  <ul>
    <li>{{ $request->name }}</li>
    <li>{{ implode(', ', $request->months ?? []) }}</li>
    <li>{!! nl2br(e($request->body)) !!}</li>
  </ul>

  <input type="submit" name="action" value="戻る">
  <input type="submit" name="action" value="確定">

</form>

Comments

[SENNA0510]

お早いご返信ありがとうございます！

さっそく試してみたのですが、、、ダメでした。。。

[mpyw]

よく見たら $month の渡し方がそもそもおかしいような… implodeしたら配列じゃなくて文字列になってるはずですね

[SENNA0510]

文字列に変換しないと配列のままでは以下のエラー文が出てしまうのです。
htmlspecialchars() expects parameter 1 to be string, array given

https://qiita.com/mpyw/items/f298828002bbc488fe89
上記URLなどを参照すると配列だからエラーになる？ようなことが書いてありました。

さらに文字列に変換する　というやり方は以下ページを参照しました。
https://www.webopixel.net/php/1316.html

[SENNA0510]

only('month')について

monthだけではなく他の項目として
ClientName
ClientBody
があります。

[SENNA0510]

やはり文字列に変換しないと

「htmlspecialchars() expects parameter 1 to be string, array given」が出てしまいますね。。。

[mpyw]

ご提示のコードでは $month を foreach で回していますが，これ以外の場所で $month が使われていませんか？「配列を期待している $month」「文字列を期待している $month」が混在していると思われます。名前がかぶらないようにしたほうがいいかも…

[SENNA0510]

変数名に被りがない事がわかった頂けるよう
コードをさっきよりも詳細に書き直したのでご確認して頂ければと思います。

[mpyw]

修正＆リファクタリングしてみました。