Wrong formatting in arrays, fix.

Hi, @therealparmesh I’m using object-to-formdata is very usefully.

Well I have the following example data to convert:

{
  "phones": [
   {
      "number": 9999999,
      "code": 52,
      "contact": "Maria"
   }
 ]
};

I convert the information and process it into php and the result of array is the follow:

[
    'phones' => [
        [
            'number' => '9991944318',
        ],
        [
            'code' => '52',
        ],
        [
            'contact' => 'Maria',
        ],
    ],
]

as you can see each item is within an array.

I solved this problem as follows:

function objectToFormData (obj, fd, pre) {
  fd = fd || new FormData()

  Object.keys(obj).forEach(function (prop) {
    var key = pre ? (pre + '[' + prop + ']') : prop

    if (isObject(obj[prop]) && !isArray(obj[prop]) && !isFile(obj[prop])) {
      objectToFormData(obj[prop], fd, key)
    } else if (isArray(obj[prop])) {

       // HERE.
       // I get the index from the array
      obj[prop].forEach(function (value, index) {

       // HERE.
       // I pass the index to makeArrayKey.
        var arrayKey = makeArrayKey(key, index)
        if (isObject(value) && !isFile(value)) {
          objectToFormData(value, fd, arrayKey)
        } else {
          fd.append(arrayKey, value)
        }
      })
    } else {
      fd.append(key, obj[prop])
    }
  })

// HERE.
// I receive the index as default "null".
function makeArrayKey (key, index = null) {
  if (key.length > 2 && key.lastIndexOf('[]') === key.length - 2) {
    console.log('aqui en solo key');
    return key
  } else {
    // HERE.
    // I valid the existence of the index to indicate that it 
    // is added within the same array.
    return index != null ? key + '['+index+']' : key + '[]';
  }
}

And then the result is the follow It’s just like I expected.

[
    'phones' => [
        [
            'number' => '9991944318',
            'code' => '52',
            'contact' => 'Maria',
        ],
    ],
]

That’s all. Thank you.

[therealparmesh]

Closing this as I added a configuration flag to support this use case in a recent commit.

[benoj]

Will give this a test over the weekend and get back to you 😃