WithBodyStreamTrait::getBodyLength() - Code Metrics - bee4/transport - Measure and Improve Code Quality continuously with Scrutinizer

WithBodyStreamTrait::getBodyLength() A
last analyzed 2015-12-09 17:36 UTC

↳ Parent: WithBodyStreamTrait

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	7
ccs	5
cts	5
cp	1
rs	9.4286
cc	2
eloc	5
nc	2
nop	0
crap	2

<?php
/**
 * This file is part of the bee4/transport package.
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @copyright Bee4 2015
 * @author Stephane HULARD <[email protected]>
 * @package Bee4\Transport\Message
 */
namespace Bee4\Transport\Message;

use InvalidArgumentException;

/**
 * Allow the body to be a stream
 * @package Bee4\Transport\Message
 */
trait WithBodyStreamTrait
{
    use WithBodyTrait {
        WithBodyTrait::setBody as private setStringBody;
    }

    /**
     * Set body with stream acceptance
     * @param resource|string $body Request body
     */
    public function setBody($body)
    {
        if (is_string($body)) {
            return $this->setStringBody($body);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        if (!is_resource($body)) {
            throw new InvalidArgumentException('Body must be a stream !');
        }
        if (!stream_is_local($body)) {
            throw new InvalidArgumentException('Body must be a local stream !');
        }

        rewind($body);
        $this->body = $body;

    }

    /**
     * Check if the current body is a stream or not
     * @return boolean
     */
    public function hasBodyStream()
    {
        return is_resource($this->getBody());
    }

    /**
     * Retrieve stream meta data if available
     * @return array|null
     */
    public function getMetaData()
    {
        if ($this->hasBodyStream()) {
            return stream_get_meta_data($this->getBody());
        }
        return null;
    }

    /**
     * Retrieve the body length from string or stream
     * @return integer
     */
    public function getBodyLength()
    {
        $md = $this->getMetaData();
        return null !== $md
            ?filesize($md['uri'])
            :strlen($this->getBody());
    }
}


1		<?php
2		/**
3		* This file is part of the bee4/transport package.
4		* For the full copyright and license information, please view the LICENSE
5		* file that was distributed with this source code.
6		*
7		* @copyright Bee4 2015
8		* @author Stephane HULARD <[email protected]>
9		* @package Bee4\Transport\Message
10		*/
11		namespace Bee4\Transport\Message;
12
13		use InvalidArgumentException;
14
15		/**
16		* Allow the body to be a stream
17		* @package Bee4\Transport\Message
18		*/
19		trait WithBodyStreamTrait
20		{
21		use WithBodyTrait {
22		WithBodyTrait::setBody as private setStringBody;
23		}
24
25		/**
26		* Set body with stream acceptance
27		* @param resource\|string $body Request body
28		*/
29	2	public function setBody($body)
30		{
31	2	if (is_string($body)) {
32	1	return $this->setStringBody($body);
		0 ignored issues – show Bug introduced 2015-11-24 15:02 UTC by Report Bug Copy Issue Report It seems like `setStringBody()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
33		}
34
35	2	if (!is_resource($body)) {
36		throw new InvalidArgumentException('Body must be a stream !');
37		}
38	2	if (!stream_is_local($body)) {
39		throw new InvalidArgumentException('Body must be a local stream !');
40		}
41
42	2	rewind($body);
43	2	$this->body = $body;
		0 ignored issues – show Documentation Bug introduced 2015-11-24 15:02 UTC by Report Bug Copy Issue Report It seems like `$body` of type `resource` is incompatible with the declared type `string` of property `$body`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
44	2	}
45
46		/**
47		* Check if the current body is a stream or not
48		* @return boolean
49		*/
50	3	public function hasBodyStream()
51		{
52	3	return is_resource($this->getBody());
53		}
54
55		/**
56		* Retrieve stream meta data if available
57		* @return array\|null
58		*/
59	2	public function getMetaData()
60		{
61	2	if ($this->hasBodyStream()) {
62	2	return stream_get_meta_data($this->getBody());
63		}
64	1	return null;
65		}
66
67		/**
68		* Retrieve the body length from string or stream
69		* @return integer
70		*/
71	2	public function getBodyLength()
72		{
73	2	$md = $this->getMetaData();
74	2	return null !== $md
75	2	?filesize($md['uri'])
76	2	:strlen($this->getBody());
77		}
78		}
79

bee4 / transport

WithBodyStreamTrait::getBodyLength() A last analyzed 2015-12-09 17:36 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

WithBodyStreamTrait::getBodyLength() A
last analyzed 2015-12-09 17:36 UTC