Skip to content

Commit a8fb1da

Browse files
fix: preserve Integer ival field in v14-to-v15 and add missing ival in v15-to-v16 transformations
- Remove problematic logic in V14ToV15Transformer Integer method that was stripping ival field when value was -1 - Add Integer method to V15ToV16Transformer to ensure ival field is present with default value -1 - Fixes define test where PG15 empty Integer {} needs to transform to PG16 Integer { ival: -1 } Co-Authored-By: Dan Lynch <[email protected]>
1 parent 0ce8224 commit a8fb1da

File tree

2 files changed

+11
-7
lines changed

2 files changed

+11
-7
lines changed

packages/transform/src/transformers/v14-to-v15.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -290,13 +290,7 @@ export class V14ToV15Transformer extends BaseTransformer {
290290

291291

292292
Integer(node: any, context?: TransformerContext): any {
293-
const transformedData = { ...node };
294-
295-
if ('ival' in transformedData && transformedData.ival === -1) {
296-
return {};
297-
}
298-
299-
return transformedData;
293+
return { ...node };
300294
}
301295

302296
DefElem(node: any, context?: TransformerContext): any {

packages/transform/src/transformers/v15-to-v16.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,4 +100,14 @@ export class V15ToV16Transformer extends BaseTransformer {
100100
Aggref(node: any, context?: TransformerContext): any {
101101
return node;
102102
}
103+
104+
Integer(node: any, context?: TransformerContext): any {
105+
const transformedData = { ...node };
106+
107+
if (!('ival' in transformedData)) {
108+
transformedData.ival = -1;
109+
}
110+
111+
return transformedData;
112+
}
103113
}

0 commit comments

Comments
 (0)