Task #48359
Work Package #48275: TypoScript consistency
Simple Value / Eel Expression / TypoScript Object Interoperability
Status: | Resolved | Start date: | 2013-05-17 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Sebastian Kurfuerst | % Done: | 100% |
|
Category: | - | |||
Target version: | Base Distribution - 1.0 beta 1 |
Description
currently, there is a big distinction between simple values and eel expressions, and sub-TypoScript objects; despite them sharing the same namespace.
However, basically, all of the above mentioned three TS data types produce a value which they return to their parent.
That's why the following should be made possible:
page.template = "adf" // works today page.template = ${...} // works today page.template = Case { // does NOT work today; but we want to make this possible. foo.condition = ${...} foo.then = "myTemplate" ... }
Same for the other way around:
page.body.sections.main = Template // works today page.body.sections.main = "my static string" // does NOT work today, but we want to make this possible.
Related issues
Associated revisions
[FEATURE] Value / Eel Expression / TypoScript-Object interoperability
Currently, there is a big distinction between simple values and Eel
expressions, and sub-TypoScript objects; despite them sharing the same
namespace.
However, basically, all of the above mentioned three TS data types produce
a value which they return to their parent.
This functionality must be tested in the following three dimensions:
1.a using Runtime::evaluate()
1.b using $this->tsValue() inside a TypoScript object
2.a without processors
2.b with processors
3.a pass in a simple type
3.b pass in an Eel Expression
3.c pass in a TypoScript object
This change implements the functionality in a test-driven way, testing
all the 12 cases from above.
This change also removes all references to "implementationClassName" which
was deprecated for a long time now as well. Using
"implementationClassName" instead of "@class" was broken anyway so this
is not considered breaking.
Resolves: #48359
Change-Id: I3f2569025808f7fa13bee5164b94cbfc982889bf
Reviewed-on: https://review.typo3.org/21003
Reviewed-by: Christian Mueller
Tested-by: Christian Mueller
History
#1 Updated by Sebastian Kurfuerst about 2 years ago
TODO: also update the TS reference (TYPO3.TypoScript:Value); and completely remove this type.
#2 Updated by Gerrit Code Review about 2 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#3 Updated by Sebastian Kurfuerst about 2 years ago
- Status changed from Accepted to Under Review
- % Done changed from 0 to 30
#4 Updated by Gerrit Code Review about 2 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#5 Updated by Gerrit Code Review about 2 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#6 Updated by Gerrit Code Review about 2 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#7 Updated by Gerrit Code Review about 2 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#8 Updated by Gerrit Code Review about 2 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#9 Updated by Gerrit Code Review about 2 years ago
Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#10 Updated by Gerrit Code Review about 2 years ago
Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#11 Updated by Gerrit Code Review about 2 years ago
Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#12 Updated by Gerrit Code Review about 2 years ago
Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#13 Updated by Gerrit Code Review about 2 years ago
Patch set 11 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#14 Updated by Gerrit Code Review about 2 years ago
Patch set 12 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#15 Updated by Gerrit Code Review about 2 years ago
Patch set 13 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#16 Updated by Gerrit Code Review about 2 years ago
Patch set 14 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#17 Updated by Gerrit Code Review about 2 years ago
Patch set 15 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#18 Updated by Gerrit Code Review about 2 years ago
Patch set 16 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#19 Updated by Gerrit Code Review about 2 years ago
Patch set 17 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#20 Updated by Gerrit Code Review almost 2 years ago
Patch set 18 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#21 Updated by Gerrit Code Review almost 2 years ago
Patch set 19 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#22 Updated by Gerrit Code Review almost 2 years ago
Patch set 20 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#23 Updated by Gerrit Code Review almost 2 years ago
Patch set 21 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21003
#24 Updated by Sebastian Kurfuerst almost 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 30 to 100
Applied in changeset af27c1837f290746ff497656692d089807d0be24.