eeverman / eeverman/andhow

Manual exports should have an option to exclude nulls in the collector

Open
#586 0 comments 0 reactions 0 assignees View on GitHub
Exports
Dominant language
Java
Stars
25
Forks
34
PR merge metrics
No merged PRs in 30d

Description

Unsure what the best solution would be here.
* Doing it at the Collector level is another permutation of collectors to think about, but it give the control to the calling code and would mean that the collector doesn't need a `valueIfNull` argument.
* Could add an `exportSkipNulls(...)` method. This would make it easier to apply value transforms b/c there would be no nulls. Exotic Properties could have a case where the value is an empty string and `getValueAsString()` returns null, causing the need for null checks still. That would be a bad Property citizen.

Possibly update the Property and PropertyType docs to make it explicit that if `getValue()` returns null, `getValueAsString()` should return null as well, and visa versa.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.