Skip to content

Commit 43be3c5

Browse files
authored
Merge pull request #230 from phpcr/fix-deprecations
fix some php 8.5 deprecations
2 parents c0ad706 + d9ffce2 commit 43be3c5

File tree

5 files changed

+34
-10
lines changed

5 files changed

+34
-10
lines changed

src/PHPCR/Util/CND/Scanner/GenericToken.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
* @license http://opensource.org/licenses/MIT MIT License
1010
* @author Daniel Barsotti <daniel.barsotti@liip.ch>
1111
*/
12-
class GenericToken extends Token
12+
class GenericToken extends Token implements \Stringable
1313
{
1414
public const TK_WHITESPACE = 0;
1515
public const TK_NEWLINE = 1;

src/PHPCR/Util/CND/Scanner/Token.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
* @license http://opensource.org/licenses/MIT MIT License
1414
* @author Daniel Barsotti <daniel.barsotti@liip.ch>
1515
*/
16-
class Token
16+
class Token implements \Stringable
1717
{
1818
public function __construct(
1919
private int $type = 0,

tests/PHPCR/Tests/Util/CND/Reader/BufferReaderTest.php

Lines changed: 24 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,22 @@ public function testConstruct(): void
1616

1717
$reflection = new \ReflectionClass($reader);
1818
$bufferProperty = $reflection->getProperty('buffer');
19-
$bufferProperty->setAccessible(true);
19+
// remove when we drop PHP 8.0 support
20+
if (PHP_VERSION_ID < 80100) {
21+
$bufferProperty->setAccessible(true);
22+
}
2023
$this->assertSame(str_replace("\r\n", "\n", $buffer).$reader->getEofMarker(), $bufferProperty->getValue($reader));
2124
$startPos = $reflection->getProperty('startPos');
22-
$startPos->setAccessible(true);
25+
// remove when we drop PHP 8.0 support
26+
if (PHP_VERSION_ID < 80100) {
27+
$startPos->setAccessible(true);
28+
}
2329
$this->assertSame(0, $startPos->getValue($reader));
2430
$forwardPos = $reflection->getProperty('forwardPos');
25-
$forwardPos->setAccessible(true);
31+
// remove when we drop PHP 8.0 support
32+
if (PHP_VERSION_ID < 80100) {
33+
$forwardPos->setAccessible(true);
34+
}
2635
$this->assertSame(0, $forwardPos->getValue($reader));
2736

2837
$this->assertEquals(1, $reader->getCurrentLine());
@@ -100,13 +109,22 @@ public function testConstructEmptyString(): void
100109

101110
$reflection = new \ReflectionClass($reader);
102111
$buffer = $reflection->getProperty('buffer');
103-
$buffer->setAccessible(true);
112+
// remove when we drop PHP 8.0 support
113+
if (PHP_VERSION_ID < 80100) {
114+
$buffer->setAccessible(true);
115+
}
104116
$this->assertSame($reader->getEofMarker(), $buffer->getValue($reader));
105117
$startPos = $reflection->getProperty('startPos');
106-
$startPos->setAccessible(true);
118+
// remove when we drop PHP 8.0 support
119+
if (PHP_VERSION_ID < 80100) {
120+
$startPos->setAccessible(true);
121+
}
107122
$this->assertSame(0, $startPos->getValue($reader));
108123
$forwardPos = $reflection->getProperty('forwardPos');
109-
$forwardPos->setAccessible(true);
124+
// remove when we drop PHP 8.0 support
125+
if (PHP_VERSION_ID < 80100) {
126+
$forwardPos->setAccessible(true);
127+
}
110128
$this->assertSame(0, $forwardPos->getValue($reader));
111129

112130
$this->assertEquals(1, $reader->getCurrentLine());

tests/PHPCR/Tests/Util/CND/Scanner/TokenQueueTest.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,10 @@ public function testAdd(): void
5454
$queue = new TokenQueue();
5555
$reflection = new \ReflectionClass($queue);
5656
$tokens = $reflection->getProperty('tokens');
57-
$tokens->setAccessible(true);
57+
// remove when we drop PHP 8.0 support
58+
if (PHP_VERSION_ID < 80100) {
59+
$tokens->setAccessible(true);
60+
}
5861
$this->assertSame([], $tokens->getValue($queue));
5962

6063
$queue->add($this->token0);

tests/PHPCR/Tests/Util/Console/Helper/PhpcrConsoleDumperHelperTest.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,10 @@ public function testGetTreeWalker(array $options): void
5656

5757
$reflection = new \ReflectionClass($tw);
5858
$propVisitorProp = $reflection->getProperty('propertyVisitor');
59-
$propVisitorProp->setAccessible(true);
59+
// remove when we drop PHP 8.0 support
60+
if (PHP_VERSION_ID < 80100) {
61+
$propVisitorProp->setAccessible(true);
62+
}
6063
$propVisitor = $propVisitorProp->getValue($tw);
6164

6265
if (true === $options['show_props']) {

0 commit comments

Comments
 (0)