[Fix] stringify: avoid encoding arrayformat comma when `encodeValue…

…sOnly = true` Solves 3/5 of failing tests in ljharb#410
adnan-creator · Nov 5, 2021 · 867898d · 867898d
1 parent 363d483
commit 867898d
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/lib/stringify.js b/lib/stringify.js
@@ -102,6 +102,14 @@ var stringify = function stringify(
     if (isNonNullishPrimitive(obj) || utils.isBuffer(obj)) {
         if (encoder) {
             var keyValue = encodeValuesOnly ? prefix : encoder(prefix, defaults.encoder, charset, 'key', format);
+            if (generateArrayPrefix === 'comma' && encodeValuesOnly) {
+                var valuesArray = String(obj).split(',');
+                var valuesJoined = "";
+                for (var i = 0; i < valuesArray.length; ++i) {
+                    valuesJoined = valuesJoined.concat((i===0 ? '' : ',') + formatter(encoder(valuesArray[i], defaults.encoder, charset, 'value', format)));
+                }
+                return [formatter(keyValue) + '=' + valuesJoined];
+            }
             return [formatter(keyValue) + '=' + formatter(encoder(obj, defaults.encoder, charset, 'value', format))];
         }
         return [formatter(prefix) + '=' + formatter(String(obj))];