Task #59015
closedStory #58777: Fixes/Tasks for impexp / distributions
Catch all errors on adding files of sys_files to export
100%
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
Updated by Gerrit Code Review over 10 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30318
Updated by Armin Vieweg over 10 years ago
- File 2014-05-22_1124.png 2014-05-22_1124.png added
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.
Updated by Marc Bastian Heinrichs over 10 years ago
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?
Updated by Gerrit Code Review over 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30318
Updated by Gerrit Code Review almost 10 years ago
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
Updated by Gerrit Code Review over 9 years ago
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
Updated by Gerrit Code Review over 9 years ago
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
Updated by Marc Bastian Heinrichs over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 3b86ba556219921a5700217b3c25dedde8d6b0db.
Updated by Anja Leichsenring almost 9 years ago
- Sprint Focus set to On Location Sprint
Updated by Mathias Schreiber over 8 years ago
- Sprint Focus deleted (
On Location Sprint)