search

biocodellc / biocode-fims-commons

Biocode Field Information Management System
3 stars 0 forks source link

Make getColNames() consistent across readers #26

Closed jdeck88 closed 7 years ago

jdeck88 commented 7 years ago

The following commit may break the ExcelReader class and any classes that extend it:

https://github.com/biocodellc/biocode-fims-commons/commit/b3c4f0d75af520d4b4b8b7818ca96dceda65dc97

However, we should be using the getColNames() method here instead of tableGetNextRow() as it is clearer.

Check that the ExcelReader does not break with the use of getColNames() here.