Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(lib-dynamodb): skip function properties in data objects for DynamoDB #5697

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

Merged
merged 7 commits into from
Jan 31, 2024
Merged

fix(lib-dynamodb): skip function properties in data objects for DynamoDB #5697

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c58b50b
fix(lib-dynamodb): check for function types in processKeys
siddsriv Jan 19, 2024
69bf854
test(lib-dynamodb): adding cases for skipping function properties whe…
siddsriv Jan 22, 2024
a9246f3
fix(lib-dynamodb): skip function in processKeys for array case
siddsriv Jan 23, 2024
9f9db09
test(lib-dynamodb): adding additional test cases for function properties
siddsriv Jan 24, 2024
053a4a8
fix(lib-dynamodb): additional checks for function type in processObj
siddsriv Jan 24, 2024
d0e3e44
fix(lib-dynamodb): add a function to skip function properties processing
siddsriv Jan 26, 2024
bc0cabe
test(lib-dynamodb): correct attribute objects for cases
siddsriv Jan 26, 2024
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
Next Next commit
fix(lib-dynamodb): check for function types in processKeys
  • Loading branch information
@siddsriv
siddsriv committed Jan 19, 2024
commit c58b50bd9418aecb6856eaefd8f3798dbd7a6612
13 changes: 12 additions & 1 deletion lib/lib-dynamodb/src/commands/utils.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,18 @@ const processKeysInObj = (obj: any, processFunc: Function, keyNodes: KeyNodeChil
if (Array.isArray(obj)) {
accumulator = [...obj];
} else {
accumulator = { ...obj };
accumulator = {};
for (const [k, v] of Object.entries(obj)) {
if (typeof v !== "function") {
accumulator[k] = v;
}
}
}

for (const [nodeKey, nodes] of Object.entries(keyNodes)) {
if (typeof obj[nodeKey] === "function") {
continue;
}
const processedValue = processObj(obj[nodeKey], processFunc, nodes);
if (processedValue !== undefined) {
accumulator[nodeKey] = processedValue;
Expand All @@ -82,6 +90,9 @@ const processAllKeysInObj = (obj: any, processFunc: Function, keyNodes: KeyNodes
return obj.map((item) => processObj(item, processFunc, keyNodes));
}
return Object.entries(obj).reduce((acc, [key, value]) => {
if (typeof value === "function") {
return acc;
}
const processedValue = processObj(value, processFunc, keyNodes);
if (processedValue !== undefined) {
acc[key] = processedValue;
Expand Down