Completed
Push — master ( 73536c...874498 )
by Joschi
03:03
created

Dom::createFromFile()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 12
Code Lines 6

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 6
CRAP Score 2

Importance

Changes 0
Metric Value
dl 0
loc 12
ccs 6
cts 6
cp 1
rs 9.4285
c 0
b 0
f 0
cc 2
eloc 6
nc 2
nop 1
crap 2
1
<?php
2
3
/**
4
 * dom-factory
5
 *
6
 * @category Jkphl
7
 * @package Jkphl\Domfactory
8
 * @subpackage Jkphl\Domfactory\Ports
9
 * @author Joschi Kuphal <[email protected]> / @jkphl
10
 * @copyright Copyright © 2017 Joschi Kuphal <[email protected]> / @jkphl
11
 * @license http://opensource.org/licenses/MIT The MIT License (MIT)
12
 */
13
14
/***********************************************************************************
15
 *  The MIT License (MIT)
16
 *
17
 *  Copyright © 2017 Joschi Kuphal <[email protected]> / @jkphl
18
 *
19
 *  Permission is hereby granted, free of charge, to any person obtaining a copy of
20
 *  this software and associated documentation files (the "Software"), to deal in
21
 *  the Software without restriction, including without limitation the rights to
22
 *  use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
23
 *  the Software, and to permit persons to whom the Software is furnished to do so,
24
 *  subject to the following conditions:
25
 *
26
 *  The above copyright notice and this permission notice shall be included in all
27
 *  copies or substantial portions of the Software.
28
 *
29
 *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
30
 *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
31
 *  FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
32
 *  COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
33
 *  IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
34
 *  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
35
 ***********************************************************************************/
36
37
namespace Jkphl\Domfactory\Ports;
38
39
use Jkphl\Domfactory\Infrastructure\Dom as InternalDom;
40
41
/**
42
 * DOM factory
43
 *
44
 * @package Jkphl\Domfactory
45
 * @subpackage Jkphl\Domfactory\Ports
46
 */
47
class Dom extends InternalDom
48
{
49
    /**
50
     * Create a DOM document from a URI
51
     *
52
     * @param string $url HTTP / HTTPS URL
53
     * @return \DOMDocument DOM document
54
     * @api
55
     */
56 1
    public static function createFromUri($url)
57
    {
58 1
        return extension_loaded('curl') ? self::createViaHttpClient($url) : self::createViaStreamWrapper($url);
59
    }
60
61
    /**
62
     * Create a DOM document from a string
63
     *
64
     * @param string $str String
65
     * @return \DOMDocument DOM document
66
     * @api
67
     */
68 3
    public static function createFromString($str)
69
    {
70 3
        return parent::createFromString($str);
71
    }
72
73
    /**
74
     * Create a DOM document from a file
75
     *
76
     * @param string $file File
77
     * @return \DOMDocument DOM document
78
     * @throws InvalidArgumentException If the file is not readable
79
     * @api
80
     */
81 2
    public static function createFromFile($file)
82
    {
83
        // If the file is not readable
84 1
        if (!is_readable($file)) {
85 1
            throw new InvalidArgumentException(
86 1
                sprintf(InvalidArgumentException::INVALID_FILE_STR, $file),
87
                InvalidArgumentException::INVALID_FILE
88 1
            );
89
        }
90
91 2
        return parent::createFromString(file_get_contents($file));
0 ignored issues
show
Comprehensibility Bug introduced by Joschi Kuphal
It seems like you call parent on a different method (createFromString() instead of createFromFile()). Are you sure this is correct? If so, you might want to change this to $this->createFromString().

This check looks for a call to a parent method whose name is different than the method from which it is called.

Consider the following code:

class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}

The getFirstName() method in the Son calls the wrong method in the parent class.

Loading history...
92
    }
93
}
94