Skip to content

Commit 289de46

Browse files
committed
we should not have tried to upgrade valueFromAST, it's been deprecated and works with internal values
1 parent 7726dfe commit 289de46

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/utilities/__tests__/valueFromAST-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ describe('valueFromAST', () => {
191191
const testInputObj = new GraphQLInputObjectType({
192192
name: 'TestInput',
193193
fields: {
194-
int: { type: GraphQLInt, externalDefaultValue: 42 },
194+
int: { type: GraphQLInt, defaultValue: 42 },
195195
bool: { type: GraphQLBoolean },
196196
requiredBool: { type: nonNullBool },
197197
},

src/utilities/valueFromAST.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@ export function valueFromAST(
114114
for (const field of Object.values(type.getFields())) {
115115
const fieldNode = fieldNodes.get(field.name);
116116
if (fieldNode == null || isMissingVariable(fieldNode.value, variables)) {
117-
if (field.externalDefaultValue !== undefined) {
118-
coercedObj[field.name] = field.externalDefaultValue.value;
117+
if (field.defaultValue !== undefined) {
118+
coercedObj[field.name] = field.defaultValue;
119119
} else if (isNonNullType(field.type)) {
120120
return; // Invalid: intentionally return no value.
121121
}

0 commit comments

Comments
 (0)