exiftool icon indicating copy to clipboard operation
exiftool copied to clipboard

Different default value for format for setting metadata

Open vladimirr9 opened this issue 3 years ago • 2 comments

When we look at the two methods for setting image metadata public void setImageMeta(File image, Map<? extends Tag, String> tags) and public void setImageMeta(File image, ExifToolOptions options, Map<? extends Tag, String> tags) throws IOException The first one assumes the format to be numeric, while with the second the default builder value is human_readable, not sure if this is intended, but it took me quite some time debugging and I thought I'd bring it to your attention.

vladimirr9 avatar Mar 21 '22 15:03 vladimirr9

Thanks for the report @vladimirr9 !

not sure if this is intended

No, it's not, and you're 100% right, it's quite misleading. I fear that it would be hard to change it now as it would be a breaking change.

Do you have any suggestion? What could have help you that could help the next one? Maybe updating the README explaining the difference?

mjeanroy avatar Apr 07 '22 20:04 mjeanroy

If it can't outright be fixed, I think the best way to go about it would perhaps be updating the documentation comments or perhaps an example in the readme highlighting the difference.

vladimirr9 avatar Apr 16 '22 20:04 vladimirr9