Bug #69833

Processed files don't support not-local storages

Added by Nicole Cordes over 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2015-09-15
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

Use case:

- a ftp storage (https://github.com/adgrafik/fal_ftp) marked as not writable with processing folder 1:/_processed_/FTP
- Text with image content record with an image from ftp storage

Problem:

- image isn't shown (neither in frontend nor backend)

Analysis:

- for frontend processing the driver creates a tempfile as the file on the other server isn't available for processing
- this is as defined in the function description of the DriverInterface
- the processing task tries to process (crop/mask) the image and confess there isn't any change needed
- the task returns NULL and the original file is used
- the processed file is saved with wrong information and the temp file is lost


Related issues

Related to TYPO3 Core - Bug #72554: f:image renders empty src attribute since 6.2.16 Closed 2016-01-06

Associated revisions

Revision e325d476 (diff)
Added by Nicole Cordes over 4 years ago

[BUGFIX] Use correct storage when getting public url of processed file

If the processed file object is marked configured to use the original
file we have to ensure the right storage is used when creating the
publicUrl.

Resolves: #69833
Releases: master, 6.2
Change-Id: Iba0b4c7b8ef8b65cceae86ada0a7c7e85f338fdf
Reviewed-on: http://review.typo3.org/43364
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Sascha Egerer <>
Reviewed-by: Markus Sommer <>
Tested-by: Markus Sommer <>
Reviewed-by: Frans Saris <>
Tested-by: Frans Saris <>

Revision cc04ac11 (diff)
Added by Nicole Cordes over 4 years ago

[BUGFIX] Use correct storage when getting public url of processed file

If the processed file object is marked configured to use the original
file we have to ensure the right storage is used when creating the
publicUrl.

Resolves: #69833
Releases: master, 6.2
Change-Id: Iba0b4c7b8ef8b65cceae86ada0a7c7e85f338fdf
Reviewed-on: http://review.typo3.org/43575
Reviewed-by: Frans Saris <>
Tested-by: Frans Saris <>

History

#1 Updated by Gerrit Code Review over 4 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 http://review.typo3.org/43338

#2 Updated by Gerrit Code Review over 4 years ago

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43364

#3 Updated by Anja Leichsenring over 4 years ago

  • Sprint Focus changed from On Location Sprint to Stabilization Sprint

#4 Updated by Gerrit Code Review over 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43364

#5 Updated by Gerrit Code Review over 4 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/43364

#6 Updated by Gerrit Code Review over 4 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/43364

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

#8 Updated by Nicole Cordes over 4 years ago

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

#9 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF