Bug #82931

File reference on new page gets pid 0 instead of the pid of the record

Added by Michiel Roos over 1 year ago. Updated 3 months ago.

Status:
Under Review
Priority:
Must have
Assignee:
-
Category:
FormEngine aka TCEforms
Target version:
Candidate for patchlevel
Start date:
2017-11-06
Due date:
% Done:

0%

TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Steps to reproduce:
  1. Go to 'list' mode
  2. Create a new page (so it has uid NEW_*)
  3. Attach a resource like an image
  4. Save the record

Expected behavior:
The sys_file_reference entry has a pid of the newly created page

Observed behavior:
The sys_file_reference entry has a pid of 0

Result:
A regular 'mortal' user can never remove the reference from the page object because it has no access to remove stuff from pid 0.

I only observed this behavior in page records. tt_content for example, seems to work fine

History

#1 Updated by Michiel Roos over 1 year ago

I'm working on a fix.

#2 Updated by Gerrit Code Review over 1 year 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/54598

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

#4 Updated by Gerrit Code Review over 1 year ago

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

#5 Updated by Gerrit Code Review over 1 year ago

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

#6 Updated by Gerrit Code Review over 1 year ago

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

#7 Updated by Benni Mack over 1 year ago

  • Category set to FormEngine aka TCEforms
  • Sprint Focus set to On Location Sprint

#8 Updated by Gerrit Code Review over 1 year ago

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

#9 Updated by Gerrit Code Review over 1 year ago

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

#10 Updated by Gerrit Code Review over 1 year ago

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

#11 Updated by Mathias Brodala over 1 year ago

  • Sprint Focus deleted (On Location Sprint)

#12 Updated by Gerrit Code Review over 1 year ago

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

#13 Updated by Gerrit Code Review over 1 year ago

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

#14 Updated by Gerrit Code Review over 1 year ago

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

#15 Updated by Vladimir Falcon 9 months ago

I am having the same problem in TYPO3 7.6.31

#16 Updated by Gerrit Code Review 9 months ago

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

#17 Updated by Gerrit Code Review 7 months ago

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

#18 Updated by Gerrit Code Review 5 months ago

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

#19 Updated by Josef Glatz 5 months ago

A customer encountered the same problem in TYPO3 8.7 where the PID of the sys_language_uid is 0. But it's somehow funny, because the editor doesn't have the list module enabled for his usergroup. I can't find a way to reproduce the behaviour. Only the title of the patch on Gerrit seems to be the same problem.

#20 Updated by Josef Glatz 3 months ago

  • Priority changed from Should have to Must have
  • Target version set to Candidate for patchlevel

Also available in: Atom PDF