From fa52ed3b168dd42762b8e27c7eca4e003d8bc720 Mon Sep 17 00:00:00 2001
From: Krzysztof Kotlarek <kotlarek.krzysztof@gmail.com>
Date: Thu, 5 Dec 2019 08:25:49 +1100
Subject: [PATCH] FIX: bufferedValuesString.split is not a function (#8463)

Fix an error `bufferedValuesString.split is not a function`.
It is happening when bufferedValue is a positive boolean (checkbox value)
---
 .../admin/mixins/setting-component.js.es6     | 21 ++++++++++---------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/app/assets/javascripts/admin/mixins/setting-component.js.es6 b/app/assets/javascripts/admin/mixins/setting-component.js.es6
index 63fa9bcf4b..4d7ea03c34 100644
--- a/app/assets/javascripts/admin/mixins/setting-component.js.es6
+++ b/app/assets/javascripts/admin/mixins/setting-component.js.es6
@@ -26,6 +26,14 @@ const CUSTOM_TYPES = [
 
 const AUTO_REFRESH_ON_SAVE = ["logo", "logo_small", "large_icon"];
 
+function splitPipes(str) {
+  if (typeof str === "string") {
+    return str.split("|").filter(Boolean);
+  } else {
+    return [];
+  }
+}
+
 export default Mixin.create({
   classNameBindings: [":row", ":setting", "overridden", "typeClass"],
   content: alias("setting"),
@@ -101,23 +109,16 @@ export default Mixin.create({
   },
 
   @discourseComputed("buffered.value")
-  bufferedValues(bufferedValuesString) {
-    return (
-      bufferedValuesString && bufferedValuesString.split("|").filter(Boolean)
-    );
-  },
+  bufferedValues: splitPipes,
 
   @discourseComputed("setting.defaultValues")
-  defaultValues(defaultValuesString) {
-    return (
-      defaultValuesString && defaultValuesString.split("|").filter(Boolean)
-    );
-  },
+  defaultValues: splitPipes,
 
   @discourseComputed("defaultValues", "bufferedValues")
   defaultIsAvailable(defaultValues, bufferedValues) {
     return (
       defaultValues &&
+      defaultValues.length > 0 &&
       !defaultValues.every(value => bufferedValues.includes(value))
     );
   },