Treat bit map values in ToNull filter as boolean rather than numbers

Fix for issue #50

Author: David Stockton

src/ToNull.php

@@ -78,9 +78,9 @@ public function setType($type = null)
             $detected = 0;
             foreach ($type as $value) {
                 if (is_int($value)) {
-                    $detected += $value;
+                    $detected |= $value;
                 } elseif (in_array($value, $this->constants)) {
-                    $detected += array_search($value, $this->constants);
+                    $detected |= array_search($value, $this->constants);
                 }
             }
 
@@ -125,48 +125,42 @@ public function filter($value)
         $type = $this->getType();
 
         // FLOAT (0.0)
-        if ($type >= self::TYPE_FLOAT) {
-            $type -= self::TYPE_FLOAT;
+        if ($type & self::TYPE_FLOAT) {
             if (is_float($value) && ($value == 0.0)) {
                 return;
             }
         }
 
         // STRING ZERO ('0')
-        if ($type >= self::TYPE_ZERO_STRING) {
-            $type -= self::TYPE_ZERO_STRING;
+        if ($type & self::TYPE_ZERO_STRING) {
             if (is_string($value) && ($value == '0')) {
                 return;
             }
         }
 
         // STRING ('')
-        if ($type >= self::TYPE_STRING) {
-            $type -= self::TYPE_STRING;
+        if ($type & self::TYPE_STRING) {
             if (is_string($value) && ($value == '')) {
                 return;
             }
         }
 
         // EMPTY_ARRAY (array())
-        if ($type >= self::TYPE_EMPTY_ARRAY) {
-            $type -= self::TYPE_EMPTY_ARRAY;
+        if ($type & self::TYPE_EMPTY_ARRAY) {
             if (is_array($value) && ($value == [])) {
                 return;
             }
         }
 
         // INTEGER (0)
-        if ($type >= self::TYPE_INTEGER) {
-            $type -= self::TYPE_INTEGER;
+        if ($type & self::TYPE_INTEGER) {
             if (is_int($value) && ($value == 0)) {
                 return;
             }
         }
 
         // BOOLEAN (false)
-        if ($type >= self::TYPE_BOOLEAN) {
-            $type -= self::TYPE_BOOLEAN;
+        if ($type & self::TYPE_BOOLEAN) {
             if (is_bool($value) && ($value == false)) {
                 return;
             }

test/ToNullTest.php

@@ -100,6 +100,40 @@ public function testGettingDefaultType()
         $this->assertEquals(63, $filter->getType());
     }
 
+    /**
+     * Ensures that providing a duplicate initializing type results in the expected type
+     *
+     * @param mixed $type Type to duplicate initialization
+     * @param mixed $expected Expected resulting type
+     *
+     * @dataProvider duplicateTypeProvider
+     */
+    public function testDuplicateInitializationResultsInCorrectType($type, $expected)
+    {
+        $filter = new ToNullFilter([$type, $type]);
+        $this->assertEquals($expected, $filter->getType());
+    }
+
+    public static function duplicateTypeProvider()
+    {
+        return [
+            [ToNullFilter::TYPE_BOOLEAN, ToNullFilter::TYPE_BOOLEAN,],
+            [ToNullFilter::TYPE_INTEGER, ToNullFilter::TYPE_INTEGER,],
+            [ToNullFilter::TYPE_EMPTY_ARRAY, ToNullFilter::TYPE_EMPTY_ARRAY,],
+            [ToNullFilter::TYPE_STRING, ToNullFilter::TYPE_STRING,],
+            [ToNullFilter::TYPE_ZERO_STRING, ToNullFilter::TYPE_ZERO_STRING,],
+            [ToNullFilter::TYPE_FLOAT, ToNullFilter::TYPE_FLOAT,],
+            [ToNullFilter::TYPE_ALL, ToNullFilter::TYPE_ALL,],
+            ['boolean', ToNullFilter::TYPE_BOOLEAN,],
+            ['integer', ToNullFilter::TYPE_INTEGER,],
+            ['array', ToNullFilter::TYPE_EMPTY_ARRAY,],
+            ['string', ToNullFilter::TYPE_STRING,],
+            ['zero', ToNullFilter::TYPE_ZERO_STRING,],
+            ['float', ToNullFilter::TYPE_FLOAT,],
+            ['all', ToNullFilter::TYPE_ALL,],
+        ];
+    }
+
     public static function defaultTestProvider()
     {
         return [