From 797a42e6d23e8dce97fdde5bc5422bcb0eace27b Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Fri, 11 May 2018 14:56:57 +0200
Subject: [PATCH] Bigfix #234

Rimozione dell'opzione vuote nel caso di select multipli (necessaria invece per select singoli).
---
 src/HTMLBuilder/Handler/SelectHandler.php | 38 ++++++++++-------------
 1 file changed, 17 insertions(+), 21 deletions(-)

diff --git a/src/HTMLBuilder/Handler/SelectHandler.php b/src/HTMLBuilder/Handler/SelectHandler.php
index d5d2be198..f36975f37 100644
--- a/src/HTMLBuilder/Handler/SelectHandler.php
+++ b/src/HTMLBuilder/Handler/SelectHandler.php
@@ -40,30 +40,26 @@ class SelectHandler implements HandlerInterface
             if (!empty($values['value']) || is_numeric($values['value'])) {
                 $result .= $this->select2($values['ajax-source'], $values['value']);
             }
-        }
+        } else {
+            if (!in_array('multiple', $extras)) {
+                $result .= '
+            <option></option>';
+            }
 
-        // Gestione del select da query specifica (se il campo "values" è impostato a "query=SQL")
-        elseif (preg_match_all('/^query=(.+?)$/', $values['values'], $matches)) {
-            $result .= '
-        <option></option>';
+            // Gestione del select da query specifica (se il campo "values" è impostato a "query=SQL")
+            if (preg_match_all('/^query=(.+?)$/', $values['values'], $matches)) {
+                $result .= $this->selectQuery($matches[1][0], $values['value']);
+            }
 
-            $result .= $this->selectQuery($matches[1][0], $values['value']);
-        }
+            // Gestione del select dal formato JSON parziale (valori singoli)
+            elseif (preg_match_all('/^list=(.+?)$/', $values['values'], $matches)) {
+                $result .= $this->selectList(json_decode('{'.$matches[1][0].'}', true), $values);
+            }
 
-        // Gestione del select dal formato JSON parziale (valori singoli)
-        elseif (preg_match_all('/^list=(.+?)$/', $values['values'], $matches)) {
-            $result .= '
-        <option></option>';
-
-            $result .= $this->selectList(json_decode('{'.$matches[1][0].'}', true), $values);
-        }
-
-        // Gestione del select dal formato JSON completo, convertito in array
-        elseif (is_array($values['values'])) {
-            $result .= '
-        <option></option>';
-
-            $result .= $this->selectArray($values['values'], $values['value']);
+            // Gestione del select dal formato JSON completo, convertito in array
+            elseif (is_array($values['values'])) {
+                $result .= $this->selectArray($values['values'], $values['value']);
+            }
         }
 
         // Impostazione del placeholder