tomkralidis commented 5 months ago

Overview

This PR updates the PostgreSQL provider to determine numberMatched prior to applying sort/limit/offset.

Related Issue / discussion

Additional information

Fixes #1594

Dependency policy (RFC2)

[x] I have ensured that this PR meets RFC2 requirements

Updates to public demo

[x] I have ensured that breaking changes to the pygeoapi master demo server have been addressed
- [ ] https://github.com/geopython/demo.pygeoapi.io/blob/master/services/pygeoapi_master/local.config.yml

Contributions and licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution
[x] I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

david-i-berry commented 5 months ago

This appears to fix https://github.com/geopython/pygeoapi/issues/1594.

Screenshot 1 - using master

Screenshot 2 - using https://github.com/tomkralidis/pygeoapi/tree/issue-1594

Note incorrect number of matches in first screenshot and lack of next link. In the second screenshot the number of matches is correct and includes the next link as expected.