Bug 59548 - PowerPointExtractor has PowerPointExtractor(HSLFSlideShowImpl ss) constructor instead of PowerPointExtractor(HSLFSlideShow ss)
Summary: PowerPointExtractor has PowerPointExtractor(HSLFSlideShowImpl ss) constructor...
Status: RESOLVED WONTFIX
Alias: None
Product: POI
Classification: Unclassified
Component: HSLF (show other bugs)
Version: 3.14-FINAL
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on: 62319
Blocks:
  Show dependency tree
 
Reported: 2016-05-13 14:09 UTC by Valeria Chernenko
Modified: 2018-04-20 13:04 UTC (History)
1 user (show)


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Valeria Chernenko 2016-05-13 14:09:29 UTC 
Current PowerPointExtractor constructor can take only HSLFSlideShowImpl but not HSLFSlideShow. 

If one need to extract text and images from the .ppt file, he for she now needs to analyse say InputStream twice: once for text and once for images.

Could be much easier to create an extractor from HSLFSlideShow as ExcelExtractor does — has the constructor with HSSFWorkbook.
Comment 1 Javen O'Neal 2016-05-17 10:00:23 UTC 
I think HSLFSlideShow is a wrapper around HSLFSlideShowImpl.

Does this work?
PowerPointExtractor(new HSLFSlideShow(myHSLFSlideShowImpl))
Comment 2 Andreas Beeker 2018-04-20 13:04:26 UTC 
In the future 4.0.0 release, there's a org.apache.poi.sl.extractor.SlideShowExtractor class which takes a SlideShow in the constructor. So this issue seems to be obsolete.