Fixed this to fix a bug in SPARROW where the last value written into a DBF file would not map with Tom's multi-dbf-datastore.  Looking at this, I believe this was always an issue but was never noticed.
* Added a source code attachment for the build.
Method now retains its original name - a rename will be done in a major dot release.
* rm source attachment pom plugin
* Improved duplicate ID checking (check in bulk at the end rather than for each value and log a warning)
* Improved comments
* Added tests using a sample dbf file
Merge pull request #17 from USGS-CIDA/FieldIndexedDbaseFileReader_bug_fix_for_skipping_last_value_in_dbf

Fixed this to fix a bug in SPARROW where the last value written into a D...