Bug #39960

levelmedia retrieves deleted records

Added by Marcel Burkhalter almost 3 years ago. Updated about 2 years ago.

Status:Resolved Start date:2012-08-19
Priority:Must have Due date:
Assigned To:Steffen Ritter % Done:

100%

Category:- Spent time: -
Target version:-
TYPO3 Version: Is Regression:
PHP Version: Sprint Focus:
Complexity:

Description

Precondition: A page with one FAL media record (sys_file_reference UID 152) and the following TS:

page.5 = FILES
page.5 {
references.data = levelmedia:-1,slide
renderObj = TEXT
renderObj.data = file:current:publicUrl
}
at the UIDs and the database table sys_file_reference reveals that the IDs 143, 150 and 178 are marked as deleted.

RootlineUtility.php.patch Magnifier (706 Bytes) Witali Rott, 2013-06-05 09:22


Related issues

related to Core - Bug #39955: FILES TS Object is not able to deal with comma separated ... Resolved 2012-08-19

Associated revisions

Revision 03050f9b
Added by Steffen Ritter almost 3 years ago

[BUGFIX] Rootline must not include references to deleted records

The rootline resolval includes deleted records. Deleted
records should never be used in frontend. Add the missing
deleteClause to the request.

Change-Id: I2039d3343479ca080c747b1c39961d03efb34e0f
Fixes: #39960
Releases: 6.0
Reviewed-on: http://review.typo3.org/14302
Reviewed-by: Mario Rimann
Tested-by: Marcel Burkhalter
Reviewed-by: Georg Ringer
Reviewed-by: Stefan Neufeind
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter

History

#1 Updated by Steffen Ritter almost 3 years ago

  • Status changed from New to Accepted

#2 Updated by Steffen Ritter almost 3 years ago

  • Target version set to 6.0 beta 2

#3 Updated by Steffen Ritter almost 3 years ago

  • Subject changed from FILES TS Object with levelmedia retrieves deleted records to levelmedia retrieves deleted records

#4 Updated by Steffen Ritter almost 3 years ago

  • Project changed from File Abstraction Layer to Core
  • Assigned To set to Steffen Ritter
  • Target version deleted (6.0 beta 2)

rootline should add delete clause

#5 Updated by Gerrit Code Review almost 3 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/14302

#6 Updated by Steffen Ritter almost 3 years ago

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

#7 Updated by Witali Rott about 2 years ago

!!!IMPORTANT!!!
Please change deleteClause to enableFields for use it with TCA 'enablecolumns'

Also available in: Atom PDF