[ https://issues.apache.org/jira/browse/TIKA-504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12907391#action_12907391 ]
Staffan Olsson commented on TIKA-504: ------------------------------------- A question on the implementation: Should Tika really have a fallback to string on fields with expected non-string types. For example Object length = directory.getObject(ExifDirectory.TAG_ORIENTATION); if(length instanceof Integer) { metadata.set(Metadata.ORIENTATION, Integer.toString( ((Integer)length).intValue() )); } else { metadata.set(Metadata.ORIENTATION, directory.getString(ExifDirectory.TAG_ORIENTATION)); } Isn't there a risk that this produces type errors for the application that uses Tika? I think it would be better if the value is skipped when it can't be parsed to the expected type. > Support XMP metadata keys for more of the common EXIF tags > ---------------------------------------------------------- > > Key: TIKA-504 > URL: https://issues.apache.org/jira/browse/TIKA-504 > Project: Tika > Issue Type: Improvement > Components: parser > Affects Versions: 0.7 > Reporter: Nick Burch > Assignee: Nick Burch > Priority: Minor > Fix For: 0.8 > > > Following the work on TIKA-442, we now have some XMP inspired image metadata > keys and type properties. This currently only covers 4 core exif tags > There are a couple of other common tags that we should probably handle in the > same way. The plan would be to define more entries in the TIFF metadata area, > using the same definitions as in XMP, and update the Exif parser to > additionally map onto these. (The raw exif tags would continue to be output > too, for compatibility) > The tags I propose to support are: > * exif:ExposureTime > * exif:FNumber > * exif:Flash > * exif:FocalLength > * exif:IsoSpeedRatings > * exif:Manufacturer > * exif:Model > * exif:Software > * exif:Orientation > * exif:XResolution > * exif:YResolution > * exif:ResolutionUnit > Plus one extra date, which we don't really seem to have a suitable existing > key for, but maybe should be made more generic? > * exif:DateTimeOriginal -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.