Bug #100775: TCEFORM altLabels for empty values don't work - TYPO3 Core - TYPO3 Forge

Custom queries

Accessibility
Easy tasks
Forge Triage
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
My open issues
No feedback within 90 days
Open Bugs
Open issues of 10
Recently modified
Regressions
Stabilization Issues
Usability or UX

Watchers (6)

Česlav Przywara
Christian Hackl
Christian Kuhn
Garvin Hicking
Jessica Jagodzinski
Jonas Eberle

Actions

Copy link

Bug #100775

open

TCEFORM altLabels for empty values don't work

Added by David Mellen over 1 year ago. Updated 4 months ago.

Status:

New

Priority:

Should have

Assignee:

Category:

FormEngine aka TCEforms

Target version:

Start date:

2023-04-28

Due date:

% Done:

Estimated time:

TYPO3 Version:

PHP Version:

8.2

Tags:

Complexity:

Is Regression:

Sprint Focus:

On Location Sprint

Description

Hi,

setting an altLabel for an empty value in TCEFORM doesn't seem to work anymore.
My tsconfig file contains these lines:

space_after_class {
  altLabels {
    . = Custom label
  }
}

After the update from 11 to 12 I get the following error:

#1655824257 LogicException
Identifier token stream must not be empty

If I change the tsconfig to

space_after_class.altLabels.. = Custom label

The error doesn't occur anymore, but the label doesn't change as well.

Files

clipboard-202311091157-ptx5z.png (44.3 KB) clipboard-202311091157-ptx5z.png

Marcin Sągol, 2023-11-09 10:57

Related issues 1 (0 open — 1 closed)

Related to TYPO3 Core - Feature #97816: New TypoScript parser

Closed

2022-06-27

Actions

Issue # Delay: days Cancel

History
Notes
Property changes

Actions

Copy link

Updated by Christian Kuhn over 1 year ago

Well, from TS parser point of view, that's invalid syntax.

It looks as if this syntax exception is only documented / usable for "altLabels", see https://docs.typo3.org/m/typo3/reference-tsconfig/main/en-us/PageTsconfig/TceForm.html#altlabels

Overriding the 'default' this way is odd in the first place. I must admit, I did not had this special case on my radar, that's why new ts parser explodes here. Guess we'll have to play around with this to find some solution since I think the "empty dot" isn't something we should allow since it collides with syntax specs, and was just a weird side effect of old parser.

Actions

Copy link

Updated by Christian Kuhn over 1 year ago

Related to Feature #97816: New TypoScript parser added

Actions

Copy link

Updated by Benni Mack over 1 year ago

Sprint Focus set to Stabilization Sprint

Actions

Copy link

Updated by Benni Mack over 1 year ago

Sprint Focus changed from Stabilization Sprint to On Location Sprint

Actions

Copy link

Updated by Marcin Sągol about 1 year ago

File clipboard-202311091157-ptx5z.png clipboard-202311091157-ptx5z.png added

@Christian Kuhn do you have any suggestion how we should solve this issue? There are several topic to consider here, and i will try to describe them.

First of all with current implementation (and as mentioned in this issue description) if we try to override this default value using the multiline syntax, we end up with the exception in the backend and it will break pretty much all modules inside Web section in TYPO3 panel, and only way to fix them, would be to go do database and remove fault tsconfig configuration from there to be able to load those modules again. So the question is - should we throw exception in such case, or handle this wrong configuration some other way (also to consider why this exception is not thronw for single line syntax).

Second, as you mentioned, having it working with old TS parser was a side effect, but even with there not all cases worked. For example you can't remove items with empty strig as a value with the `.removeItmes` method. This will ignore such item anyway.

Third, to make it work we could try to fix it in the new TS parser, but the question is if we should? Or shall we indeed think about it as a wrong/unsupported syntax. We could also for this single case (mentioned in issue description) set default value for this item as `default` no empty string. This way we can reference and modify valid value. But this could be a breaking change and also rises question if we should change it only for this single column in core or also for all others that exist and have default value as a empty string?

We should also update documentation. Now it states that we can use double dot syntax to override default value which is a empty string:

Actions

Copy link

Updated by Garvin Hicking about 1 year ago

How about inventing a custom placeholder like "__empty" or "_null" something like this? And letting that replace this "dot" syntax:

TCEFORM.tt_content.space_before_class.altLabels.__empty = foo
TCEFORM.tt_content.space_after_class {
  altLabels {
    __empty = FoobyDoo
  }
}

Of course it would mean adapting all the code that previously reacted on the "dot".

Using the invalid dot-syntax would then still explode (that's fair I guess), and there's no real BC here if the former syntax currently doesn't work, but I'd suggest to declare it as a BC-level impact ChangeLog entry to let people recognize formally, that the notation was dropped (and be reflected in the docs).

Actions

Copy link

Updated by Christian Hackl 8 months ago

I would advocate the third option (i.e. add "default" as a value for all fields where this is missing).
Because in a form with a select field and a selected "option" without a value, the "innerHTML" string is always transferred... And we could theoretically translate this "innerHTML", then depending on the translation something else would be in the request <- stupid.
It would also be consistent!

The alternative would be to completely rebuild the select option thing by only using the "innerHTML" values as the value and giving the option the "label" attribute.

Simple example to understand:

<!DOCTYPE html>
<html lang="en">
<head>
    <title>test</title>
</head>
<body>
    <form action="/test.php">
        <select name="test">
            <option>empty</option>
            <option label="cool">dd</option>
            <option value="10">10</option>
        </select>
        <input type="submit" value="send">
    </form>

    <div>
        <?php
            var_dump($_REQUEST);
        ?>
    </div>
</body>
</html>

Actions

Copy link

Updated by Garvin Hicking 4 months ago

Category set to FormEngine aka TCEforms

Actions

Copy link

Also available in: Atom PDF

Project

General

Profile

TYPO3 Core

Custom queries

Watchers (6)

Bug #100775

TCEFORM altLabels for empty values don't work

Updated by Christian Kuhn over 1 year ago

Updated by Christian Kuhn over 1 year ago

Updated by Benni Mack over 1 year ago

Updated by Benni Mack over 1 year ago

Updated by Marcin Sągol about 1 year ago

Updated by Garvin Hicking about 1 year ago

Updated by Christian Hackl 8 months ago

Updated by Garvin Hicking 4 months ago