EGA-archive / beacon2-ri-api

Beacon v2 Reference Implementation (API)
Apache License 2.0
16 stars 38 forks source link

fix response summary #311

Closed gsfk closed 5 months ago

gsfk commented 5 months ago

numTotalResults in the response summary is supposed to show the total number of results found by the query, regardless of pagination (see the spec).

In some cases, the reference implementation will spuriously return pagination values instead of the total number of results. Here is an example:

The fix is to not overwrite numTotalResults with a different value. There were a number of cases and one function parameter that are no longer necessary.