vladzadvorny / http-build-query Goto Github PK

 let params = {
            string_value: 'test_value',
            null_value: null,
            empty_string: '',
            number_string: 196645,
            zero_value: 0,
            params: {
                string_value: 'test_value',
                null_value: null,
                empty_string: '',
                number_string: 196645,
                zero_value: 0,
            }
        };

var httpBuildQuery = function(queryData, numericPrefix, argSeparator, tempKey) {
  numericPrefix = numericPrefix || null;
  argSeparator = argSeparator || '&';
  tempKey = tempKey || null;
// fix 1 start  
 if(queryData === null){
     return '';
 }  
// fix 1 end
  var query = Object.keys(queryData).map(k => {
    var res;
    var key = k;

    if (tempKey) {
      key = tempKey + '[' + key + ']';
    }

    if (typeof queryData[k] === 'object') {
      res = httpBuildQuery(queryData[k], null, argSeparator, key);
    } else {
      if (numericPrefix) {
        key = isNumeric(key) ? numericPrefix + Number(key) : key;
      }
     // fix 2 not push empty string
       if(queryData[k]!==''){
               var val = queryData[k];
                if (val === true) {
                    val = '1';
                } else if (val === false) {
                    val = '0';
                }
            res = esc(key) + '=' + esc(val);
       }
    }

    return res;
  });

  return query.join(argSeparator).replace(/[!'()*]/g, '');
};

``
Также я считаю если пушить пустое значение - это также ошибка. fix2

Psy Shell v0.9.3 (PHP 7.2.4 — cli) by Justin Hileman
>>> http_build_query(['key1' => '', 'key2' => '']);
=> "key1=&key2="
>>>