Repository navigation
Method .getValue() returns only 100 items from *CollectionResponse #1498
Description
Expected behavior
Method .getValue() will return all items from *CollectionResponse
Actual behavior
Method .getValue() returns always first 100 items from *CollectionResponse
Steps to reproduce the behavior
//Fetching the members of a group who are of the type User
//graph.microsoft.com/v1.0/groups/{group-id}/members/microsoft.graph.user
UserCollectionResponse usersInGroup = graphClient
.groups()
.byGroupId("group-id")
.members()
.graphUser()
.get();
List<User> users = usersInGroup.getValue();
System.out.println(users.size());While result from iterating it with Page iterator returns different size.
Lib:
<dependency>
<groupId>com.microsoft.graph</groupId>
<artifactId>microsoft-graph</artifactId>
<version>6.1.0</version>
<exclusions>
<exclusion>
<groupId>com.microsoft.graph</groupId>
<artifactId>microsoft-graph-core</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>com.microsoft.graph</groupId>
<artifactId>microsoft-graph-core</artifactId>
<version>3.1.1</version>
</dependency>
Activity
baywet commentedon Feb 12, 2024
Hi michalchotvac (@MichalChotvac)
This was already addressed by #1485 which was released on last Friday with version 3.1.1 (for core).
Can you make sure your dependencies are up to date please and re-open the issue if you're still facing difficulties?
Thanks!
baywet commentedon Feb 12, 2024
Sorry, I just saw you had the proper dependencies versions.
This is expected behaviour, we're still in the process of rolling out the documentation for the v6 GA, but for future reference the updated snippet will be available here
And in the meantime you can find a snippet here
MichalChotvac commentedon Feb 13, 2024
Hello Vincent Biret (@baywet) so it is rather better to use always Iterator then getValues() to get all values and not only first 100?
baywet commentedon Feb 13, 2024
If your application needs more than the first 100 results, yes. That's how the SDK was designed.
MichalChotvac commentedon Feb 13, 2024
Thanks for clarification Vincent Biret (@baywet)