Skip to content

Option to treat numbers as object keys rather than arrays when transposing. #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Option to treat numbers as object keys rather than arrays when transposing. #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
4fe2fc8
Expose Numbers as array options when transposing
ashubham Oct 20, 2016
6c1de39
Update README.md
ashubham Oct 20, 2016
2981b10
Merge remote-tracking branch 'upstream/main'
bhavya2306 Jan 29, 2025
5cead9f
Fixed isNumber function insided isTreatLikeArray Method
bhavya2306 Jan 29, 2025
41d9071
Merge pull request #1 from bhavya2306/master
bhavya2306 Feb 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ That is, where `obj` is equivalent to:

flat = DataObjectParser.untranspose(structured)

structured = DataObjectParser.transpose(flat)
structured = DataObjectParser.transpose(flat, [options] /*default: {transposeNumAsArray : false}*/)

Where the equivalent results would be:

Expand Down
16 changes: 13 additions & 3 deletions lib/utils/dataobject-parser.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,10 @@ function _isDate(value) {
return value instanceof Date;
}

function DataObjectParser($data){
function DataObjectParser($data, options){
this._data = $data || {};
options = options || {};
this._numbersAreArrays = !!options.transposeNumAsArray;
}

/**
Expand Down Expand Up @@ -69,6 +71,14 @@ DataObjectParser.prototype.set = function($path, $value) {
$obj[$key] = $data;
}
};

var isTreatLikeArray = function(key) {
if(_isNumber(key)) {
return this._numbersAreArrays;
}

return key === "[]";
}

while(pathList.length > 0) {
parentKey = pathList.shift().replace(/["']/g, '');
Expand All @@ -78,8 +88,8 @@ DataObjectParser.prototype.set = function($path, $value) {
throw new Error('Prototype pollution attempt detected');
}

// Number, treat it as an array
if (!isNaN(+parentKey) || parentKey === "[]") {
// Number, check whether to treat as Array
if (isTreatLikeArray(parentKey)) {
if(!_isArray(parent) /* prevent overwritting */ ) {
parent = [];
addObj(grandParent, grandParentKey, parent);
Expand Down