Merge pull request #42 from clue-labs/blob

Report correct field length for fields longer than 16k chars

Author: jsor

src/Protocal/Parser.php

@@ -256,7 +256,7 @@ public function parse($data)
                     $u                    = unpack('v', $this->read(2));
                     $field['charset']     = $u[1];
 
-                    $u                    = unpack('v', $this->read(4));
+                    $u                    = unpack('V', $this->read(4));
                     $field['length']      = $u[1];
 
                     $field['type']        = ord($this->read(1));

tests/ResultQueryTest.php

@@ -61,6 +61,26 @@ public function testSelectStaticTextWithQuestionMark()
         $loop->run();
     }
 
+    public function testSelectLongStaticTextWithProperType()
+    {
+        $loop = \React\EventLoop\Factory::create();
+
+        $connection = new \React\MySQL\Connection($loop, $this->getConnectionOptions());
+        $connection->connect(function () {});
+
+        $length = 40000;
+
+        $connection->query('SELECT ?', function ($command, $conn) use ($length) {
+            $this->assertEquals(false, $command->hasError());
+            $this->assertCount(1, $command->resultFields);
+            $this->assertEquals($length * 3, $command->resultFields[0]['length']);
+            $this->assertInstanceOf('React\MySQL\Connection', $conn);
+        }, str_repeat('.', $length));
+
+        $connection->close();
+        $loop->run();
+    }
+
     public function testSimpleSelect()
     {
         $loop = \React\EventLoop\Factory::create();