6

i have a form with a select input, validation is fine, but upon failure the select field doesn't populate the old value here is my select field

  <div class="control-group">
   <label class="control-label">Gender :</label>
    <div class="controls">
        <select  name="gender" value= "{{ Input::old('gender') }}">
            <option>Select</option>

                <option value="M">Male</option>
                <option value="F">Female</option>
        </select>
    </div>
</div> 

how can i solve this?

Marcin Nabiałek
  • 109,655
  • 42
  • 258
  • 291
dxcoder1
  • 289
  • 2
  • 4
  • 14

5 Answers5

11

If you don't want to use Laravel Form build, you need to do it this way:

 <div class="control-group">
   <label class="control-label">Gender :</label>
    <div class="controls">
        <select  name="gender">
            <option>Select</option>

                <option value="M" @if (Input::old('gender') == 'M') selected="selected" @endif>Male</option>
                <option value="F" @if (Input::old('gender') == 'F') selected="selected" @endif>Female</option>
        </select>
    </div>
</div> 
Marcin Nabiałek
  • 109,655
  • 42
  • 258
  • 291
3

I personally think this code is cleaner. There is no right or wrong here, I can understand why some would prefer the Laravel @if @endif statements, I just think they look too visually disturbing.

<option {{ old('gender')=='male' ? 'selected="selected"' : '' }}>Test option</option>
Henrik
  • 490
  • 2
  • 10
  • 16
1

This is due to HTML select element.

If you want to assign some default option value to the select element use the attribute selected="selected" for that particular option otherwise by default it will show the first option value.

<select  name="gender">
    <option>Select</option>
        <option value="M" @if (Input::old('gender') == 'M') selected="selected" @endif>Male</option>
        <option value="F" @if (Input::old('gender') == 'F') selected="selected" @endif>Female</option>
</select>
0

This is the way I do, very dynamic.

<select id="gender" name="gender">
    <option>Select</option>
        <option value="M">Male</option>
        <option value="F">Female</option>
</select>

<script>
    var currentGender = null;
    for(var i=0; i!=document.querySelector("#gender").querySelectorAll("option").length; i++)
    {
        currentGender = document.querySelector("#gender").querySelectorAll("option")[i];
        if(currentGender.getAttribute("value") == "{{ old("gender") }}")
        {
            currentGender.setAttribute("selected","selected");
        }
    }
</script>
0

I would recommend using Laravel Collective package.

{!! Form::label('gender', 'Gender', ['class' => 'control-label']) !!}
{!! Form::select('gender',
    ['M' => 'Male', 'F' => 'Female'],
    'F', //set default value (or null)
    ['class' => 'form-control', 'required' => true,]
) !!}

It takes care of the old value without using that ugly if-else injection into every select value.

bdereta
  • 883
  • 9
  • 20