Skip to content
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

Fix EZP-18486 - PHP notice while running cronjobs/indexcontent.php cronjob #1169

Merged
merged 1 commit into from May 18, 2015

Conversation

pedroresende
Copy link
Contributor

https://jira.ez.no/browse/EZP-18486

Description

Occurs when indexing a file.

Notice: ob_end_clean(): failed to delete buffer. No buffer to delete. in /home/dp/dev/ezpublish/kernel/classes/datatypes/ezbinaryfile/plugins/ezpdfparser.php on line 28

Call Stack:
    0.0007     405456   1. {main}() /home/dp/dev/ezpublish/runcronjobs.php:0
    0.0920    6220844   2. eZRunCronjobs::runScript() /home/dp/dev/ezpublish/runcronjobs.php:329
    0.1043    6288448   3. include('/home/dp/dev/ezpublish/cronjobs/indexcontent.php') /home/dp/dev/ezpublish/kernel/classes/ezruncronjobs.php:59
    0.1412    9039224   4. eZSolr->addObject() /home/dp/dev/ezpublish/cronjobs/indexcontent.php:52
    0.2469   14888064   5. eZSolr->addFieldBaseToDoc() /home/dp/dev/ezfind/search/plugins/ezsolr/ezsolr.php:561
    0.2469   14888064   6. ezfSolrDocumentFieldBase->getData() /home/dp/dev/ezfind/search/plugins/ezsolr/ezsolr.php:626
    0.2475   14888444   7. eZContentObjectAttribute->metaData() /home/dp/dev/ezfind/classes/ezfsolrdocumentfieldbase.php:89
    0.2475   14888444   8. eZBinaryFileType->metaData() /home/dp/dev/ezpublish/kernel/classes/ezcontentobjectattribute.php:1130
    0.2496   14954300   9. eZBinaryFile->metaData() /home/dp/dev/ezpublish/kernel/classes/datatypes/ezbinaryfile/ezbinaryfiletype.php:596
    0.2559   14971580  10. eZPDFParser->parseFile() /home/dp/dev/ezpublish/kernel/classes/datatypes/ezbinaryfile/ezbinaryfile.php:206
    0.2560   14971668  11. ob_end_clean() /home/dp/dev/ezpublish/kernel/classes/datatypes/ezbinaryfile/plugins/ezpdfparser.php:28

@pedroresende
Copy link
Contributor Author

ping @joaoinacio, @andrerom, @yannickroger

@@ -25,7 +25,8 @@ function parseFile( $fileName )

// save the buffer contents
$buffer = ob_get_contents();
ob_end_clean();
if ( $buffer )
ob_end_clean();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add curly brackets.

@lolautruche
Copy link
Contributor

Except the curly brackets remark, +1

@pedroresende
Copy link
Contributor Author

@lolautruche added :)

@andrerom
Copy link
Contributor

+1

@yannickroger
Copy link
Contributor

+1

Thanks @pedroresende

yannickroger added a commit that referenced this pull request May 18, 2015
Fix EZP-18486 - PHP notice while running cronjobs/indexcontent.php cronjob
@yannickroger yannickroger merged commit 9c57b7d into ezsystems:master May 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants