From 4aca2d40ff5b2c125e41823e1d3a8a3ba43b5d43 Mon Sep 17 00:00:00 2001
From: Samuel Georges <sam@daftspunk.com>
Date: Mon, 7 Nov 2016 08:43:52 +1100
Subject: [PATCH] Remove "allowClear" and tidy up #2459 Allow clear is not
 necessary, equivalent of allowClear false is "placeholder" and allowClear
 true is "emptyOption".

---
 modules/backend/formwidgets/Relation.php             |  2 --
 .../widgets/form/partials/_field_dropdown.htm        | 12 +++++-------
 modules/system/assets/ui/js/select.js                |  9 +--------
 modules/system/assets/ui/storm-min.js                |  2 ++
 4 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/modules/backend/formwidgets/Relation.php b/modules/backend/formwidgets/Relation.php
index ab60d0bde..17aef5e08 100644
--- a/modules/backend/formwidgets/Relation.php
+++ b/modules/backend/formwidgets/Relation.php
@@ -112,8 +112,6 @@ class Relation extends FormWidgetBase
                 $field->type = 'dropdown';
             }
 
-            $field->placeholder = $this->emptyOption;
-
             // It is safe to assume that if the model and related model are of
             // the exact same class, then it cannot be related to itself
             if ($model->exists && (get_class($model) == get_class($relationModel))) {
diff --git a/modules/backend/widgets/form/partials/_field_dropdown.htm b/modules/backend/widgets/form/partials/_field_dropdown.htm
index e81a8ee55..c51e4eee0 100644
--- a/modules/backend/widgets/form/partials/_field_dropdown.htm
+++ b/modules/backend/widgets/form/partials/_field_dropdown.htm
@@ -1,12 +1,7 @@
 <?php
     $fieldOptions = $field->options();
     $useSearch = $field->getConfig('showSearch', true);
-    $allowClear = $field->getConfig('allowClear', false);
-    // If allowClear is true, define a default placeholder in case there is not one already defined.
-    if ($allowClear == true)
-    {
-        $field->placeholder = $field->getConfig('placeholder', e(trans('backend::lang.form.select_placeholder')));
-    }
+    $emptyOption = $field->getConfig('emptyOption', $field->placeholder);
 ?>
 <!-- Dropdown -->
 <?php if ($this->previewMode): ?>
@@ -15,10 +10,13 @@
     <select
         id="<?= $field->getId() ?>"
         name="<?= $field->getName() ?>"
-        class="form-control custom-select <?= $useSearch ? '' : 'select-no-search' ?> <?= $allowClear ? 'select-allow-clear' : '' ?>"
+        class="form-control custom-select <?= $useSearch ? '' : 'select-no-search' ?>"
         <?= $field->getAttributes() ?>
         <?= $field->placeholder ? 'data-placeholder="'.e(trans($field->placeholder)).'"' : '' ?>
         >
+        <?php if ($emptyOption): ?>
+            <option value=""><?= e(trans($emptyOption)) ?></option>
+        <?php endif ?>
         <?php foreach ($fieldOptions as $value => $option): ?>
             <?php
                 if (!is_array($option)) $option = [$option];
diff --git a/modules/system/assets/ui/js/select.js b/modules/system/assets/ui/js/select.js
index c02d10ec1..d8e648253 100644
--- a/modules/system/assets/ui/js/select.js
+++ b/modules/system/assets/ui/js/select.js
@@ -90,17 +90,10 @@
             }
 
             var placeholder = $element.data('placeholder')
-            if (placeholder)
-            {
+            if (placeholder) {
                 extraOptions.placeholder =  placeholder;
             }
 
-            if ($element.hasClass('select-allow-clear'))
-            {
-                extraOptions.allowClear = true;
-            }
-
-
             $element.select2($.extend({}, selectOptions, extraOptions))
         })
     })
diff --git a/modules/system/assets/ui/storm-min.js b/modules/system/assets/ui/storm-min.js
index 8149d8704..cd423ce3a 100644
--- a/modules/system/assets/ui/storm-min.js
+++ b/modules/system/assets/ui/storm-min.js
@@ -3184,6 +3184,8 @@ if($element.hasClass('select-no-dropdown')){extraOptions.selectOnClose=true
 extraOptions.closeOnSelect=false
 $element.on('select2:closing',function(){$('.select2-dropdown.select-no-dropdown:first .select2-results__option--highlighted').removeClass('select2-results__option--highlighted')
 $('.select2-dropdown.select-no-dropdown:first .select2-results__option:first').addClass('select2-results__option--highlighted')})}}
+var placeholder=$element.data('placeholder')
+if(placeholder){extraOptions.placeholder=placeholder;}
 $element.select2($.extend({},selectOptions,extraOptions))})})
 $(document).on('disable','select.custom-select',function(event,status){if($(this).data('select2')!=null){$(this).select2('enable',!status)}})})(jQuery);+function($){"use strict";var LoadIndicator=function(element,options){this.$el=$(element)
 this.options=options||{}