Powerpoint graphic import isn't working (in the 0.8.5 VM) #879

Closed
augustf opened this Issue Feb 15, 2014 · 13 comments

Comments

Projects
None yet
2 participants
Owner

augustf commented Feb 15, 2014

In the process of trying to migrate all format conversion in ImageMagick, I didn't get past step 1. I found that powerpoint (and presumably other imports) aren't working at all - even in the default 0.8.5 VM with all the right packages. I get the error "Unable to convert practicepowerpoint.ppt, see log for more details." Unfortunately, I was unable to find a log with any error. There is a brief refresh when I select the file, as if the preview code is doing its work, but no preview appears.

This isn't sometime I've tested before (as my Mac doesn't have many of the requisite libs installed) and I'm not sure how long this has been broken. Possible related, when the import fails, I see two file selection widgets.

screen shot 2014-02-15 at 12 37 20 am

Member

mfrederickson commented Feb 15, 2014

I'm seeing something different
missing attribute: file_data
from here
https://github.com/concerto/concerto/blob/master/lib/concerto/content_converter.rb#L68
It's like the media object doesn't have a file_data attribute on it for some reason... I don't know what happened to it or why it would not show up. It does show up in the attributes list when I look at original_media.class. I do see the blob in the database for the .ppt and I can even export it and open it as a ppt.

I'm not seeing the same problem as you, nor the duplicate Browse buttons.

Owner

augustf commented Feb 15, 2014

Just for the sake of having identical environments, are you using the 0.8.5 VM as I was?

Member

mfrederickson commented Feb 15, 2014

No, I was not. I was using a slight stale development version.

Owner

augustf commented Feb 15, 2014

Well I'd definitely take note of the error you're getting as well. But just to make troubleshooting easy, it's nice to start with the same identical platform that has all the packages we think are needed to make the unholy attachable/docsplit/OO craziness work.

Member

mfrederickson commented Feb 15, 2014

Yeah, I can't remember if I ever tested that functionality on the 0.8.5 vm.
But I know I had it working in my test environment. And I don't have any
of the GM removal commits in my stale copy of dev yet....

On Feb 15, 2014, at 5:52 AM, August notifications@github.com wrote:

Well I'd definitely take note of the error you're getting as well. But just
to make troubleshooting easy, it's nice to start with the same identical
platform that has all the packages we think are needed to make the unholy
attachable/docsplit/OO craziness work.

Reply to this email directly or view it on
GitHubhttps://github.com/concerto/concerto/issues/879#issuecomment-35157861
.

Owner

augustf commented Feb 15, 2014

You wouldn't; the GM work is in an experimental branch.

---August

On February 15, 2014 at 12:40:51 PM EST, Marvin Frederickson wrote:Yeah, I can't remember if I ever tested that functionality on the 0.8.5 vm. But I know I had it working in my test environment. And I don't have any of the GM removal commits in my stale copy of dev yet.... On Feb 15, 2014, at 5:52 AM, August notifications@github.com wrote: Well I'd definitely take note of the error you're getting as well. But just to make troubleshooting easy, it's nice to start with the same identical platform that has all the packages we think are needed to make the unholy attachable/docsplit/OO craziness work. -- Reply to this email directly or view it on GitHubhttps://github.com/concerto/concerto/issues/879#issuecomment-35157861 . —Reply to this email directly or view it on GitHub.

Member

mfrederickson commented Feb 16, 2014

@augustf I should have been using .file_contents instead of .file_data all along. This solved the problem in it's entirety for me on my development machine with the current master. I'm able to upload pdf's ppt's doc's, etc.

Member

mfrederickson commented Feb 16, 2014

We're probably going to need another ticket on this to expand it's functionality. Right now it only pulls in the first page of any document. Ideally the original upload media would retain the key of original, once the first page was processed it would be the preview, and then the other pages (media entries) would have a key of processed.

Member

mfrederickson commented Feb 16, 2014

@augustf I'm downloading the .0.8.5 ova right now, but wont get to look at it until Monday sometime.

Member

mfrederickson commented Feb 16, 2014

@august have you tried running docsplit from the command line?

Owner

augustf commented Feb 16, 2014

I just ran this latest code in the VM. It got a little further and generated a broken preview image - after which I got the same "Unable to convert presentations-tips.ppt, see log for more details". Incidentally, we probably shouldn't tell people to look in the log if this process doesn't really generate anything loggable as yet.

screen shot 2014-02-15 at 9 37 13 pm

Owner

augustf commented Feb 16, 2014

So upon running this from the command line, I got:
bundle exec docsplit images --format jpg test.ppt
/usr/share/concerto/vendor/bundle/ruby/1.9.1/gems/docsplit-0.7.2/lib/docsplit/pdf_extractor.rb:91:in `office_executable': No office software found (Docsplit::PdfExtractor::OfficeNotFound)

Well, that's awkward. I'm installing LibreOffice now...

Owner

augustf commented Feb 16, 2014

Ok-so after installing LibreOffice et al - conversion works. But preview still fails with the broken image symbol.

augustf closed this Feb 16, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment