Bug #75881

Improper usage of processed files

Added by Philipp Gampe about 5 years ago. Updated about 1 year ago.

Should have
File Abstraction Layer (FAL)
Target version:
Start date:
Due date:
% Done:


Estimated time:
TYPO3 Version:
PHP Version:
Is Regression:
Sprint Focus:


$file->process(...) returns a ProcessedFile object, even it could not process the image.
The ProcessedFile has a method isProcessed() that needs to be used to check the result, otherwise ProcessedFile returns the properties of the original file.


Updated by Gerrit Code Review about 5 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/47871


Updated by Philipp Gampe almost 5 years ago

  • Status changed from Under Review to New
  • Assignee deleted (Philipp Gampe)
  • Complexity set to medium

Updated by Susanne Moog over 2 years ago

  • Status changed from New to Needs Feedback

After revisiting this topic a few years later, I've come to the conclusion that I either don't get what the issue is anymore or there is no issue. Basically, the naming of "processed" is a bit ambivalent as in this case it simply means "hey, I've seen that file at some point" and not "I've done something with it". From the (current) code, I see no issue. Do you?


Updated by Philipp Gampe over 2 years ago

I did not verify lately, but at least the back in the days, the API returned an unprocessed file object in case of errors. If this is the expected behavior, then this should be documented.

Ambivalence in the API is an own bug category :/

You can as well go ahead and close this one. If nobody else bother since several years now, it must be good enough and time may be better spend elsewhere.


Updated by Susanne Moog about 1 year ago

  • Status changed from Needs Feedback to Closed

Thanks for the feedback, closing here (just as a side note: language itself is pretty ambivalent ;))

Also available in: Atom PDF