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 2009/01/30 15:04:16 UTC
DO NOT REPLY [Bug 46635] New: Request: Add a method to remove
slides.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46635
Summary: Request: Add a method to remove slides.
Product: POI
Version: unspecified
Platform: PC
OS/Version: Linux
Status: NEW
Severity: enhancement
Priority: P2
Component: HSLF
AssignedTo: dev@poi.apache.org
ReportedBy: erwin94@gmx.net
Created an attachment (id=23202)
--> (https://issues.apache.org/bugzilla/attachment.cgi?id=23202)
sample implementation
Hi,
i want to propose adding a method to remove a slide from a slideshow.
My usecase:
I have a template ppt created in Powerpoint, that is filled with values through
POI. If there is less data than expected I want to remove superflous slides.
The attached patch works for me, but it could probably share code with
reorderSlides and is missing better documentation and tests.
have a nice day, David.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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
DO NOT REPLY [Bug 46635] Request: Add a method to remove slides.
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46635
Yegor Kozlov <ye...@dinom.ru> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #2 from Yegor Kozlov <ye...@dinom.ru> 2009-02-01 07:11:27 PST ---
Added in r739775 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=739775 )Please note, that my implementation uses 0-based arguments, not 1-based as
suggested in your patch. I would rather change SlideShow.reorderSlide to accept
0-based arguments than use 1-based throughout the code.
Yegor
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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
DO NOT REPLY [Bug 46635] Request: Add a method to remove slides.
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46635
--- Comment #1 from Yegor Kozlov <ye...@dinom.ru> 2009-01-30 06:22:10 PST ---
It's rather "hiding", not a true deleting because the slide data remains, you
only remove the reference to it from the Document.
Anyway, thanks for the patch. I will review and apply later.
Yegor
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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