fix: remove escaping code for negative integer

It is inconsistent, with negative numbers we get a string. For positive, we get numbers.

Author: kenjis

system/Database/BaseConnection.php

@@ -1202,10 +1202,6 @@ public function escape($str)
             return ($str === false) ? 0 : 1;
         }
 
-        if (is_numeric($str) && $str < 0) {
-            return "{$str}";
-        }
-
         return $str ?? 'NULL';
     }
 

tests/system/Database/BaseQueryTest.php

@@ -363,6 +363,20 @@ public function testSetQueryBindsWithSetEscapeNegativeIntegers()
         $this->assertSame($expected, $query->getQuery());
     }
 
+    public function testSetQueryNamedBindsWithNegativeIntegers()
+    {
+        $query = new Query($this->db);
+
+        $query->setQuery(
+            'SELECT * FROM product WHERE date_pickup < DateAdd(month, :num:, Convert(date, GetDate())',
+            ['num' => -6]
+        );
+
+        $expected = 'SELECT * FROM product WHERE date_pickup < DateAdd(month, -6, Convert(date, GetDate())';
+
+        $this->assertSame($expected, $query->getQuery());
+    }
+
     /**
      * @see https://github.com/codeigniter4/CodeIgniter4/issues/2762
      */

tests/system/Database/Live/EscapeTest.php

@@ -40,7 +40,7 @@ protected function setUp(): void
      */
     public function testEscapeProtectsNegativeNumbers()
     {
-        $this->assertSame('-100', $this->db->escape(-100));
+        $this->assertSame(-100, $this->db->escape(-100));
     }
 
     public function testEscape()