Fixes #52 - getAttachmentContent() to return string instead of Stream…

README.md

@@ -98,6 +98,8 @@ $attachment_content = $ticket_article->getAttachmentContent(23);
 
 In the above example 23 is the ID of the attachment. This ID can be found within the `attachments` array of the ticket article data. Usually you want to loop over this array to fetch the content of all attachments.
 
+`getAttachmentContent()` returns the attachment content as a string, ready to use.
+
 ### Updating Resource objects
 If you fetched a `Resource` object and changed some values, you have to send your changes to Zammad. You do this with a simple call:
 ```php

src/Resource/TicketArticle.php

@@ -117,6 +117,9 @@ public function getAttachmentContent($attachment_id)
         }
 
         $content = $response->getBody();
+        if ( $content instanceof \Psr\Http\Message\StreamInterface ) {
+            $content = $content->getContents();
+        }
         return $content;
     }
 }

test/ZammadAPIClient/Resource/TicketArticleTest.php

@@ -166,7 +166,7 @@ public function testCreate( $values, $expected_success )
 
                     // Fetch attachment content
                     $content = $object->getAttachmentContent( $attachment['id'] );
-                    $this->assertEquals(
+                    $this->assertSame(
                         $expected_content,
                         $content,
                         "Content of file $filename must match expected one."