modifyPath does not work for empty array path

[valerii15298]

In this case nothing happens:
modifyPath([], append(1), []) => []

But logically modifyPath should apply function for empty array and in result there should be [1] and not []
But actual output is an empty array []

[semmel]

What path would an empty array represent?

This is what works:

modifyPath([0], append(1), [[]]);
// -> [[1]]

[JuanGalilea]

Shouldn't an empty path point to the supplied object?
Something like this:

modifyPath([], f, a) === f(a)

But the function has a guard fast returning the final argument in case of an empty path

if ((!_isObject(object) && !_isArray(object)) || path.length === 0) {
  return object;
}

Something like this would satisfy what's being asked:

if (!_isObject(object) && !_isArray(object)) {
  return object;
}
if (path.length === 0) {
  return fn(object);
}

I'm not sure if this goes against any of the principles of the library though. Or if its really that useful to support something like this.

[valerii15298]

@JuanGalilea yep, that is what I meant, thanks.
@CrossEye what do you think?

[JuanGalilea]

Finally brought myself to open a pull request for this.

Checked all other path functions, all of them already supported this behaviour (except for R.assocPath, but i dont know how would that work ).