[jira] [Commented] (TIKA-4850) Mark audio cover art as THUMBNAIL embedded resources

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TIKA-4850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18109388#comment-18109388 ] 

ASF GitHub Bot commented on TIKA-4850:
--------------------------------------

Copilot commented on code in PR #3090:
URL: https://github.com/apache/tika/pull/3090#discussion_r3886190271


##########
tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-audiovideo-module/src/main/java/org/apache/tika/parser/ogg/OggAudioParser.java:
##########
@@ -24,6 +24,7 @@
 import java.text.DecimalFormat;
 import java.text.NumberFormat;
 import java.util.Arrays;
+import java.util.ArrayList;
 import java.util.Base64;
 import java.util.List;
 import java.util.Locale;

Review Comment:
   The java.util imports are not in the same alphabetical order used elsewhere in this module (e.g., OpusParser), which can trigger checkstyle/import-order failures.





> Mark audio cover art as THUMBNAIL embedded resources
> ----------------------------------------------------
>
>                 Key: TIKA-4850
>                 URL: https://issues.apache.org/jira/browse/TIKA-4850
>             Project: Tika
>          Issue Type: New Feature
>            Reporter: Dominik Schmidt
>            Priority: Major
>
> The audio parsers (Mp3Parser, MP4Parser/TikaUserDataBox, OggAudioParser, FlacParser) emit embedded pictures with tk:embedded-resource-type=INLINE. For a client that wants "the representative image of this file" that is inconsistent with the other container parsers: ODF, OOXML, iWork, GeoGebra and RawTiffParser all mark that image THUMBNAIL, so a client can pick the first THUMBNAIL without knowing the format. For audio it currently needs a special case (INLINE image, and for ID3/FLAC preferably the one whose dc:description is "Cover (front)").
> INLINE also does not describe what cover art is: it is not a picture placed in the document's text flow, it is the picture that stands for the file.
> Proposal:
> - ID3v2 APIC and FLAC/Ogg PICTURE blocks: emit picture type 3 "Cover (front)" as THUMBNAIL. If a tag has no front cover, emit the first picture as THUMBNAIL. All other pictures stay INLINE (or become ATTACHMENT).
> - MP4 covr: the atom carries no picture type; emit the first cover as THUMBNAIL, further ones INLINE.
> - Keep dc:description with the picture type name as it is.
> This changes the resource type of one embedded document per audio file; clients filtering on INLINE to find cover art would need to accept THUMBNAIL as well. Worth a CHANGES note.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)