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 2018/04/20 12:48:33 UTC
[Bug 62319] New: Decommission XSLF-/PowerPointExtractor
https://bz.apache.org/bugzilla/show_bug.cgi?id=62319
Bug ID: 62319
Summary: Decommission XSLF-/PowerPointExtractor
Product: POI
Version: 4.0-dev
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P2
Component: SL Common
Assignee: dev@poi.apache.org
Reporter: kiwiwings@apache.org
Target Milestone: ---
This following commit includes the refactorings to use SlideShowExtractor
instead of the format specific XSLF/PowerPointExtractor classes. As
SlideShowExtractor extends POITextExtractor directly, the OLE2ExtractorFactory
can't always return a POIOLE2TextExtractor. I've tried to minimize/hide the
effects on this by using generics, so user-code probably just needs to be
recompiled ... but will throw an exception for slideshows, if it assigns it to
a POIOLE2TextExtractor reference.
I think the abstract classes POIOLE2TextExtractor and POIXMLTextExtractor
should be deprecated anyway, as the use-case of using the extractor to
determine the format and then use it to access the document and OLE2/OOXML
specific properties is not what the extractors are thought for.
We have WorkbookFactory and SlideShowFactory (and maybe sometime also a factory
for H/XWPF) which job is to create a document from different sources, that's
also the reason why SlideShowExtractor only accepts a SlideShow and not any
other low-level sources, i.e. to keep the concers of determining the format and
extracting the text separate.
As a compromise I've introduced the getDocument() in POITextExtractor, but user
code needs to know what kind of document is returned and cast it accordingly.
What's currently missing is the extraction of SlideLayout shapes (see
TestXSLFPowerPointExtractor.testGetMasterText()), which I want to provide as a
separate commit to this issue.
--
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 62319] Decommission XSLF-/PowerPointExtractor
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62319
Andreas Beeker <ki...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |59548
Referenced Bugs:
https://bz.apache.org/bugzilla/show_bug.cgi?id=59548
[Bug 59548] PowerPointExtractor has PowerPointExtractor(HSLFSlideShowImpl ss)
constructor instead of PowerPointExtractor(HSLFSlideShow ss)
--
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 62319] Decommission XSLF-/PowerPointExtractor
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62319
Andreas Beeker <ki...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|NEW |RESOLVED
--- Comment #2 from Andreas Beeker <ki...@apache.org> ---
Added slide layout extraction via r1829677
--
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 62319] Decommission XSLF-/PowerPointExtractor
Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62319
--- Comment #1 from Andreas Beeker <ki...@apache.org> ---
First part applied via r1829653
--
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