-
Notifications
You must be signed in to change notification settings - Fork 526
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error: wants to repair powerpoint #89
Comments
@chuebert Have you got always the problem ? With develop branch ? If yes, could you give us a file (simple) generated with PowerPoint2010 ? |
Similar problem in PowerPoint For Mac 2011 (version 14.0). |
@dami95 could you give us a file (simple) generated with PowerPoint For Mac 2011 (version 14.0) ? |
No problem http://grab.by/I4Ko :) |
no i dont have solve the problem with repair. Do you need the file? |
File that needs repair: |
@chuebert Thanks for the corrupted. But it's a huge file (hard to find the error). Have you got a short presentation (max. 3 slides) with the problem ? Could you send a sample file (with no error) generated by PPT2010, too ? |
generated file, not repaired: repaired with office 2010: |
@chuebert Could you do some Tests ? Number 1 : // Bold
$objWriter->writeAttribute('b', ($element->getFont()->isBold() ? '1' : '0'));
// Italic
$objWriter->writeAttribute('i', ($element->getFont()->isItalic() ? '1' : '0')); Number 2 : // a:prstGeom
$objWriter->startElement('a:prstGeom');
$objWriter->writeAttribute('prst', 'rect');
// a:prstGeom\a:avLst
$objWriter->writeElement('a:avLst', null);
// > a:prstGeom
$objWriter->endElement(); Number 3: } else {
$objWriter->writeElement('a:endParaRPr');
} |
I'm seeing the same problem with Mac PP 2011. For whatever it's worth, the exact same file opens without problem on Mac PP 2008 and Win PP 2007. I'm happy to help test and debug, if needed. |
@rjakes Thanks for your help... We need to find what MacPP2011 doens't like in our PPTX file. So can you test my previous comment ? |
I ran all three tests and got the same results - newer versions of PP complain about the file. |
@rjakes Could you send a sample file (with no error) generated by PPT2010, too ? Thanks :) |
I am having the Require Repair issue as well when opening with Powerpoint 2011 for Mac. I am attaching two small files for comparison. One is the original Powerpoint generated by PHPPresentation and the second is the Powerpoint file after it has been repaired. Thanks. |
@steve30000 Hi, could you test some things : In src/PhpPresentation/Writer/PowerPoint2007/LayoutPack/PackDefault.php, could you replace : Warning ! It's not a Less than symbol (or open angled bracket) , it's a Single left-pointing angle quotation. |
Hi. I did the modification, but now the PowerPoint file needs to be repaired for the PC version. I've attached simple reports files before and after modification. Not sure it solves the isse for Mac, but it clearly creates a new issue for PC. Thanks for your quick response, though. |
Hi. I just wanted to let you know that, indeed, I screwed up when I tested your solution, and used regular < and > signs. With the correct characters, it opens right up on a PC and Mac without issue. |
@chuebert @dami95 @rjakes @steve30000 @adebruyn @mikestp27 PR merged. |
I just tested with develop branch with #89 and #167 and I still have the issue (on Mac Office 2016)... I still need the changes to Slide.php that you suggested on June 17: #89 (comment) |
@mikestp27 Bugfix pushed at the moment. Could you test again, please ? |
Perfect. I close it :) |
I'm running into the same issue with the current 0.6.0 code. Problem occurs with Mac Powerpoint 2011 and 2016. Example output PPT here - https://drive.google.com/open?id=0B8hTI70phHLQcHNMdEN0WXhIcFE |
And with the develop branch ? |
Sorry I meant to say with the dev branch from 2 weeks ago. I just grabbed the latest dev branch and I still see the error. Example output PPT of the same content here - https://drive.google.com/open?id=0B8hTI70phHLQV0g0ak1CXzR1WTg |
Any thoughts on what might be causing the problem? |
Any updates? I'm curious what specifically in the generated output causes the warning. |
@vce-wabbot Could you try with reverting this commit f201ae3 ? Thanks |
Similar issues here using PHPOffice 0.7.0, and Mac PPT 2016. File is very simple here. File needs repairing, and then some content unreadable. |
Hi, when i tried to open presentation in mac it showing "font calibire is not available and replaced by Trebuchet MS" message is showing. But it is working fine in windows. |
Behieve(2).pptx |
@vce-wabbott Have you got always the problem with latest version of the develop branch ? CC: @meriemagg (No Problem, I'm french), @Johnfredadams @santhosh1330 |
Hi, With Mac PPT 2011 I get a PPT crash with no message. |
Hi, I'd like to know if there is already some kind of update on this issue. Regards |
@Gerdon262 Hi, the project is actually opened to pull request for fixing this issue. |
@Progi1984 Could you please update if the issue is fixed? If yes, is it in master branch or develop? |
@kailashgajara The issue is not fixed. We are opened to pull request for fixing this issue. |
In case this helps anyone I just noticed something that helped me solve this issue |
I get an error with powerpoint 2010.
I have the current version and get an error that wants to repair the powerpoint.
How can i fix this?
The text was updated successfully, but these errors were encountered: