AttachmentsBody::build() - Code Metrics - Inspection of "adding support to send and fetch attachments" - php-xapi/client - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#17)

by Christian

created 2016-10-08 21:54 UTC

AttachmentsBody::build() A

↳ Parent: AttachmentsBody

Complexity

Conditions	2
Paths	2

Size

Total Lines	22
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	22
ccs	0
cts	19
cp	0
rs	9.2
c	0
b	0
f	0
cc	2
eloc	16
nc	2
nop	0
crap	6

<?php

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

namespace Xabbuh\XApi\Client\Http;

use Xabbuh\XApi\Model\Attachment;

/**
 * @author Christian Flothmann <[email protected]>
 */
class AttachmentsBody
{
    private $boundary;
    private $json;
    private $attachments;

    /**
     * @param string       $json
     * @param Attachment[] $attachments
     */
    public function __construct($json, array $attachments)
    {
        $this->boundary = uniqid();
        $this->json = $json;
        $this->attachments = $attachments;
    }

    public function getBoundary()
    {
        return $this->boundary;
    }

    public function build()
    {
        $body = '--'.$this->boundary."\r\n";
        $body .= "Content-Type: application/json\r\n";
        $body .= 'Content-Length: '.strlen($this->json)."\r\n";
        $body .= "\r\n";
        $body .= $this->json."\r\n";

        foreach ($this->attachments as $attachment) {
            $body .= '--'.$this->boundary."\r\n";
            $body .= 'Content-Type: '.$attachment->getContentType()."\r\n";
            $body .= "Content-Transfer-Encoding: binary\r\n";
            $body .= 'Content-Length: '.$attachment->getLength()."\r\n";
            $body .= 'X-Experience-API-Hash: '.$attachment->getSha2()."\r\n";
            $body .= "\r\n";
            $body .= $attachment->getContent()."\r\n";

        }

        $body .= '--'.$this->boundary.'--'."\r\n";

        return $body;
    }
}


1			<?php
2
3			/*
4			* This file is part of the xAPI package.
5			*
6			* (c) Christian Flothmann <[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 Xabbuh\XApi\Client\Http;
13
14			use Xabbuh\XApi\Model\Attachment;
15
16			/**
17			* @author Christian Flothmann <[email protected]>
18			*/
19			class AttachmentsBody
20			{
21			private $boundary;
22			private $json;
23			private $attachments;
24
25			/**
26			* @param string $json
27			* @param Attachment[] $attachments
28			*/
29			public function __construct($json, array $attachments)
30			{
31			$this->boundary = uniqid();
32			$this->json = $json;
33			$this->attachments = $attachments;
34			}
35
36			public function getBoundary()
37			{
38			return $this->boundary;
39			}
40
41			public function build()
42			{
43			$body = '--'.$this->boundary."\r\n";
44			$body .= "Content-Type: application/json\r\n";
45			$body .= 'Content-Length: '.strlen($this->json)."\r\n";
46			$body .= "\r\n";
47			$body .= $this->json."\r\n";
48
49			foreach ($this->attachments as $attachment) {
50			$body .= '--'.$this->boundary."\r\n";
51			$body .= 'Content-Type: '.$attachment->getContentType()."\r\n";
52			$body .= "Content-Transfer-Encoding: binary\r\n";
53			$body .= 'Content-Length: '.$attachment->getLength()."\r\n";
54			$body .= 'X-Experience-API-Hash: '.$attachment->getSha2()."\r\n";
55			$body .= "\r\n";
56			$body .= $attachment->getContent()."\r\n";
			0 ignored issues – show Bug introduced 2016-10-08 21:57 UTC by Report Bug Copy Issue Report The method `getContent()` does not exist on `Xabbuh\XApi\Model\Attachment`. Did you maybe mean `getContentType()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
57			}
58
59			$body .= '--'.$this->boundary.'--'."\r\n";
60
61			return $body;
62			}
63			}
64

php-xapi / client

Pull Request — master (#17)

AttachmentsBody::build() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like