fix bug : CFChoice was not rendered correctly with _other option

This commit is contained in:
Julien Fastré 2014-11-27 13:54:58 +01:00
parent 65b8499458
commit a925533fc8
2 changed files with 9 additions and 12 deletions

View File

@ -185,11 +185,8 @@ class CustomFieldChoice implements CustomFieldInterface
public function render($value, CustomField $customField)
{
//extract the data. They are under a _choice key if they are stored with allow_other
$data = (isset($value['_choices'])) ? $value['_choices'] :
is_null($value) ? array() : $value;
$selected = (is_array($data)) ? $data : array($data);
$data = (isset($value['_choices'])) ? $value['_choices'] : $value;
$selected = (is_array($data)) ? $data : array($data);
$choices = $customField->getOptions()[self::CHOICES];
if (in_array('_other', $selected)){

View File

@ -5,15 +5,15 @@
{%- if is_selected -%}
<li class="{% if is_selected or expanded == 1 %} selected {% endif %}">
{%- if is_selected -%}
<i class="fa fa-check-square-o"></i>
<i class="fa fa-check-square-o"></i>&nbsp;
{%- else -%}
<i class="fa fa-square-o"></i>
<i class="fa fa-square-o"></i>&nbsp;
{%- endif -%}
{%- if choice['slug'] is not same as('_other') -%}
{{ choice['name'][app.request.locale]|default(choice['name'][locale]|default('Not available in your language'|trans)) }}
{%- else -%}
{{ choice['name'] }}
{%- endif -%}
{%- if choice['slug'] is not same as('_other') -%}
{{ choice['name'][app.request.locale]|default(choice['name'][locale]|default('Not available in your language'|trans)) }}
{%- else -%}
<span class="other">{{ 'Other value'|trans }}&nbsp;:</span> {{ choice['name'] }}
{%- endif -%}
</li>
{%- endif -%}
{%- endfor -%}