Skip to content

Commit

Permalink
Execute: Remove excessive arguments (#1268)
Browse files Browse the repository at this point in the history
  • Loading branch information
IvanGoncharov authored and leebyron committed Mar 29, 2018
1 parent 920fb87 commit 4dfb993
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/execution/execute.js
Original file line number Diff line number Diff line change
Expand Up @@ -731,7 +731,7 @@ export function buildResolveInfo(
// The resolve function's optional fourth argument is a collection of
// information about the current execution state.
return {
fieldName: fieldNodes[0].name.value,
fieldName: fieldDef.name,
fieldNodes,
returnType: fieldDef.type,
parentType,
Expand Down Expand Up @@ -1171,7 +1171,6 @@ function completeObjectValue(
exeContext,
returnType,
fieldNodes,
info,
path,
result,
);
Expand All @@ -1187,7 +1186,6 @@ function completeObjectValue(
exeContext,
returnType,
fieldNodes,
info,
path,
result,
);
Expand All @@ -1208,7 +1206,6 @@ function collectAndExecuteSubfields(
exeContext: ExecutionContext,
returnType: GraphQLObjectType,
fieldNodes: $ReadOnlyArray<FieldNode>,
info: GraphQLResolveInfo,
path: ResponsePath,
result: mixed,
): MaybePromise<ObjMap<mixed>> {
Expand Down Expand Up @@ -1260,7 +1257,7 @@ function defaultResolveTypeFn(
context: mixed,
info: GraphQLResolveInfo,
abstractType: GraphQLAbstractType,
): ?GraphQLObjectType | string | Promise<?GraphQLObjectType | string> {
): MaybePromise<?GraphQLObjectType | string> {
// First, look for `__typename`.
if (
value !== null &&
Expand Down

0 comments on commit 4dfb993

Please sign in to comment.