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

Nulls in arrays #87

Open
lviggiano opened this issue Aug 12, 2014 · 0 comments
Open

Nulls in arrays #87

lviggiano opened this issue Aug 12, 2014 · 0 comments

Comments

@lviggiano
Copy link
Collaborator

From a mail:

Dear Luigi,

while looking into using your very nice library, we discovered a minor issue:

When reading a config value as an ArrayList of a class with a string constructor where this constructor throws an exception for malformed config values, the result is as expected (UnsupportedOperationException indicating that the string cannot be converted) if the first value in the list is malformed - however, if another value is malformed, as long as the first value was valid, this later position in the list is simply set to NULL.

Please find a small patch against your current github master branch attached to remedy this.

All the best, and thanks for an extremely useful tool,
Daniel

https://gist.github.com/lviggiano/4ce8330e157144c0eb91

Looks like a valid bug.

Need to create a couple of unit tests to integrate this patch.

@lviggiano lviggiano added this to the 1.0.6 milestone Aug 12, 2014
lviggiano pushed a commit that referenced this issue Feb 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant