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

Sebastian Schlatow edited comment on IMAGING-100 at 11/12/24 2:57 PM:
----------------------------------------------------------------------

It would be great if this could be fixed. I found an old patched 
commons-imaging in our company that is still in use. We have the following 
patch:
PhotoshopApp13Data photoshopApp13Data = null;for (final AbstractSegment s : 
abstractSegments) {final App13Segment segment = (App13Segment) s;final 
PhotoshopApp13Data data = segment.parsePhotoshopSegment(params);if (data != 
null && photoshopApp13Data != null)

{code}

        PhotoshopApp13Data photoshopApp13Data = null;

        for (final AbstractSegment s : abstractSegments) {
            final App13Segment segment = (App13Segment) s;

            final PhotoshopApp13Data data = 
segment.parsePhotoshopSegment(params);
            if (data != null && photoshopApp13Data != null) {
                return null;
            }

            photoshopApp13Data = data;
        }

        if (null == photoshopApp13Data) {
            return null;
        }

{code}

Why is there progress with this ticket?


was (Author: buzz-dee):
It would be great if this could be fixed. I found an old patched 
commons-imaging in our company that is still in use. We have the following 
patch:
        PhotoshopApp13Data photoshopApp13Data = null;for (final AbstractSegment 
s : abstractSegments) {final App13Segment segment = (App13Segment) s;final 
PhotoshopApp13Data data = segment.parsePhotoshopSegment(params);if (data != 
null && photoshopApp13Data != null) {return null;            }            
photoshopApp13Data = data;        }if (null == photoshopApp13Data) {return 
null;        }
Why is there progress with this ticket?

> Need to support images with multiple App13 segments
> ---------------------------------------------------
>
>                 Key: IMAGING-100
>                 URL: https://issues.apache.org/jira/browse/IMAGING-100
>             Project: Commons Imaging
>          Issue Type: Bug
>          Components: Format: JPEG
>    Affects Versions: 0.97
>            Reporter: Luis Arias
>            Assignee: Damjan Jovanovic
>            Priority: Minor
>         Attachments: MultipleApp13Segments.patch, 
> PixarInConcert_150903_art.jpg, employee.jpg
>
>
> The attached image gives rise to the following exception when using sanselan 
> to retrieve exif orientation information as follows:
>       IImageMetadata metadata = Sanselan.getMetadata(bytes);
>       JpegImageMetadata jpegMetadata = (JpegImageMetadata) metadata;
>       TiffField orientation = 
> jpegMetadata.findEXIFValue(TiffConstants.EXIF_TAG_ORIENTATION);
>       orientation.getIntValue();
> |  org.apache.sanselan.ImageReadException: Jpeg contains more than one 
> Photoshop App13 segment.
>       at 
> org.apache.sanselan.formats.jpeg.JpegImageParser.getPhotoshopMetadata(JpegImageParser.java:609)
>       at 
> org.apache.sanselan.formats.jpeg.JpegImageParser.getMetadata(JpegImageParser.java:309)
>       at org.apache.sanselan.Sanselan.getMetadata(Sanselan.java:978)
>       at org.apache.sanselan.Sanselan.getMetadata(Sanselan.java:871)
>       at org.apache.sanselan.Sanselan.getMetadata(Sanselan.java:846)



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

Reply via email to