From 39b92286e21ce144b81ba7b6335169de05b94949 Mon Sep 17 00:00:00 2001 From: Brad Baker Date: Sun, 10 Mar 2024 17:54:03 +1100 Subject: [PATCH] Removed deprecated methods in GraphQLCodeRegistry --- .../graphql/schema/GraphQLCodeRegistry.java | 52 ------------------- 1 file changed, 52 deletions(-) diff --git a/src/main/java/graphql/schema/GraphQLCodeRegistry.java b/src/main/java/graphql/schema/GraphQLCodeRegistry.java index d1b0f94d4c..93b5c6306f 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 *