Bug #33956

Bug #46389: Faulty Workspace Placeholders

FAL is not yet compatible with the tceform placeholder feature

Added by Tolleiv Nietsch over 8 years ago. Updated almost 7 years ago.

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

90%

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

Related to TYPO3 Core - Task #33745: Integrate File Abstraction Layer Closed 2011-11-29

History

#1 Updated by Tolleiv Nietsch over 8 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 8 years ago

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

#3 Updated by Oliver Hader over 8 years ago

  • Assignee set to Oliver Hader

#4 Updated by Oliver Hader over 8 years ago

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

#5 Updated by Tolleiv Nietsch over 8 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 8 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 8 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 8 years ago

  • Project changed from File Abstraction Layer 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)

#9 Updated by Oliver Hader over 7 years ago

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

#10 Updated by Ingo Schmitt almost 7 years ago

  • Is Regression set to No

Does this problem still exist in 6.2 Beta 4

#11 Updated by Tolleiv Nietsch almost 7 years ago

  • Status changed from Under Review to Closed

Kidding?

Also available in: Atom PDF