Task #59015
closed
Story #58777: Fixes/Tasks for impexp / distributions
Catch all errors on adding files of sys_files to export
Added by Marc Bastian Heinrichs over 10 years ago.
Updated about 6 years ago.
Category:
Import/Export (T3D)
Description
Adding files from sys_file records is not robust enough.
E.g. if there are sys_file records, but no related files anymore, what
could happen for files in /uploads/tx_foo till #58967 is fixed a RuntimeException
is thrown.
Solution: catch all Exceptions thrown from Resources in export_addSysFile
Files
- Status changed from New to Under Review
I've tested it, now I get this warning in ImpExp module:
Core: Error handler (BE): PHP Warning: fileperms(): stat failed for /var/www/vhosts/.../htdocs/htdocs
in /var/www/t3sources/typo3_src-6.2.2/typo3/sysext/core/Classes/Resource/Driver/LocalDriver.php line 1097
And for every time I open the ImpExp module again (for root node) another warning appears:
In this example I've opened the module six times. I've also opened it on other pages, then it works. But when I switch back to root, a new warning appears additionally.
Ok, thx for testing. Perhaps we should prevent php warnings for fileperms...
Could give some details? You are on Win or Linux? What storages are configured?
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/30318
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/30318
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37566
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Sprint Focus set to On Location Sprint
- Sprint Focus deleted (
On Location Sprint)
- Status changed from Resolved to Closed
Also available in: Atom
PDF