plantbreeding / brapi-Java-TestServer

BrAPI Java Test Server
https://brapi.org
MIT License
6 stars 7 forks source link

Germplasm pagination metadata is incorrect #65

Closed timparsons closed 1 year ago

timparsons commented 1 year ago

When fetching germplasm with specific filters included, the pagination information in the metadata shows the total count as the total number of germplasm in the database, not the total that meet the search criteria.

For example, calling to http://localhost:8080/brapi/v2/germplasm?pageSize=1 returns pagination information of {"pageSize":1,"totalCount":101002,"totalPages":101002,"currentPage":0}. Calling http://localhost:8080/brapi/v2/germplasm?pageSize=1&externalReferenceId=48f1a249-1403-49a6-ad8c-9b8158a37902 should return a smaller totalCount, but does not

timparsons commented 1 year ago

🐑 was user error....

was executing curl, but didn't wrap my URL in double quotes, so the second parameter was being dropped.