Project

General

Profile

Actions

Bug #39904

closed

Unresized image not displayed in FE after move in filelist

Added by internezzo ag over 12 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Must have
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2012-08-17
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

When an image is moved in the filelist which is used unresized in a content element (CE), the wrong path is used in the FE.

Steps to reproduce:
  1. Create folder test1 and test2 and upload an image to test1
  2. Create CE with type image and integrate the image
  3. Check FE: image is directly linked to fileadmin/user_upload/test1/image.jpg
  4. Move image to test2 folder in filelist
  5. Clear cache / reindex FAL with scheduler task
  6. Check FE: image is still linked to test1 folder --> image is not displayed
  7. Edit CE: resize image
  8. Check FE: image is displayed correctly from processed folder
  9. Move image back and forth: FE remains intact as long as the image is resized

Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #40033: Relation lost after renaming folderClosedSteffen Ritter2012-08-21

Actions
Related to TYPO3 Core - Bug #40669: FAL files get reindexed in storage null when used with tt_contentClosedSteffen Ritter2012-09-06

Actions
Blocked by TYPO3 Core - Bug #43059: File processing is brokenClosedAndreas Wolf2012-11-17

Actions
Actions #1

Updated by Andreas Wolf about 12 years ago

  • Status changed from New to Needs Feedback

What I don't get from your description: Did you move the file manually or by using FAL methods? Because the former method might sometimes break stuff, though we have taken some measures to ensure consistency of the file index; the latter should definitely not break anything. Maybe this issue is related to #40033 - we'll have to check that.

Actions #2

Updated by Marcel Burkhalter about 12 years ago

I moved the file in the backend module filelist.

Actions #3

Updated by Andreas Wolf about 12 years ago

Ok. Could you check if the index record in sys_file is updated, i.e. if the identifier of the file is changed to .../test2/...? This should normally happen, but maybe something is broken here.

Actions #4

Updated by Andreas Wolf about 12 years ago

Andreas Wolf wrote:

Ok. Could you check if the index record in sys_file is updated, i.e. if the identifier of the file is changed to .../test2/...? This should normally happen, but maybe something is broken here.

Forget what I said here... I found the reason for this bug: Even for files where no processing is applied (because they don't have to be resized etc.), a ProcessedFile object is created and linked to the content record. This record points to the original location of the file, but is not updated when the file is moved. Thus the relation breaks.

A solution could be to simply not create the ProcessedFile altogether when a file is not changed. I don't know why it happens at all or why there is no check if the file has been processed before creating the record...

Actions #5

Updated by Andreas Wolf about 12 years ago

  • Category set to Frontend
  • Status changed from Needs Feedback to Accepted
  • Target version set to 6.0 final
Actions #6

Updated by Andreas Wolf about 12 years ago

  • Project changed from 1401 to TYPO3 Core
  • Category changed from Frontend to Frontend
  • Target version deleted (6.0 final)
Actions #7

Updated by Andreas Wolf about 12 years ago

  • Category changed from Frontend to File Abstraction Layer (FAL)
  • TYPO3 Version set to 6.0
Actions #8

Updated by Gerrit Code Review about 12 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #9

Updated by Steffen Ritter about 12 years ago

  • Assignee set to Steffen Ritter
Actions #10

Updated by Steffen Ritter about 12 years ago

  • Target version set to 6.0.0-beta2
Actions #11

Updated by Gerrit Code Review about 12 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #12

Updated by Gerrit Code Review about 12 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #13

Updated by Gerrit Code Review about 12 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #14

Updated by Steffen Ritter about 12 years ago

  • Status changed from Under Review to On Hold

processed files handling is completely in rewrite

Actions #15

Updated by Gerrit Code Review about 12 years ago

  • Status changed from On Hold to Under Review

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #16

Updated by Gerrit Code Review about 12 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #17

Updated by Gerrit Code Review about 12 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #18

Updated by Steffen Ritter about 12 years ago

  • Target version changed from 6.0.0-beta2 to 6.0.0-RC1
Actions #19

Updated by Gerrit Code Review about 12 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #20

Updated by Gerrit Code Review about 12 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #21

Updated by Gerrit Code Review about 12 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #22

Updated by Gerrit Code Review about 12 years ago

Patch set 11 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #23

Updated by Gerrit Code Review about 12 years ago

Patch set 12 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #24

Updated by Gerrit Code Review about 12 years ago

Patch set 13 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #25

Updated by Gerrit Code Review about 12 years ago

Patch set 21 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14310

Actions #26

Updated by Anonymous about 12 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF