You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2015/02/09 15:40:42 UTC

[Bug 57552] New: XMLSlideShow.addPicture() finds the incorrect image

https://issues.apache.org/bugzilla/show_bug.cgi?id=57552

            Bug ID: 57552
           Summary: XMLSlideShow.addPicture() finds the incorrect image
           Product: POI
           Version: unspecified
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XSLF
          Assignee: dev@poi.apache.org
          Reporter: Mark.Olesen@gmx.net

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org

[Bug 57552] XMLSlideShow.addPicture() finds the incorrect image

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=57552

--- Comment #2 from Nick Burch <ap...@gagravarr.org> ---
Might be worth posting to the dev list - I seem to recall that a few months ago
someone was talking about fixing up some problem with opc relations that was
going to need some moderately major internal changes to do things cleanly. You
might find this helps with that, or you can otherwise assist with the refactor
:)

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org

[Bug 57552] XMLSlideShow.addPicture() finds the incorrect image

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=57552

--- Comment #1 from mark.o <Ma...@gmx.net> ---
Created attachment 32443
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=32443&action=edit
demonstration of possible fix for posted bug

The patch has been implemented as a static 'fix-up' to allow easy testing
without touching the original sources.

However, I think that it would make more sense to have the
OPCPackage.getPartsByName() with a comparator, so that the indices are properly
sorted to start with. Then there would only be the need to deal 0-based vs.
1-based.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org