Project

General

Profile

Actions
Copy link

Task #86196

closed

Centralize ExpressionLanguage Registration via API

Added by Susanne Moog about 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Susanne Moog
Category:
Code Cleanup
Target version:
9 LTS
Start date:
2018-09-07
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Currently Symfony ExpressionLanguage is used in two contexts:

- the form framework
- typoscript condition matching

registration and providing context is done in two different ways and is extensible in two different ways. Before the LTS release these should be unified and documented.

Tasks:
- Add Configuration/ExpressionLanguage.php and corresponding loader with caching


// Config

<?php

return [
    'conditions' => [
            \TYPO3\CMS\Core\ExpressionLanguage\Typo3ConditionProvider
    ],
    'ext-form' => [
            \TYPO3\CMS\Core\ExpressionLanguage\Typo3ConditionProvider,
            \TYPO3\CMS\Form\Domain\Condition\ConditionProvider
    ]
];

// Usage:
$expressionLanguage = new \TYPO3\CMS\Core\ExpressionLanguage\Resolver('ext-form');

- Load expressionLanguage via loader with context argument in form and conditionMatcher
- Remove yaml config in form

Related issues 3 (0 open — 3 closed)

Related to TYPO3 Core - Feature #84133: Variants - Frontend implementationClosedRalf Zimmermann2018-05-24

Actions
Related to TYPO3 Core - Bug #92471: condition contentObject["pid"] not working in variantClosed2020-10-02

Actions
Related to TYPO3 Core - Bug #97023: EXT:form - Outdated documentation of expression languageClosedElias Häußler2022-02-24

Actions
Actions
Copy link
 #1

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #3

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #4

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #5

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #6

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #7

Updated by Benni Mack about 6 years ago

just don't call it `ext-form` please. either "form" or form-framework. I'm for "form".

Also "typoscript-conditions"?

Because we're going to have "typoscript-if" and "tca-display-conditions" and maybe "routing"

Actions
Copy link
 #8

Updated by Susanne Moog about 6 years ago

  • Status changed from New to In Progress
  • Assignee set to Susanne Moog
Actions
Copy link
 #9

Updated by Susanne Moog about 6 years ago

@Benni Mack while building I noticed we don't have that many contexts. We should not distinguish TypoScript conditions and other scopes at all. In fact conditions are conditions. The TypoScript conditions basically provide TYPO3 conditions that should be usable in any context (extensions PHP code, typoscript, with an expression language viewhelper even in fluid) as the methods and vars provided there aren't specific to TypoScript.

We can talk about that again, I'll push the WIP now.

Actions
Copy link
 #10

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #11

Updated by Susanne Moog about 6 years ago

  • Description updated (diff)
Actions
Copy link
 #12

Updated by Gerrit Code Review about 6 years ago

  • Status changed from In Progress 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/58232

Actions
Copy link
 #13

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #14

Updated by Susanne Moog about 6 years ago

Actions
Copy link
 #15

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #16

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #17

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #18

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #19

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #20

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #21

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #22

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #23

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #24

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #25

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #26

Updated by Gerrit Code Review about 6 years 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/58232

Actions
Copy link
 #27

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #28

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #29

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #30

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #31

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #32

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #33

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #34

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #35

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #36

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #37

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #38

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #39

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #40

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #41

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #42

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #43

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #44

Updated by Gerrit Code Review about 6 years ago

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

Actions
Copy link
 #45

Updated by Susanne Moog about 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #46

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions
Copy link
 #47

Updated by Christian Kuhn over 3 years ago

  • Related to Bug #92471: condition contentObject["pid"] not working in variant added
Actions
Copy link
 #48

Updated by Elias Häußler over 2 years ago

  • Related to Bug #97023: EXT:form - Outdated documentation of expression language added
Actions
Copy link

Also available in: Atom PDF