Bug #75756

Fatal error occurs when sys_file_processedfile table is cleared but processed file is still present

Added by Frans Saris over 3 years ago. Updated 12 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2016-04-18
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
hard
Is Regression:
Yes
Sprint Focus:

Description

When the record from sys_file_processedfile is deleted but the processed file itself still exitst you run into this fatal error:

#1: PHP Catchable Fatal Error: Argument 1 passed to TYPO3\CMS\Core\Resource\ResourceStorage::getFileForLocalProcessing() must implement interface TYPO3\CMS\Core\Resource\FileInterface, null given, called in typo3/cms/typo3/sysext/core/Classes/Resource/Processing/LocalImageProcessor.php on line 118 and defined in typo3/cms/typo3/sysext/core/Classes/Resource/ResourceStorage.php line 1337 

TYPO3\CMS\Core\Error\Exception thrown in file
typo3/cms/typo3/sysext/core/Classes/Error/ErrorHandler.php in line 111.

This was introduced with https://review.typo3.org/#/c/46905


Related issues

Related to TYPO3 Core - Task #67126: Make sure ProcessedFile and LocalImageProcessor use correct FAL API Closed 2015-05-25

Associated revisions

Revision 036c6ba1 (diff)
Added by Arno Schoon over 3 years ago

[BUGFIX] Create processed file object if file exists but reference doesn't

Create the target file object using the identifier of an already existing
processed file to prevent regenerating

Resolves: #75756
Releases: master, 7.6
Change-Id: I74fd770baf6690674c044288d6a61ec250b590d8
Reviewed-on: https://review.typo3.org/47783
Reviewed-by: Frans Saris <>
Tested-by: Frans Saris <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

Revision a286b433 (diff)
Added by Arno Schoon over 3 years ago

[BUGFIX] Create processed file object if file exists but reference doesn't

Create the target file object using the identifier of an already existing
processed file to prevent regenerating

Resolves: #75756
Releases: master, 7.6
Change-Id: I74fd770baf6690674c044288d6a61ec250b590d8
Reviewed-on: https://review.typo3.org/47795
Reviewed-by: Frans Saris <>
Tested-by: Frans Saris <>

History

#1 Updated by Frans Saris over 3 years ago

  • Description updated (diff)

#2 Updated by Frans Saris over 3 years ago

  • Description updated (diff)

#3 Updated by Matthias Eberlein over 3 years ago

Hi, i get nearly the same, but the error shows up together with Extension "Media & Vidi" in Terminal ./typo3/cli_dispatch.phpsh extbase thumbnail:generate and the same error in Be Modul Media

Oops, an error occurred: Argument 1 passed to TYPO3\CMS\Core\Resource\ResourceStorage::getFileForLocalProcessing() must implement interface TYPO3\CMS\Core\Resource\FileInterface, null given, called in /var/www/virtual/domingo/sources/typo3_src-7.6.5/typo3/sysext/core/Classes/Resource/Processing/LocalImageProcessor.php on line 118

When i go back to TYPO3 Core 7.6.4 all runs fine.

Regards
Matthias

#4 Updated by Frans Saris over 3 years ago

  • Status changed from New to In Progress

Arno Schoon is working on a fix

#5 Updated by Gerrit Code Review over 3 years ago

  • Status changed from In Progress 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/47783

#6 Updated by Gerrit Code Review over 3 years ago

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/47787

#7 Updated by Gerrit Code Review over 3 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/47783

#8 Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47795

#9 Updated by Arno Schoon over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#10 Updated by Benni Mack 12 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF