Вам нужно заключить ключ value
в:
echo "<option value=\"" . $month_name . "\">" . $month_name . "</option>";
Кроме того, я бы предпочел что-то вроде этого:
$months = array("Jan", "Feb", "Mar", ..., "Dec");
foreach ($months as $month) {
echo "<option value=\"" . $month . "\">" . $month . "</option>";
}
Кажется странным, что вы сделали бы все эти ненужные вызовы date
и mktime
, когда вы знаете , какими должны быть значения.
Эта версия массива имеет такое же количество строк, и кажется, что намерение намного яснее (по крайней мере, мне).