From c46842f165354993899aa8bc7d49801af4be04de Mon Sep 17 00:00:00 2001
From: Patrick Martel <patrick@deuxhuithuit.co>
Date: Thu, 12 Mar 2020 11:20:15 -0400
Subject: [PATCH] Fix required setting and force user to pick value when
 required

---
 assets/publish.remote_selectbox.js |  7 +------
 fields/field.remote_selectbox.php  | 15 +++++++++++++++
 2 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/assets/publish.remote_selectbox.js b/assets/publish.remote_selectbox.js
index 4d42219..5f83b7f 100644
--- a/assets/publish.remote_selectbox.js
+++ b/assets/publish.remote_selectbox.js
@@ -15,11 +15,6 @@
 			if (!!data && !!data.length) {
 				var options = [];
 				var selectedValue = t.attr('data-value');
-				var required = ~~t.attr('data-required');
-				
-				if (required) {
-					t.empty();
-				}
 				
 				$.each(data, function (index, d) {
 					var o = $('<option />')
@@ -43,4 +38,4 @@
 	
 	$(init);
 	
-})(jQuery);
\ No newline at end of file
+})(jQuery);
diff --git a/fields/field.remote_selectbox.php b/fields/field.remote_selectbox.php
index e9863fd..30a71e6 100644
--- a/fields/field.remote_selectbox.php
+++ b/fields/field.remote_selectbox.php
@@ -222,6 +222,21 @@ public function displayPublishPanel(XMLElement &$wrapper, $data = null, $flagWit
 			if($flagWithError != null) $wrapper->appendChild(Widget::Error($label, $flagWithError));
 			else $wrapper->appendChild($label);
 		}
+		
+		public function checkPostFieldData($data, &$message, $entry_id = null)
+		{
+			$message = null;
+
+			$has_no_value = is_array($data) ? empty($data['value']) : strlen(trim($data)) == 0;
+
+			if ($this->get('required') === 'yes' && $has_no_value) {
+				$message = __('‘%s’ is a required field.', array($this->get('label')));
+
+				return self::__MISSING_FIELDS__;
+			}
+
+			return self::__OK__;
+		}
 
 		public function processRawFieldData($data, &$status, &$message=null, $simulate=false, $entry_id=NULL){
 			$status = self::__OK__;