Bug #45165

Wrong result on unequal globalVar conditions

Added by Marc Bastian Heinrichs over 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
TypoScript
Target version:
Start date:
2013-02-05
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Test with

[globalVar = LIT:10 != 10]
page.5 = COA_INT
page.5.10 = TEXT
page.5.10.value = test
[global]

or see attached unit test, which fails.

Was introduced with #39700 and a type save check in != comparison.


Files

globalVarConditionMatchesNotOnNotEqualExpression.diff (874 Bytes) globalVarConditionMatchesNotOnNotEqualExpression.diff Marc Bastian Heinrichs, 2013-02-05 11:47
45165.diff (1.05 KB) 45165.diff Marc Bastian Heinrichs, 2013-02-05 11:49

Related issues

Related to TYPO3 Core - Feature #39700: TypoScript: Allow easy comparison against multiple valuesClosed2012-08-11

Actions
#1

Updated by Marc Bastian Heinrichs over 8 years ago

Here is the patch

#2

Updated by Thorsten Kahler over 8 years ago

  • Status changed from New to Accepted
  • Complexity set to easy
#3

Updated by Oliver Hader almost 8 years ago

  • Assignee set to Oliver Hader
#4

Updated by Oliver Hader almost 8 years ago

  • Subject changed from Condition [globalVar = LIT:10 != 10] is true to Wrong result on unequal globalVar conditions
#6

Updated by Gerrit Code Review almost 8 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21661

#7

Updated by Gerrit Code Review almost 8 years ago

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/21663

#8

Updated by Gerrit Code Review almost 8 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/21664

#9

Updated by Gerrit Code Review almost 8 years ago

Patch set 2 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/21663

#10

Updated by Oliver Hader almost 8 years ago

  • Target version set to next-patchlevel
#11

Updated by Marc Bastian Heinrichs almost 8 years ago

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

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF