Skip to content
This repository

handle ConResCtxt to allow moreConnection query #68

Closed
wants to merge 27 commits into from

2 participants

benib Fabian Vogler
benib
benib commented

This is another approach to #56.
Maybe both approaches have their right to exist.
The query API uri has to be changed if we want to query for more connections as @dol has discoverd.

The reference gets returned in the /connections result and can then be used in a request like:
/v1/moreConnections?reference=ca.012807207.1336656571%231&forwardCount=3
or
/v1/moreConnections?reference=ca.012807207.1336656571%231&backwardCount=6
where only one of forwardCount and backwardCount shall be set and a max of 6 is allowed.
This returnes an array of connections and a new reference to be used for subsequent moreConnection requests.
The fields parameter can be set to the same as with the /v1/connections query to limit the result.

With this approach the query for more connections is only done on request by the API consumer.

i'll do the documentation if you consider merging.

benib benib closed this
benib benib reopened this
Fabian Vogler
Collaborator
fabian commented

Hey I've just added my own implementation of this based on your ideas, see 51ceec5. I experienced problems with xmlfahrplan.sbb.ch as well so I switched to xmlfahrplan.sbb.ch.

Next steps could be to higher the limit but I'd first like to get some feedback on the changes.

Fabian Vogler fabian closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 27 unique commits by 2 authors.

May 10, 2012
benib benib handle ConResCtxt to allow moreConnection query 33ad32d
May 30, 2012
benib benib small fix af63e9a
benib benib handle ConResCtxt to allow moreConnection query a624439
benib benib small fix bf1926e
benib benib Merge branch 'feature-next-prev-connections' of github.com:benib/Tran…
…sport into feature-next-prev-connections
77c6ebe
Jun 03, 2012
benib benib simplify result array building af2a254
Jun 18, 2012
Fabian Vogler fabian Updated Installation 11e0106
Jun 19, 2012
benib benib handle ConResCtxt to allow moreConnection query 23ffe63
benib benib small fix c7d97c7
benib benib handle ConResCtxt to allow moreConnection query e9c81c6
benib benib simplify result array building 0e22c84
benib benib Merge branch 'feature-next-prev-connections' of github.com:benib/Tran…
…sport into feature-next-prev-connections
87da563
Jun 20, 2012
Fabian Vogler fabian Moved examples f6b0b7e
Fabian Vogler fabian Updated Bootstrap 06fbce3
Fabian Vogler fabian Link to documentation b80e2ab
Fabian Vogler fabian Added connections search example 425217b
Fabian Vogler fabian Fixed URL to connections search 495e3a1
Fabian Vogler fabian Check for active search 2074206
Jun 24, 2012
benib benib handle ConResCtxt to allow moreConnection query 94b1684
benib benib small fix ebefd07
benib benib simplify result array building 631bec5
benib benib Merge branch 'feature-next-prev-connections' of github.com:benib/Tran…
…sport into feature-next-prev-connections
aee3773
benib benib cleanup eed721d
benib benib handle ConResCtxt to allow moreConnection query 726b45b
benib benib small fix a243e58
benib benib simplify result array building e9aac9d
benib benib Merge branch 'feature-next-prev-connections' of github.com:benib/Tran…
…sport into feature-next-prev-connections
28a9ee0
Something went wrong with that request. Please try again.