Support parsing messages with multiple values for a single field

Author: clue

src/Protocol/Parser.php

@@ -54,7 +54,14 @@ private function parseMessage($message)
                 $key = substr($line, 0, $pos);
             }
 
-            $fields[$key] = $value;
+            if (isset($fields[$key])) {
+                if (!is_array($fields[$key])) {
+                    $fields[$key] = array($fields[$key]);
+                }
+                $fields[$key][] = $value;
+            } else {
+                $fields[$key] = $value;
+            }
         }
 
         reset($fields);

tests/Protocol/ParserTest.php

@@ -49,6 +49,22 @@ public function testParsingMultipleEvents()
         $this->assertEquals('TestA', $first->getName());
     }
 
+    public function testParseResponseMultipleValues()
+    {
+        $parser = new Parser();
+        $this->assertEquals(array(), $parser->push("Asterisk Call Manager/1.3\r\n"));
+
+        $ret = $parser->push("Response: Success\r\nMessage: one\r\nMessage: two\r\n\r\n");
+        $this->assertCount(1, $ret);
+
+        $first = reset($ret);
+        /* @var $first Clue\React\Ami\Protocol\Response */
+
+        $this->assertInstanceOf('Clue\React\Ami\Protocol\Response', $first);
+        $this->assertEquals('one', $first->getFieldValue('Message'));
+        $this->assertEquals(array('one', 'two'), $first->getFieldValues('Message'));
+    }
+
     public function testParsingCommandResponse()
     {
         $parser = new Parser();