wrongly assumes empty string is not a key

[kapouer]

It is perfectly okay to use an empty string for a key.

var jsonPath = require('@irrelon/path');
assert.deepEqual(jsonPath.set({}, 'toto.test', 1), {toto: {test: 1}});
assert.deepEqual(jsonPath.set({}, 'toto.', 2), {toto: {"": 2}});

[kapouer]

I suppose the culprits are:

irrelon-path/dist/Path.js

Line 366 in a189223

if (!internalPath) {

irrelon-path/dist/Path.js

Line 300 in a189223

if (!internalPath) {

[Irrelon]

Irrelon Path started its life as a dot-notation parser for use in a MongoDB-like database before it broke out into a fully fledged project of its own. That legacy means that support for blank string literals in a path was never considered because the database doesn't support storing data without a path or key name.

I can't see anything immediately "wrong" with allowing this, but it is raising my eyebrow since I can't think of any practical use.

[Irrelon]

A PR has been raised against this. Comments and discussion will continue there. #5