Bug #54194

Setting a property on a TS object resets its processors

Added by Bastian Waidelich over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2013-12-04
Priority:Should have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%
Category:-
Target version:-

Description

Setting the property on a TypoScript object seems to reset its processor:

Setup:

page = TYPO3.Neos.NodeTypes:Text {
    text.@process.1 = ${String.toUpperCase(value)}
    text = 'foo bar'
}

Expected:

FOO BAR

Actual:

foo bar

When using Eel syntax:

text = ${'foo bar'}

the processor is not reset!

Also if the process line appears after the text assignment, obviously

Associated revisions

Revision 1292e176
Added by Sebastian Kurfuerst over 1 year ago

[BUGFIX] make sure simple values do not override processors

Setting a simple property on a TypoScript object has reset its processors,
as shown in the following example::

page = TYPO3.Neos.NodeTypes:Text {
         = ${String.toUpperCase(value)}
        text = 'foo bar'
    }

Expected::

FOO BAR

Actual (without this change)::

foo bar

Change-Id: I4453120f8c12ec3adda74178da569ea0c332625b
Fixes: #54194
Reviewed-on: https://review.typo3.org/25952
Reviewed-by: Bastian Waidelich
Tested-by: Bastian Waidelich
Reviewed-by: Aske Ertmann
Tested-by: Aske Ertmann

History

#1 Updated by Sebastian Kurfuerst over 1 year ago

  • Status changed from New to Accepted
  • Assigned To set to Sebastian Kurfuerst
  • Target version set to 1.0 beta 1

I'll fix that ASAP. bugfix still coming today

#2 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Accepted to Under Review

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

#3 Updated by Gerrit Code Review over 1 year ago

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

#4 Updated by Gerrit Code Review over 1 year ago

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

#5 Updated by Bastian Waidelich over 1 year ago

  • Target version deleted (1.0 beta 1)

#6 Updated by Sebastian Kurfuerst over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF