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.

Sign up for GitHub

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

Jump to bottom

Collection fields #58

Closed
mattjohnsonpint opened this issue Feb 25, 2014 · 3 comments
Closed

Collection fields #58

mattjohnsonpint opened this issue Feb 25, 2014 · 3 comments

Comments

@mattjohnsonpint
Copy link
Contributor

It would appear I can only map a collection property as a field if it is declared as IEnumerable<T>. It should really support any type that inherits from IEnumerable<T>, including lists, arrays, IList<T>, etc...
@chriseldredge
Copy link
Owner

There's a TODO here that would need some attention.

There may be other places where changes are required.

@chriseldredge
Copy link
Owner

See also #32.

@mattjohnsonpint
Copy link
Contributor Author

Thanks. If I get some time, I'll see if I can tackle that todo and send a PR.

mattjohnsonpint added a commit to mattjohnsonpint/Lucene.Net.Linq that referenced this issue Apr 30, 2014
@mattjohnsonpint
Improve detection of collection fields
9377c21 
Partially addresses chriseldredge#58
@mattjohnsonpint mattjohnsonpint mentioned this issue Apr 30, 2014
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chriseldredge @mattjohnsonpint