[FEATURE] TypoScript: Allow easy comparison against multiple values

Allow easy comparison like:
[globalVar = TSFE:id = 10|12|15]   (in list)
[globalVar = TSFE:id != 10|12|15]  (not in list)

Change-Id: Iae920720ae6058c2cd741f74204c2fbce779e00f
Resolves: #39700
Releases: 6.0
Reviewed-on: http://review.typo3.org/13589
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Ingo Renner
Tested-by: Ingo Renner

diff --git a/t3lib/matchcondition/class.t3lib_matchcondition_abstract.php b/t3lib/matchcondition/class.t3lib_matchcondition_abstract.php
index f125461..53bfdc4 100644 (file)
--- a/t3lib/matchcondition/class.t3lib_matchcondition_abstract.php
+++ b/t3lib/matchcondition/class.t3lib_matchcondition_abstract.php
@@ -482,7 +482,19 @@ abstract class t3lib_matchCondition_abstract {
                                        return ($leftValue <= doubleval($rightValue));
                                break;
                                case '!=':
-                                       return ($leftValue != doubleval($rightValue));
+                                               // multiple values may be split with '|'
+                                               // see if none matches ("not in list")
+                                       $found = FALSE;
+
+                                       $rightValueParts = t3lib_div::trimExplode('|', $rightValue);
+                                       foreach ($rightValueParts as $rightValueSingle) {
+                                               if ($leftValue === doubleval($rightValueSingle)) {
+                                                       $found = TRUE;
+                                                       break;
+                                               }
+                                       }
+
+                                       return ($found === FALSE);
                                break;
                                case '<':
                                        return ($leftValue < doubleval($rightValue));
@@ -492,7 +504,20 @@ abstract class t3lib_matchCondition_abstract {
                                break;
                                default:
                                                // nothing valid found except '=', use '='
-                                       return ($leftValue == trim($rightValue));
+
+                                               // multiple values may be split with '|'
+                                               // see if one matches ("in list")
+                                       $found = FALSE;
+
+                                       $rightValueParts = t3lib_div::trimExplode('|', $rightValue);
+                                       foreach ($rightValueParts as $rightValueSingle) {
+                                               if ($leftValue == $rightValueSingle) {
+                                                       $found = TRUE;
+                                                       break;
+                                               }
+                                       }
+
+                                       return $found;
                                break;
                        }
                }

diff --git a/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_backendTest.php b/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_backendTest.php
index 6736c81..e146857 100644 (file)
--- a/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_backendTest.php
+++ b/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_backendTest.php
@@ -377,6 +377,22 @@ class t3lib_matchCondition_backendTest extends tx_phpunit_testcase {
                $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 == 10.1]'));
        }
 
+       /**
+        * Tests whether numerical comparison matches.
+        * @test
+        */
+       public function globalVarConditionMatchesOnEqualExpressionWithMultipleValues() {
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10 = 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 = 10.1|20.2|30.3]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20 = 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20.2 = 10.1|20.2|30.3]'));
+
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10 == 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 == 10.1|20.2|30.3]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20 == 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20.2 == 10.1|20.2|30.3]'));
+       }
+
        /**
         * Tests whether numerical comparison matches.
         * @test
@@ -386,6 +402,15 @@ class t3lib_matchCondition_backendTest extends tx_phpunit_testcase {
                $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 != 10.2]'));
        }
 
+       /**
+        * Tests whether numerical comparison matches.
+        * @test
+        */
+       public function globalVarConditionMatchesOnNotEqualExpressionWithMultipleValues() {
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10 != 20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 != 10.2|20.3]'));
+       }
+
        /**
         * Tests whether numerical comparison matches.
         * @test

diff --git a/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_frontendTest.php b/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_frontendTest.php
index 88827e9..ffc5eb4 100644 (file)
--- a/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_frontendTest.php
+++ b/tests/Unit/t3lib/matchcondition/class.t3lib_matchcondition_frontendTest.php
@@ -347,6 +347,22 @@ class t3lib_matchCondition_frontendTest extends tx_phpunit_testcase {
                $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 == 10.1]'));
        }
 
+       /**
+        * Tests whether numerical comparison matches.
+        * @test
+        */
+       public function globalVarConditionMatchesOnEqualExpressionWithMultipleValues() {
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10 = 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 = 10.1|20.2|30.3]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20 = 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20.2 = 10.1|20.2|30.3]'));
+
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10 == 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 == 10.1|20.2|30.3]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20 == 10|20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:20.2 == 10.1|20.2|30.3]'));
+       }
+
        /**
         * Tests whether numerical comparison matches.
         * @test
@@ -356,6 +372,15 @@ class t3lib_matchCondition_frontendTest extends tx_phpunit_testcase {
                $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 != 10.2]'));
        }
 
+       /**
+        * Tests whether numerical comparison matches.
+        * @test
+        */
+       public function globalVarConditionMatchesOnNotEqualExpressionWithMultipleValues() {
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10 != 20|30]'));
+               $this->assertTrue($this->matchCondition->match('[globalVar = LIT:10.1 != 10.2|20.3]'));
+       }
+
        /**
         * Tests whether numerical comparison matches.
         * @test