Skip to content
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

Bugfix and additional collection APIs #13

Open
wants to merge 4 commits into
base: next
Choose a base branch
from

Conversation

xzachtli
Copy link

Fix bug when cached type has multiple item indexers
Add additional collection APIs: import, info, properties, count, truncate

xzachtli added 2 commits July 24, 2015 10:34
JObject has 2 indexers Item[string] and Item[int] that error when
caching the type's properties
Allow collection name to be specified by string in ArangoCollection
constructor
Add additional collection APIs: import, info, properties, count,
truncate
Add resource property to HttpCommand URL builder
Allow JsonContent to send multiple documents via "documents" or "array"
serialization methods
@ra0o0f
Copy link
Owner

ra0o0f commented Jul 25, 2015

thanks a lot @xzachtli for the PR

about Prevent CommonUtility from returning class indexers the problem doesn't occurred in pcl version of the client. i change how to find index properties because pcl doesn't have Type.GetDefaultMembers e7355e1

thakns again for the bugfix, i will merge it as soon as i test other parts of your PR

Allow querying named collections by overriding the collection name
Modify In Linq operator to change the most recently referenced
collection to a named collection by string name (supports main from
clause, additional from clauses, update and return, insert and return,
remove and return)
Add API to drop collections
Set default SharedSettings BatchSize to 128 to prevent accidental large
unbounded queries
Settings.Count does not need to be nullable since the database is false
by default
Restructure Client csproj files to make it easier to diff
@xzachtli
Copy link
Author

Thanks for that insight. I do not have any experience with the portable compiler or its limitations.

If/when this PR gets merged, I have another commit in my fork that I can submit if you like that allows overriding or specifying string collection names. This allows casting the result to dynamic or ExpandoObject if the result type does not map to a class.

@aligin
Copy link

aligin commented May 23, 2018

Will it be merged? I need the *Collection().Truncate() function for tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants