Project

General

Profile

Actions

Bug #33956

closed

Bug #46389: Faulty Workspace Placeholders

FAL is not yet compatible with the tceform placeholder feature

Added by Tolleiv Nietsch about 12 years ago. Updated about 10 years ago.

Status:
Closed
Priority:
Must have
Category:
-
Target version:
-
Start date:
2012-02-15
Due date:
% Done:

90%

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

Description

The feature merged in https://review.typo3.org/#change,7823 #33235 causes errors in the backend module when used with the new file stuff....


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #33745: Integrate File Abstraction LayerClosed2011-11-29

Actions
Actions #1

Updated by Tolleiv Nietsch about 12 years ago

Fixed in commit:f04e09f613219a5d526ef3d08dde1673f0f42702

Would be great if someone could confirm that the "__row|__foreign|name" syntax for the placeholder is really final - in this case I could move this to the Core review directly.

Actions #2

Updated by Tolleiv Nietsch about 12 years ago

  • Status changed from Accepted to Needs Feedback
  • Assignee deleted (Tolleiv Nietsch)
  • % Done changed from 0 to 90
Actions #3

Updated by Oliver Hader about 12 years ago

  • Assignee set to Oliver Hader
Actions #4

Updated by Oliver Hader about 12 years ago

Seems like documentation is missing here as well.
extract from http://forge.typo3.org/issues/33235

Actions #5

Updated by Tolleiv Nietsch about 12 years ago

Hmmm wait - Tobi's patch was fine and all features introduced are explained there. But I found that within the FAL branch the placeholders utilized another nesting level ("__row|__foreign|name") which wasn't available from his patchset but needed by the FAL fields. That's why I'm asking whether this should pushed to the Core (as additional RFC) or not?

Actions #6

Updated by Gerrit Code Review about 12 years ago

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

Actions #7

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/9203

Actions #8

Updated by Tolleiv Nietsch about 12 years ago

  • Project changed from 1401 to TYPO3 Core
  • Status changed from Needs Feedback to Under Review
  • Assignee changed from Oliver Hader to Tolleiv Nietsch
  • Priority changed from Should have to Must have
  • Target version deleted (6.0 beta1)
Actions #9

Updated by Oliver Hader about 11 years ago

  • Parent task set to #46389
  • TYPO3 Version set to 6.0
Actions #10

Updated by Ingo Schmitt about 10 years ago

  • Is Regression set to No

Does this problem still exist in 6.2 Beta 4

Actions #11

Updated by Tolleiv Nietsch about 10 years ago

  • Status changed from Under Review to Closed

Kidding?

Actions

Also available in: Atom PDF