Opened 14 years ago

Closed 14 years ago

#327 closed enhancement (duplicate)

Power Point 2007 Filter

Reported by: Frank J Bruzzaniti Owned by: Olly Betts
Priority: normal Milestone:
Component: Other Version: 1.0.10
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Operating System: All

Description

Here's is a filter I wrote for powerpoint 2007. It will extract all the text from the main slides.

FYI: it works in functionality like: unzip -p "Test sample 20090201.pptx" ppt/slides/slide*.xml

If you wanted the notes and comments you could extend it to add:

unzip -p "Test sample 20090201.pptx" ppt/slides/slide*.xml ppt/notesSlides/notesSlide*.xml ppt/comments/comment*.xml

But you must first check that motes or common files exists as not all pptx *.files contain them (you can get unzip to list the contents of a zip file without uncompressing the zip if you wanted this functionality.

Attachments (1)

ppt2007.patch (2.1 KB ) - added by Frank J Bruzzaniti 14 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 by Olly Betts, 14 years ago

Thanks for the patch, but...

The patch tries to extract the notes and comments unconditionally, but you suggest in your comments that this doesn't work if they don't exist. So I'm wondering if you've attached a different version of the patch to that you intended...

Also (just like your earlier patch in ticket #290) this needs corresponding documentation updates and ideally some sample files to allow testing. While I can update the documentation myself, I'm afraid I have more pressing things to work on currently, and I don't have any files in this format or the ability to create them myself.

Worse, the patch is malformed so the "patch" program won't apply it - I think you must have produced it by hand from a patch including other changes, but that means that you haven't tested the patch as sent, and I can't test any attempt to apply it by hand as I don't have any example files!

If the full patch just includes the changes in #290, it's fine to just put the full patch there instead of having a separate ticket for this addition.

by Frank J Bruzzaniti, 14 years ago

Attachment: ppt2007.patch added

comment:2 by Frank J Bruzzaniti, 14 years ago

Sorry about the mix up. I'll add the changes to #290

comment:3 by Olly Betts, 14 years ago

Resolution: duplicate
Status: newclosed

Closing in favour of #290.

Note: See TracTickets for help on using tickets.