diogo
/
symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Form] Replace methods in ChoiceView by public properties (PHP +100ms, Twig +400ms)

This commit is contained in:
Bernhard Schussek 2012-07-16 17:38:50 +02:00
parent d072f35ea0
commit 400c95bb4d
5 changed files with 8 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Symfony/Bundle/FrameworkBundle/Resources/views/Form/choice_widget_options.html.php
View File

@ -2,10 +2,10 @@
<?php if ($view['form']->isChoiceGroup($choice)): ?>
<optgroup label="<?php echo $view->escape($view['translator']->trans($index, array(), $translation_domain)) ?>">
<?php foreach ($choice as $nested_choice): ?>
<option value="<?php echo $view->escape($nested_choice->getValue()) ?>"<?php if ($view['form']->isChoiceSelected($form, $nested_choice)): ?> selected="selected"<?php endif?>><?php echo $view->escape($view['translator']->trans($nested_choice->getLabel(), array(), $translation_domain)) ?></option>
<option value="<?php echo $view->escape($nested_choice->value) ?>"<?php if ($view['form']->isChoiceSelected($form, $nested_choice)): ?> selected="selected"<?php endif?>><?php echo $view->escape($view['translator']->trans($nested_choice->label, array(), $translation_domain)) ?></option>
<?php endforeach ?>
</optgroup>
<?php else: ?>
<option value="<?php echo $view->escape($choice->getValue()) ?>"<?php if ($view['form']->isChoiceSelected($form, $choice)): ?> selected="selected"<?php endif?>><?php echo $view->escape($view['translator']->trans($choice->getLabel(), array(), $translation_domain)) ?></option>
<option value="<?php echo $view->escape($choice->value) ?>"<?php if ($view['form']->isChoiceSelected($form, $choice)): ?> selected="selected"<?php endif?>><?php echo $view->escape($view['translator']->trans($choice->label, array(), $translation_domain)) ?></option>
<?php endif ?>
<?php endforeach ?>

4
src/Symfony/Component/Form/Extension/Core/Type/ChoiceType.php
View File

@ -226,8 +226,8 @@ class ChoiceType extends AbstractType
$this->addSubForms($builder, $choiceView, $options);
} else {
$choiceOpts = array(
'value' => $choiceView->getValue(),
'label' => $choiceView->getLabel(),
'value' => $choiceView->value,
'label' => $choiceView->label,
'translation_domain' => $options['translation_domain'],
);

24
src/Symfony/Component/Form/Extension/Core/View/ChoiceView.php
View File

@ -23,14 +23,14 @@ class ChoiceView
*
* @var string
*/
private $value;
public $value;
/**
* The label displayed to humans.
*
* @var string
*/
private $label;
public $label;
/**
* Creates a new ChoiceView.
@ -43,24 +43,4 @@ class ChoiceView
$this->value = $value;
$this->label = $label;
}
/**
* Returns the choice value.
*
* @return string The view representation of the choice.
*/
public function getValue()
{
return $this->value;
}
/**
* Returns the choice label.
*
* @return string The label displayed to humans.
*/
public function getLabel()
{
return $this->label;
}
}

2
src/Symfony/Component/Form/FormRenderer.php
View File

@ -186,7 +186,7 @@ class FormRenderer implements FormRendererInterface
public function isChoiceSelected(FormView $view, ChoiceView $choice)
{
$value = $view->vars['value'];
$choiceValue = $choice->getValue();
$choiceValue = $choice->value;
if (is_array($value)) {
return false !== array_search($choiceValue, $value, true);

2
src/Symfony/Component/Form/Tests/Extension/Core/Type/CountryTypeTest.php
View File

@ -38,7 +38,7 @@ class CountryTypeTest extends LocalizedTestCase
$choices = $view->vars['choices'];
foreach ($choices as $choice) {
if ('ZZ' === $choice->getValue()) {
if ('ZZ' === $choice->value) {
$this->fail('Should not contain choice "ZZ"');
}
}