From a89acd005990aa42339d4f90fb5c5c5520edc906 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Rapha=C3=ABl=20Gertz?= Date: Sun, 8 Aug 2021 14:34:37 +0200 Subject: [PATCH] New tree layout Add strict types Improve documentation Cleanup --- Filter/CPackFilter.php | 34 ++++++++++++++++++++++++++-------- 1 file changed, 26 insertions(+), 8 deletions(-) diff --git a/Filter/CPackFilter.php b/Filter/CPackFilter.php index ca38c3b..853e94a 100644 --- a/Filter/CPackFilter.php +++ b/Filter/CPackFilter.php @@ -1,10 +1,22 @@ - + * + * For the full copyright and license information, please view the LICENSE + * file that was distributed with this source code. + */ + +namespace Rapsys\PackBundle\Filter; -use Rapsys\PackBundle\Twig\Filter\FilterInterface; use Twig\Error\Error; +use Twig\Source; +/** + * {@inheritdoc} + */ class CPackFilter implements FilterInterface { //Default bin private $bin; @@ -18,9 +30,12 @@ class CPackFilter implements FilterInterface { //Twig template line private $line; - //Configure the object - //XXX: compress can be minify or pretty - public function __construct($fileName, $line, $bin = 'cpack', $compress = 'minify') { + /** + * Setup cpack filter + * + * @xxx compress can be minify or pretty + */ + public function __construct(Source $fileName, int $line, string $bin = 'cpack', string $compress = 'minify') { //Set fileName $this->fileName = $fileName; @@ -47,7 +62,10 @@ class CPackFilter implements FilterInterface { } } - public function process($content) { + /** + * {@inheritdoc} + */ + public function process(string $content): string { //Create descriptors $descriptorSpec = array( 0 => array('pipe', 'r'), @@ -58,7 +76,7 @@ class CPackFilter implements FilterInterface { //Open process if (is_resource($proc = proc_open($this->bin, $descriptorSpec, $pipes))) { //Set stderr as non blocking - stream_set_blocking($pipes[2], 0); + stream_set_blocking($pipes[2], false); //Send content to stdin fwrite($pipes[0], $content); -- 2.41.0