Fixes issue #38, making multi-value date fields work

lugensa / scorched

Sunburnt offspring solr client

MIT License

27 stars 19 forks source link

Fixes issue #38, making multi-value date fields work #39

Closed mlissner closed 7 years ago

mlissner commented 7 years ago

This is a pretty simple PR to make adding multi-value date fields possible, including a simple test.

The changes here do two things:

Check if the value of a date field is an iterable. If so, treats it as such.
Adds a test that uses a dynamic field to try to add a multi-value date to the index. It fails before this commit, but works afterwards.

coveralls commented 7 years ago

Coverage increased (+0.02%) to 95.385% when pulling a275260c61dbcae2c4c2e248990a9dd9b2946047 on mlissner:issue-38 into 607573095b82655f62f0bf587ac02132c9404ad3 on lugensa:master.