A following code in getDeepPropFromObj function in preprocess.js doesn't work:
propPath.replace(/\[([^\]+?])\]/g, '.$1');
The .replace() method returns a replaced string, and it doesn't change a original string (i.e. propPath). The above code drops a replaced string.
The replaced string must be received:
Thank you for your PR. I've implemented it quick-and-dirty style and didn't test it. This was not meant to be part of officially documented functionality just yet.
A following code in
getDeepPropFromObj
function inpreprocess.js
doesn't work:The
.replace()
method returns a replaced string, and it doesn't change a original string (i.e. propPath). The above code drops a replaced string. The replaced string must be received:A current code doesn't catch the properties that is named 2 or more length when
VAR[PROP]
style is specified. For example:Result:
A regexp in current code:
This matches one character, except
]
,+
, and?
. I think that+?
should have been a pattern not literal:And,
[^\]]
is unnecessary because?
works:Result by the above:
A current code doesn't discern a property name via
VAR.PROP
andVAR[PROP]
, and it splits also a property name ofVAR[PROP]
. For example:Result:
This PR discerns those. Result by fixed code:
This fixed code discerns also a name like
VAR[AA].B[CC.DD][E]
correctly.