Test Failed
Pull Request — master (#345)
by
unknown
03:49 queued 01:44
created

ParserSub   A

Complexity

Total Complexity 1

Size/Duplication

Total Lines 3
Duplicated Lines 0 %

Importance

Changes 1
Bugs 1 Features 1
Metric Value
eloc 2
c 1
b 1
f 1
dl 0
loc 3
rs 10
wmc 1

1 Method

Rating   Name   Duplication   Size   Complexity  
A exposedParseObject() 0 2 1
1
<?php
2
3
/**
4
 * @file This file is part of the PdfParser library.
5
 *
6
 * @author  Konrad Abicht <[email protected]>
7
 * @date    2020-06-01
8
 *
9
 * @author  Sébastien MALOT <[email protected]>
10
 * @date    2017-01-03
11
 *
12
 * @license LGPLv3
13
 * @url     <https://github.com/smalot/pdfparser>
14
 *
15
 *  PdfParser is a pdf library written in PHP, extraction oriented.
16
 *  Copyright (C) 2017 - Sébastien MALOT <[email protected]>
17
 *
18
 *  This program is free software: you can redistribute it and/or modify
19
 *  it under the terms of the GNU Lesser General Public License as published by
20
 *  the Free Software Foundation, either version 3 of the License, or
21
 *  (at your option) any later version.
22
 *
23
 *  This program is distributed in the hope that it will be useful,
24
 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
25
 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
26
 *  GNU Lesser General Public License for more details.
27
 *
28
 *  You should have received a copy of the GNU Lesser General Public License
29
 *  along with this program.
30
 *  If not, see <http://www.pdfparser.org/sites/default/LICENSE.txt>.
31
 */
32
33
namespace Tests\Smalot\PdfParser\Integration;
34
35
use Exception;
36
use Smalot\PdfParser\Parser;
37
use Smalot\PdfParser\Document;
38
use Smalot\PdfParser\XObject\Image;
39
use Tests\Smalot\PdfParser\TestCase;
40
41
class ParserTest extends TestCase
42
{
43
    public function setUp()
44
    {
45
        parent::setUp();
46
47
        $this->fixture = new Parser();
48
    }
49
50
    public function testParseFile()
51
    {
52
        $directory = $this->rootDir.'/samples/bugs';
53
54
        if (is_dir($directory)) {
55
            $files = scandir($directory);
56
57
            foreach ($files as $file) {
58
                if (preg_match('/^.*\.pdf$/i', $file)) {
59
                    try {
60
                        $document = $this->fixture->parseFile($directory.'/'.$file);
61
                        $pages = $document->getPages();
62
                        $this->assertTrue(0 < \count($pages));
63
64
                        foreach ($pages as $page) {
65
                            $content = $page->getText();
66
                            $this->assertTrue(0 < \strlen($content));
67
                        }
68
                    } catch (Exception $e) {
69
                        if (
70
                            'Secured pdf file are currently not supported.' !== $e->getMessage()
71
                            && 0 != strpos($e->getMessage(), 'TCPDF_PARSER')
72
                        ) {
73
                            throw $e;
74
                        }
75
                    }
76
                }
77
            }
78
        }
79
    }
80
81
    public function testIssue19()
82
    {
83
        ini_set('display_errors', 1);
84
        $fixture = new ParserSub();
85
        $structure = [
86
            [
87
                '<<',
88
                [
89
                    [
90
                        '/',
91
                        'Type',
92
                        7735,
93
                    ],
94
                    [
95
                        '/',
96
                        'ObjStm',
97
                        7742,
98
                    ]
99
                ]
100
            ],
101
            [
102
                'stream',
103
                '',
104
                7804,
105
                [
106
                    "17\n0",
107
                    []
108
                ]
109
            ]
110
        ];
111
        $document = new Document();
112
        $this->catchAllErrors();
113
        $fixture->exposedParseObject('19_0', $structure, $document);
114
115
        $this->expectNotToPerformAssertions();
116
    }
117
118
    /**
119
     * Test that issue related pdf can now be parsed
120
     *
121
     * @see https://github.com/smalot/pdfparser/issues/267
122
     */
123
    public function testIssue267()
124
    {
125
        $filename = $this->rootDir.'/samples/bugs/Issue267_array_access_on_int.pdf';
126
127
        $document = $this->fixture->parseFile($filename);
128
129
        $this->assertEquals(Image::class, \get_class($document->getObjectById('128_0')));
130
        $this->assertStringContainsString('4 von 4', $document->getText());
131
    }
132
}
133
134
class ParserSub extends Parser {
135
    public function exposedParseObject($id, $structure, $document) {
136
        return $this->parseObject($id, $structure, $document);
0 ignored issues
show
Bug introduced by
Are you sure the usage of $this->parseObject($id, $structure, $document) targeting Smalot\PdfParser\Parser::parseObject() seems to always return null.

This check looks for function or method calls that always return null and whose return value is used.

class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

The method getObject() can return nothing but null, so it makes no sense to use the return value.

The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes.

Loading history...
137
    }
138
}
139