StreamOutputStream::supportsAnsi() - Code Metrics - Inspection of "Merge pull request #15 from stof/patch-1" - webmozart/console - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3b818e...bde780 )

by Bernhard

created 2016-08-09 15:28 UTC

StreamOutputStream::supportsAnsi() B

↳ Parent: StreamOutputStream

Complexity

Conditions	5
Paths	5

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	5.3906

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	8
ccs	3
cts	4
cp	0.75
rs	8.8571
cc	5
eloc	4
nc	5
nop	0
crap	5.3906

<?php

/*
 * This file is part of the webmozart/console package.
 *
 * (c) Bernhard Schussek <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Webmozart\Console\IO\OutputStream;

use Webmozart\Assert\Assert;
use Webmozart\Console\Api\IO\IOException;
use Webmozart\Console\Api\IO\OutputStream;

/**
 * An output stream that writes to a PHP stream.
 *
 * @since  1.0
 *
 * @author Bernhard Schussek <[email protected]>
 */
class StreamOutputStream implements OutputStream
{
    /**
     * @var resource
     */
    private $stream;

    /**
     * Creates the stream.
     *
     * @param resource $stream A stream resource.
     */
    public function __construct($stream)
    {
        Assert::resource($stream, 'stream');

        $this->stream = $stream;
    }

    /**
     * {@inheritdoc}
     */
    public function write($string)
    {
        if (null === $this->stream) {
            throw new IOException('Cannot read from a closed input.');
        }

        if (false === fwrite($this->stream, $string)) {
            throw new IOException('Could not write stream.');
        }
    }

    /**
     * {@inheritdoc}
     */
    public function flush()
    {
        if (null === $this->stream) {
            throw new IOException('Cannot read from a closed input.');
        }

        if (false === fflush($this->stream)) {
            throw new IOException('Could not flush stream.');
        }
    }

    /**
     * {@inheritdoc}
     */
    public function supportsAnsi()
    {
        if (DIRECTORY_SEPARATOR === '\\') {
            return false !== getenv('ANSICON') || 'ON' === getenv('ConEmuANSI') || 'xterm' === getenv('TERM');
        }

        return function_exists('posix_isatty') && @posix_isatty($this->stream);
    }

    /**
     * {@inheritdoc}
     */
    public function close()
    {
        if ($this->stream) {
            @fclose($this->stream);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
            $this->stream = null;
        }
    }

    /**
     * {@inheritdoc}
     */
    public function isClosed()
    {
        return null === $this->stream;
    }
}


1		<?php
2
3		/*
4		* This file is part of the webmozart/console package.
5		*
6		* (c) Bernhard Schussek <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Webmozart\Console\IO\OutputStream;
13
14		use Webmozart\Assert\Assert;
15		use Webmozart\Console\Api\IO\IOException;
16		use Webmozart\Console\Api\IO\OutputStream;
17
18		/**
19		* An output stream that writes to a PHP stream.
20		*
21		* @since 1.0
22		*
23		* @author Bernhard Schussek <[email protected]>
24		*/
25		class StreamOutputStream implements OutputStream
26		{
27		/**
28		* @var resource
29		*/
30		private $stream;
31
32		/**
33		* Creates the stream.
34		*
35		* @param resource $stream A stream resource.
36		*/
37	144	public function __construct($stream)
38		{
39	144	Assert::resource($stream, 'stream');
40
41	144	$this->stream = $stream;
42	144	}
43
44		/**
45		* {@inheritdoc}
46		*/
47	2	public function write($string)
48		{
49	2	if (null === $this->stream) {
50	1	throw new IOException('Cannot read from a closed input.');
51		}
52
53	1	if (false === fwrite($this->stream, $string)) {
54		throw new IOException('Could not write stream.');
55		}
56	1	}
57
58		/**
59		* {@inheritdoc}
60		*/
61	1	public function flush()
62		{
63	1	if (null === $this->stream) {
64	1	throw new IOException('Cannot read from a closed input.');
65		}
66
67		if (false === fflush($this->stream)) {
68		throw new IOException('Could not flush stream.');
69		}
70		}
71
72		/**
73		* {@inheritdoc}
74		*/
75	138	public function supportsAnsi()
76		{
77	138	if (DIRECTORY_SEPARATOR === '\\') {
78		return false !== getenv('ANSICON') \|\| 'ON' === getenv('ConEmuANSI') \|\| 'xterm' === getenv('TERM');
79		}
80
81	138	return function_exists('posix_isatty') && @posix_isatty($this->stream);
82		}
83
84		/**
85		* {@inheritdoc}
86		*/
87	3	public function close()
88		{
89	3	if ($this->stream) {
90	3	@fclose($this->stream);
		0 ignored issues – show Security Best Practice introduced 2015-12-16 14:17 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
91	3	$this->stream = null;
92		}
93	3	}
94
95		/**
96		* {@inheritdoc}
97		*/
98		public function isClosed()
99		{
100		return null === $this->stream;
101		}
102		}
103

webmozart / console

Push — master ( 3b818e...bde780 )

StreamOutputStream::supportsAnsi() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like