【laravel】日付をプルダウンで選択済みにしたい。

Question

前提・実現したいこと

PHP(laravel)でタスク管理のアプリを作成しています。
期日があるタスクを編集画面で表示し、
現在DBで保存されている期日から月だけをプルダウンですでに選択された状態にしたいと考えています。

発生している問題・エラーメッセージ

DBのデータに関わらず、プルダウンの一番最後にある「12」が選択された状態になります。
DBのデータを選択済みにするにはどのようにすればよいでしょうか。

該当のソースコード

期日がDBに記載がないタスクの月は"00"なので、"00"でない場合に選択された状態にしたいです。

     <!-- edit.blade.php -->
            <select id="month" name="month">
                <option value="">---
                </option>
                @foreach(range(1, 12) as $month)
                <option value="{{ $month }}" {{ old('month') == $month ? 'selected' : '' }}
                @if ($todo_month !== "00") selected @endif>{{ $month }}</option>
                @endforeach
            </select>
            <label for="month">月</label>

  <!-- controller -->
    public function edit($id)
    {
        //  todoおよびfrequencyテーブルのデータを取得
        $todo = Todo::find($id);
        $todo_year = date("Y",strtotime($todo->deadline));
        $todo_month = date("m",strtotime($todo->deadline));
        $todo_day = date("d",strtotime($todo->deadline));
        $frequency = frequency::where('todos_id', '=', $id)
            ->first();
        return view('todos.edit', compact('todo', 'frequency','todo_year','todo_month','todo_day'));
    }

// 日付の取得
function setDay() {
    // 年の値を取得
    const yearVal = document.getElementById('year').value;
    // 月の値を取得
    const monthVal = document.getElementById('month').value;
    // 日のセレクトボックスに挿入するHTML
    let html = '<option value="">---</option>';
    // 年月が有効な値の場合のみ日付の選択肢を加える
    if (yearVal !== '' && monthVal !== '') {
        // 特定の年月の最後の日付を取得する
        const lastDay = new Date(yearVal, monthVal, 0).getDate();
        // optionを組み立てる
        for (let day = 1; day <= lastDay; day++) {
            html += '<option value="' + day + '">' + day + '</option>';
        }
    }
    document.getElementById('day').innerHTML = html;
}

window.onload = function () {
    setDay();
    document.getElementById('year').addEventListener('change', setDay);
    document.getElementById('month').addEventListener('change', setDay);

    // リダイレクトした場合に元の入力値を復元する
    const dayElem = document.getElementById('day');
    dayElem.value = dayElem.getAttribute('data-old-value');
};

試したこと

$todo_monthはDBの月データ"02"を持ってきていることは確認しました。

補足情報（FW/ツールのバージョンなど）

PHP 7.4.9
laravel 6.18.35
MySQL 5.7.26

Comments

[YakumoSaki]

なんとなくなんですが、
@if ($todo_month !== "00") selected @endif
これだと、全部の月に selected がついてしまうように見えます。
ブラウザのソースの表示で、optionタグの出力を確認してみるとなにかヒントがあるかもしれません
（すべてのoptionにselectedがついてしまっているとか）

[youplus40]

回答ありがとうございます。
おっしゃるとおり、すべての数字がselectedになっておりました…。
optionでひとつひとつ値を指定したプルダウンの選択済みにする方法はわかるのですが、
今回のようにforeachで範囲指定している場合、範囲の中からひとつを指定することは可能でしょうか。

Answer 1

foreachが 1~12 でループするので、そもそも 00 の場合はヒットしないはずなので、
こんな感じでしょうか…
Form::select() を使うと楽にセレクトボックスを作れるのでそちらを使ったほうが良いかもしれません。

@foreach(range(1, 12) as $month) {{ old('month') == $month ? 'selected' : '' }}
<option value="{{ $month }}" @if ($todo_month == $month) selected @endif>
{{ $month }}</option>
@endforeach

Comments

[youplus40]

回答ありがとうございます。
ただいただいたコードでためしましたが、だめでした。どれもセレクトされてない状況になりました。
Form::selectも試みましたが、下記が分からかったです。
1.　$todo_monthが0の場合、「---」を表示したいがそれが難しそう
2.　valueで配列から値を指定するが、$todo_monthと同じ値を指定する方法がわからない