Bug #33956

Bug #46389: Faulty Workspace Placeholders

FAL is not yet compatible with the tceform placeholder feature

Added by Tolleiv Nietsch over 3 years ago. Updated over 1 year ago.

Status:Closed Start date:2012-02-15
Priority:Must have Due date:
Assigned To:Tolleiv Nietsch % Done:

90%

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

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

related to Core - Task #33745: Integrate File Abstraction Layer Resolved 2011-11-29

History

#1 Updated by Tolleiv Nietsch over 3 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.

#2 Updated by Tolleiv Nietsch over 3 years ago

  • Status changed from Accepted to Needs Feedback
  • Assigned To deleted (Tolleiv Nietsch)
  • % Done changed from 0 to 90

#3 Updated by Oliver Hader over 3 years ago

  • Assigned To set to Oliver Hader

#4 Updated by Oliver Hader over 3 years ago

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

#5 Updated by Tolleiv Nietsch over 3 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?

#6 Updated by Gerrit Code Review over 3 years ago

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

#7 Updated by Gerrit Code Review over 3 years ago

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

#8 Updated by Tolleiv Nietsch over 3 years ago

  • Project changed from File Abstraction Layer to Core
  • Status changed from Needs Feedback to Under Review
  • Assigned To changed from Oliver Hader to Tolleiv Nietsch
  • Priority changed from Should have to Must have
  • Target version deleted (6.0 beta1)

#9 Updated by Oliver Hader over 2 years ago

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

#10 Updated by Ingo Schmitt over 1 year ago

  • Is Regression set to No

Does this problem still exist in 6.2 Beta 4

#11 Updated by Tolleiv Nietsch over 1 year ago

  • Status changed from Under Review to Closed

Kidding?

Also available in: Atom PDF