diff --git a/src/main/java/graphql/schema/GraphQLCodeRegistry.java b/src/main/java/graphql/schema/GraphQLCodeRegistry.java index e72fadf080..768d13897e 100644 --- a/src/main/java/graphql/schema/GraphQLCodeRegistry.java +++ b/src/main/java/graphql/schema/GraphQLCodeRegistry.java @@ -49,23 +49,6 @@ public GraphqlFieldVisibility getFieldVisibility() { return fieldVisibility; } - /** - * Returns a data fetcher associated with a field within a container type - * - * @param parentType the container type - * @param fieldDefinition the field definition - * - * @return the DataFetcher associated with this field. All fields have data fetchers - * - * @see #getDataFetcher(GraphQLObjectType, GraphQLFieldDefinition) - * @deprecated This is confusing because {@link GraphQLInterfaceType}s cant have data fetchers. At runtime only a {@link GraphQLObjectType} - * can be used to fetch a field. This method allows the mapping to be made, but it is never useful if an interface is passed in. - */ - @Deprecated(since = "2023-05-13") - public DataFetcher getDataFetcher(GraphQLFieldsContainer parentType, GraphQLFieldDefinition fieldDefinition) { - return getDataFetcherImpl(FieldCoordinates.coordinates(parentType, fieldDefinition), fieldDefinition, dataFetcherMap, systemDataFetcherMap, defaultDataFetcherFactory); - } - /** * Returns a data fetcher associated with a field within an object type * @@ -252,23 +235,6 @@ private Builder markChanged(boolean condition) { return this; } - /** - * Returns a data fetcher associated with a field within a container type - * - * @param parentType the container type - * @param fieldDefinition the field definition - * - * @return the DataFetcher associated with this field. All fields have data fetchers - * - * @see #getDataFetcher(GraphQLObjectType, GraphQLFieldDefinition) - * @deprecated This is confusing because {@link GraphQLInterfaceType}s cant have data fetchers. At runtime only a {@link GraphQLObjectType} - * can be used to fetch a field. This method allows the mapping to be made, but it is never useful if an interface is passed in. - */ - @Deprecated(since = "2023-05-13") - public DataFetcher getDataFetcher(GraphQLFieldsContainer parentType, GraphQLFieldDefinition fieldDefinition) { - return getDataFetcherImpl(FieldCoordinates.coordinates(parentType, fieldDefinition), fieldDefinition, dataFetcherMap, systemDataFetcherMap, defaultDataFetcherFactory); - } - /** * Returns a data fetcher associated with a field within an object type * @@ -357,24 +323,6 @@ public Builder dataFetcher(FieldCoordinates coordinates, DataFetcher dataFetc return dataFetcher(assertNotNull(coordinates), DataFetcherFactories.useDataFetcher(dataFetcher)); } - /** - * Sets the data fetcher for a specific field inside a container type - * - * @param parentType the container type - * @param fieldDefinition the field definition - * @param dataFetcher the data fetcher code for that field - * - * @return this builder - * - * @see #dataFetcher(GraphQLObjectType, GraphQLFieldDefinition, DataFetcher) - * @deprecated This is confusing because {@link GraphQLInterfaceType}s cant have data fetchers. At runtime only a {@link GraphQLObjectType} - * can be used to fetch a field. This method allows the mapping to be made, but it is never useful if an interface is passed in. - */ - @Deprecated(since = "2023-05-13") - public Builder dataFetcher(GraphQLFieldsContainer parentType, GraphQLFieldDefinition fieldDefinition, DataFetcher dataFetcher) { - return dataFetcher(FieldCoordinates.coordinates(parentType.getName(), fieldDefinition.getName()), dataFetcher); - } - /** * Sets the data fetcher for a specific field inside an object type *