I currently output an image like this (inside a controller class):
ob_start();
header("Content-Type: image/gif");
imagegif($image_resource);
imagedestroy($image_resource);
$content_string = ob_get_clean();
$this->getResponse()->setContent($content_string);
Is there a better way without actually capturing the output buffer? since this is not very testable...
CodePudding user response:
The Only way to capture the image without output-buffer is to write the image to a file:
$tmpFile = tmpfile();
$path = stream_get_meta_data($tmpFile)['uri'];
imagegif($image_resource, $tmpFile);
$content_string = file_get_contents($path);
imagedestroy($image_resource);
fclose($tmpFile);
$this->getResponse()->setContent($content_string);
I created a temporary file with the tmpfile() method, and got the filename via stream_get_metadata() function. Then I loaded the content of the file into the string variable and closed the filehandle to the tmpfile so it gets removed.
Personally i would use the output buffer but move the code into a separate function or class. That way you can isolate and mock its behavior for testing purposes:
/** @var GDImage|resource $image */
function getImage($image): string
{
ob_start();
imagegif($image);
$returnData = ob_get_contents();
ob_end_clean();
imagedestroy($image);
return $returnData;
}
$content_string = getImage($image_resource);
$this->getResponse()->setContent($content_string);
In any way i would advise you to set the header on the response object instead of writing it manually:
$this->getResponse()->getHeaders()->addHeaders([
'Content-Type' => 'image/gif',
]);
$this->getResponse()->setContent($content_string);
