https://bz.apache.org/bugzilla/show_bug.cgi?id=60504
Greg Woolsey <greg.wool...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
OS| |All
--- Comment #1 from Greg Woolsey <greg.wool...@gmail.com> ---
While we could make it public, that would also open the door to lots of ways to
completely break a workbook. I'd prefer to see a higher level API for
replacing one image with another that would handle these details internally in
a consistent manner, with lots of unit tests.
Patches are always welcome, as waiting for a volunteer to have the time and
inclination to add a specific feature can take some time :) Short of a patch,
a robust discussion of what exactly an API improvement might look like,
including test vectors, is still helpful.
For example, it may be that different document types need to handle image
replacement differently. I haven't dug in to how the different formats do it,
but cell vs. slide vs. paragraph anchor references seem like one area that may
be different. Ideally there should be some consistency between document types
for image replacement, and perhaps some checks as well if there are constraints
or decisions needed when images differ in size, for example.
(In reply to Aurélien from comment #0)
> Currently the method POIXMLDocumentPart.removeRelation(POIXMLDocumentPart
> part) is protected.
> Would it be possible to change the visibility of this method to public?
>
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org
