-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Singleton Support #15
base: master
Are you sure you want to change the base?
Conversation
Implement QuerySingle methods and classes to support singleton operations.
Fixes #14 |
Thanks for looking into this. Copying all of |
I have trimmed out all the functions that were not used by singletons (filter, top, skip, etc). I see that I could trim a bit more from the get_options and new_query functions where they relate to those operators to reduce the size of this feature, but I think the rest of the class is being used... With regard to the Entity subclass, what would that look like? With reference to the TripPin service, we would construct the Person entity, as well as a PersonSingle sublcass? How would we execute the query? Would the query return a Person entity or a PersonSingle entity? Thanks! |
I had a look at Singletons too, and ended up rewriting some of the EntitySet handling. Check it out at 9bfff03 (new branch) I tested this with the TripPin URL you mentioned. |
That looks promising. How would you go about using this model from the Service class? Something like |
Sorry, forgot to mention the usage changes :)
Service = ODataService('http://services.odata.org/TripPinRESTierService/', reflect_entities=True)
Me = Service.entities['Me']
me = Service.query(Me).first() |
I see, so singletons are effectively just an array of one? |
I merged my singleton branch to master. |
This pull request can be closed. @tuomur implemented singleton support and merged it to master. |
Implement QuerySingle methods and classes to support singleton
operations.