Merge pull request #336 from graphql-java-kickstart/revert-330-trivial-dataloader

Revert "Add the TrivialDataFetcher interface to MethodFieldResolver so that i…"

Author: oliemansm

src/main/kotlin/com/coxautodev/graphql/tools/MethodFieldResolver.kt

@@ -3,7 +3,6 @@ package com.coxautodev.graphql.tools
 import com.coxautodev.graphql.tools.SchemaParserOptions.GenericWrapper
 import com.esotericsoftware.reflectasm.MethodAccess
 import com.fasterxml.jackson.core.type.TypeReference
-import graphql.TrivialDataFetcher
 import graphql.execution.batched.Batched
 import graphql.language.FieldDefinition
 import graphql.language.ListType
@@ -117,17 +116,7 @@ internal class MethodFieldResolver(field: FieldDefinition, search: FieldResolver
         return if (batched) {
             BatchedMethodFieldResolverDataFetcher(getSourceResolver(), this.method, args, options)
         } else {
-            if (args.size == 0
-                    && this.method.parameterCount == 0
-                    && this.method.name.startsWith("get")
-                    && this.search.type is java.lang.Class<*>
-                    && (this.search.type as java.lang.Class<*>).getMethod(this.method.getName()) != null
-            ) {
-                TrivialMethodFieldResolverDataFetcher(getSourceResolver(), this.method, args, options)
-            } else {
-                MethodFieldResolverDataFetcher(getSourceResolver(), this.method, args, options)
-            }
-
+            MethodFieldResolverDataFetcher(getSourceResolver(), this.method, args, options)
         }
     }
 
@@ -232,10 +221,6 @@ open class MethodFieldResolverDataFetcher(private val sourceResolver: SourceReso
     }
 }
 
-open class TrivialMethodFieldResolverDataFetcher(private val sourceResolver: SourceResolver, method: Method, private val args: List<ArgumentPlaceholder>, private val options: SchemaParserOptions) : MethodFieldResolverDataFetcher(sourceResolver, method, args, options), TrivialDataFetcher<Any> {
-
-}
-
 private suspend inline fun MethodAccess.invokeSuspend(target: Any, methodIndex: Int, args: Array<Any?>): Any? {
     return suspendCoroutineUninterceptedOrReturn { continuation ->
         invoke(target, methodIndex, *args + continuation)