Skip to content

<T> javadoc fix up #815

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bbakerman merged 4 commits into from
Nov 3, 2017
Merged

<T> javadoc fix up #815

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
49915b2
More aggressive property look up
bbakerman Oct 16, 2017
1fbed55
Merge remote-tracking branch 'upstream/master' into 772-more-aggressi…
bbakerman Nov 2, 2017
49d255b
<T> is for tea
bbakerman Nov 3, 2017
281e03a
Merge remote-tracking branch 'upstream/master' into 772-more-aggressi…
bbakerman Nov 3, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'upstream/master' into 772-more-aggressi… 
…ve-property-fetcher

# Conflicts:
#	src/main/java/graphql/schema/PropertyDataFetcher.java
  • Loading branch information
@bbakerman
bbakerman committed Nov 2, 2017
commit 1fbed559cddb04575c9924e71c3c0d23a441e088
5 changes: 4 additions & 1 deletion src/main/java/graphql/schema/PropertyDataFetcher.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
import graphql.Assert;
import graphql.GraphQLException;
import graphql.PublicApi;
import graphql.schema.idl.TypeRuntimeWiring;

import java.lang.reflect.Field;
import java.lang.reflect.InvocationTargetException;
Expand All @@ -13,17 +12,21 @@
import java.util.Arrays;
import java.util.Map;
import java.util.Optional;
import java.util.function.Function;

import static graphql.Scalars.GraphQLBoolean;

/**
* This is the default data fetcher used in graphql-java. It will examine
* maps and POJO java beans for values that match the desired name, typically the field name
* or it will use a provided function to obtain values.
* maps and POJO java beans for values that match the desired name.
*
* It uses the following strategies
* <ul>
* <li>If the source is null, return null</li>
* <li>If the source is a Map, return map.get(propertyName)</li>
* <li>If a function is provided, it is used</li>
* <li>Find a public JavaBean getter method named `propertyName`</li>
* <li>Find any getter method named `propertyName` and call method.setAccessible(true)</li>
* <li>Find a public field named `propertyName`</li>
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.