Skip to content

Commit

Permalink
[Form] Date, Time, DateTimeType forget translation domain
Browse files Browse the repository at this point in the history
  • Loading branch information
helmer committed Nov 30, 2011
1 parent f7fb34a commit 0b5c4b2
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 8 deletions.
Expand Up @@ -56,7 +56,8 @@ public function buildForm(FormBuilder $builder, array $options)
'empty_value',
'required',
'invalid_message',
'invalid_message_parameters'
'invalid_message_parameters',
'translation_domain',
)));
$timeOptions = array_intersect_key($options, array_flip(array(
'hours',
Expand All @@ -66,7 +67,8 @@ public function buildForm(FormBuilder $builder, array $options)
'empty_value',
'required',
'invalid_message',
'invalid_message_parameters'
'invalid_message_parameters',
'translation_domain',
)));

// If `widget` is set, overwrite widget options from `date` and `time`
Expand Down
8 changes: 5 additions & 3 deletions src/Symfony/Component/Form/Extension/Core/Type/DateType.php
Expand Up @@ -80,22 +80,24 @@ public function buildForm(FormBuilder $builder, array $options)
array_combine($options['years'], $options['years']), 4, '0', STR_PAD_LEFT
),
'empty_value' => $options['empty_value']['year'],
'required' => $options['required'],
);
$monthOptions = array(
'choice_list' => new MonthChoiceList(
$formatter, $options['months']
),
'empty_value' => $options['empty_value']['month'],
'required' => $options['required'],
);
$dayOptions = array(
'choice_list' => new PaddedChoiceList(
array_combine($options['days'], $options['days']), 2, '0', STR_PAD_LEFT
),
'empty_value' => $options['empty_value']['day'],
'required' => $options['required'],
);

// Append generic carry-along options
foreach (array('required', 'translation_domain') as $passOpt) {
$yearOptions[$passOpt] = $monthOptions[$passOpt] = $dayOptions[$passOpt] = $options[$passOpt];
}
}

$builder
Expand Down
11 changes: 8 additions & 3 deletions src/Symfony/Component/Form/Extension/Core/Type/TimeType.php
Expand Up @@ -53,14 +53,12 @@ public function buildForm(FormBuilder $builder, array $options)
array_combine($options['hours'], $options['hours']), 2, '0', STR_PAD_LEFT
),
'empty_value' => $options['empty_value']['hour'],
'required' => $options['required'],
);
$minuteOptions = array(
'choice_list' => new PaddedChoiceList(
array_combine($options['minutes'], $options['minutes']), 2, '0', STR_PAD_LEFT
),
'empty_value' => $options['empty_value']['minute'],
'required' => $options['required'],
);

if ($options['with_seconds']) {
Expand All @@ -69,9 +67,16 @@ public function buildForm(FormBuilder $builder, array $options)
array_combine($options['seconds'], $options['seconds']), 2, '0', STR_PAD_LEFT
),
'empty_value' => $options['empty_value']['second'],
'required' => $options['required'],
);
}

// Append generic carry-along options
foreach (array('required', 'translation_domain') as $passOpt) {
$hourOptions[$passOpt] = $minuteOptions[$passOpt] = $options[$passOpt];
if ($options['with_seconds']) {
$secondOptions[$passOpt] = $options[$passOpt];
}
}
}

$builder
Expand Down

0 comments on commit 0b5c4b2

Please sign in to comment.