ElementFinderFactory::getSupportedEncodings() - Code Metrics - Inspection of "Merge pull request #12 from ChrisGalliano/master" - xparse/Parser - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4a2eaa...5e2e05 )

by Shcherbak

created 2016-03-30 10:55 UTC

ElementFinderFactory::getSupportedEncodings() B

↳ Parent: Project

Complexity

Conditions	5
Paths	4

Size

Total Lines	17
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	17
ccs	11
cts	11
cp	1
rs	8.8571
cc	5
eloc	10
nc	4
nop	0
crap	5

<?php

  namespace Xparse\Parser\Helper;

  use GuzzleHttp\Psr7\Response;
  use Xparse\ElementFinder\ElementFinder;
  use Xparse\ElementFinder\Helper;

  /**
   * Create ElementFinder.
   * Convert charset to UTF-8
   * Convert relative links to absolute
   * @package Xparse\Parser\Helper
   */
  class ElementFinderFactory implements ElementFinderFactoryInterface {

    /**
     * @param Response $response
     * @param string $affectedUrl
     * @return ElementFinder
     */
    public function create(Response $response, $affectedUrl = '') {
      $html = $response->getBody();
      $html = Helper::safeEncodeStr((string) $html);
      $contentType = $response->getHeaderLine('content-type');
      $html = HtmlEncodingConverter::convertToUtf($html, $contentType);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
      $page = new ElementFinder((string) $html);

      if ($affectedUrl) {
        LinkConverter::convertUrlsToAbsolute($page, $affectedUrl);
      }

      return $page;
    }

  }

1		<?php
2
3		namespace Xparse\Parser\Helper;
4
5		use GuzzleHttp\Psr7\Response;
6		use Xparse\ElementFinder\ElementFinder;
7		use Xparse\ElementFinder\Helper;
8
9		/**
10		* Create ElementFinder.
11		* Convert charset to UTF-8
12		* Convert relative links to absolute
13		* @package Xparse\Parser\Helper
14		*/
15		class ElementFinderFactory implements ElementFinderFactoryInterface {
16
17		/**
18		* @param Response $response
19		* @param string $affectedUrl
20		* @return ElementFinder
21		*/
22	13	public function create(Response $response, $affectedUrl = '') {
23	13	$html = $response->getBody();
24	13	$html = Helper::safeEncodeStr((string) $html);
25	13	$contentType = $response->getHeaderLine('content-type');
26	13	$html = HtmlEncodingConverter::convertToUtf($html, $contentType);
		0 ignored issues – show Documentation introduced 2016-03-30 10:18 UTC by Report Bug Copy Issue Report `$contentType` is of type `string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
27	13	$page = new ElementFinder((string) $html);
28
29	13	if ($affectedUrl) {
30		LinkConverter::convertUrlsToAbsolute($page, $affectedUrl);
31		}
32
33	13	return $page;
34		}
35
36		}

xparse / Parser

GitHub Access Token became invalid

Push — master ( 4a2eaa...5e2e05 )

ElementFinderFactory::getSupportedEncodings() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like