Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit c61ce20

Browse files
dbudbu
committedMay 10, 2023
adjust code to be compatible with psr/http-message 2.0
1 parent 250fc9f commit c61ce20

26 files changed

+150
-434
lines changed
 

‎.github/workflows/static.yml

-29
Original file line numberDiff line numberDiff line change
@@ -22,35 +22,6 @@ jobs:
2222
with:
2323
args: analyze --no-progress
2424

25-
phpstan_psr-2:
26-
name: PHPStan psr/http-message 2
27-
runs-on: ubuntu-latest
28-
29-
steps:
30-
- name: Checkout code
31-
uses: actions/checkout@v3
32-
33-
- name: Setup PHP
34-
uses: shivammathur/setup-php@v2
35-
with:
36-
php-version: ${{ matrix.php }}
37-
tools: composer
38-
coverage: none
39-
40-
- name: Adjust dependencies
41-
run: |
42-
rm src/MessageFactory/SlimMessageFactory.php src/StreamFactory/SlimStreamFactory.php src/UriFactory/SlimUriFactory.php spec/MessageFactory/SlimMessageFactorySpec.php spec/StreamFactory/SlimStreamFactorySpec.php spec/UriFactory/SlimUriFactorySpec.php
43-
composer remove --dev "slim/slim" --no-interaction --no-update
44-
composer require "psr/http-message:^2.0" --no-interaction --no-update
45-
composer update --prefer-dist --prefer-stable --no-interaction --no-progress
46-
47-
- name: PHPStan
48-
uses: docker://oskarstark/phpstan-ga
49-
env:
50-
REQUIRE_DEV: true
51-
with:
52-
args: analyze --no-progress
53-
5425
php-cs-fixer:
5526
name: PHP-CS-Fixer
5627
runs-on: ubuntu-latest

‎.github/workflows/tests.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
runs-on: ubuntu-latest
1313
strategy:
1414
matrix:
15-
php: ['7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
15+
php: ['7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
1616

1717
steps:
1818
- name: Checkout code
@@ -61,7 +61,7 @@ jobs:
6161
runs-on: ubuntu-latest
6262
strategy:
6363
matrix:
64-
php: ['7.1', '7.2', '7.3', '7.4']
64+
php: ['7.2', '7.3', '7.4']
6565

6666
steps:
6767
- name: Checkout code

‎CHANGELOG.md

+8-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,14 @@ All notable changes to this project will be documented in this file.
66
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
77
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
88

9-
## [1.14.0] - 2023-04-14
9+
## [1.15.0] - 2023-05-10
10+
11+
- Actually make compatible with PSR-7 1.1 and 2.0
12+
- Drop support for PHP 7.1
13+
14+
## [1.14.0] - 2023-04-14 (broken)
15+
16+
**This release is not actually compatible with PSR-7 1.1 or 2.0**
1017

1118
- Allow installation with http-message (PSR-7) version 2 in addition to version 1.
1219
- Support for PHP 8.2

‎composer.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@
1515
}
1616
],
1717
"require": {
18-
"php": "^7.1 || ^8.0",
18+
"php": "^7.2 || ^8.0",
1919
"clue/stream-filter": "^1.5",
2020
"php-http/message-factory": "^1.0.2",
21-
"psr/http-message": "^1.0 || ^2.0"
21+
"psr/http-message": "^1.1 || ^2.0"
2222
},
2323
"provide": {
2424
"php-http/message-factory-implementation": "1.0"
@@ -28,7 +28,7 @@
2828
"ergebnis/composer-normalize": "^2.6",
2929
"guzzlehttp/psr7": "^1.0 || ^2.0",
3030
"phpspec/phpspec": "^5.1 || ^6.3 || ^7.1",
31-
"slim/slim": "^3.0 || ^4.0",
31+
"slim/slim": "^3.0",
3232
"laminas/laminas-diactoros": "^2.0 || ^3.0"
3333
},
3434
"suggest": {

‎phpstan-baseline.neon

-65
Original file line numberDiff line numberDiff line change
@@ -115,26 +115,6 @@ parameters:
115115
count: 1
116116
path: src/CookieUtil.php
117117

118-
-
119-
message: "#^Method Http\\\\Message\\\\Encoding\\\\ChunkStream\\:\\:fill\\(\\) has no return type specified\\.$#"
120-
count: 1
121-
path: src/Encoding/ChunkStream.php
122-
123-
-
124-
message: "#^Method Http\\\\Message\\\\Encoding\\\\FilteredStream\\:\\:fill\\(\\) has no return type specified\\.$#"
125-
count: 1
126-
path: src/Encoding/FilteredStream.php
127-
128-
-
129-
message: "#^Method Http\\\\Message\\\\Encoding\\\\FilteredStream\\:\\:rewind\\(\\) has no return type specified\\.$#"
130-
count: 1
131-
path: src/Encoding/FilteredStream.php
132-
133-
-
134-
message: "#^Method Http\\\\Message\\\\Encoding\\\\FilteredStream\\:\\:seek\\(\\) has no return type specified\\.$#"
135-
count: 1
136-
path: src/Encoding/FilteredStream.php
137-
138118
-
139119
message: "#^Method Http\\\\Message\\\\MessageFactory\\\\DiactorosMessageFactory\\:\\:createRequest\\(\\) has parameter \\$headers with no value type specified in iterable type array\\.$#"
140120
count: 1
@@ -180,26 +160,11 @@ parameters:
180160
count: 1
181161
path: src/RequestMatcher/RequestMatcher.php
182162

183-
-
184-
message: "#^Property Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:\\$size \\(int\\) does not accept int\\|null\\.$#"
185-
count: 1
186-
path: src/Stream/BufferedStream.php
187-
188163
-
189164
message: "#^Property Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:\\$resource \\(resource\\) does not accept resource\\|false\\.$#"
190165
count: 2
191166
path: src/Stream/BufferedStream.php
192167

193-
-
194-
message: "#^Dead catch \\- Exception is already caught#"
195-
count: 1
196-
path: src/Stream/BufferedStream.php
197-
198-
-
199-
message: "#^Method Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:detach\\(\\) should return resource\\|null but empty return statement found\\.$#"
200-
count: 1
201-
path: src/Stream/BufferedStream.php
202-
203168
-
204169
message: "#^Property Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:\\$stream \\(Psr\\\\Http\\\\Message\\\\StreamInterface\\) does not accept null\\.$#"
205170
count: 1
@@ -210,36 +175,6 @@ parameters:
210175
count: 1
211176
path: src/Stream/BufferedStream.php
212177

213-
-
214-
message: "#^Method Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:getSize\\(\\) should return int\\|null but empty return statement found\\.$#"
215-
count: 1
216-
path: src/Stream/BufferedStream.php
217-
218-
-
219-
message: "#^Method Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:tell\\(\\) should return int but returns int\\|false\\.$#"
220-
count: 1
221-
path: src/Stream/BufferedStream.php
222-
223-
-
224-
message: "#^Method Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:seek\\(\\) has no return type specified\\.$#"
225-
count: 1
226-
path: src/Stream/BufferedStream.php
227-
228-
-
229-
message: "#^Method Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:rewind\\(\\) has no return type specified\\.$#"
230-
count: 1
231-
path: src/Stream/BufferedStream.php
232-
233-
-
234-
message: "#^Parameter \\#1 \\$string of function strlen expects string, string\\|false given\\.$#"
235-
count: 1
236-
path: src/Stream/BufferedStream.php
237-
238-
-
239-
message: "#^Method Http\\\\Message\\\\Stream\\\\BufferedStream\\:\\:read\\(\\) should return string but returns string\\|false\\.$#"
240-
count: 1
241-
path: src/Stream/BufferedStream.php
242-
243178
-
244179
message: "#^Parameter \\#1 \\$stream of class Slim\\\\Http\\\\Stream constructor expects resource, resource\\|false given\\.$#"
245180
count: 1

‎spec/Decorator/MessageDecoratorSpec.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -69,9 +69,9 @@ public function it_can_check_a_header(MessageInterface $message)
6969

7070
public function it_has_a_header(MessageInterface $message)
7171
{
72-
$message->getHeader('Content-Type')->willReturn('application/xml');
72+
$message->getHeader('Content-Type')->willReturn(['application/xml']);
7373

74-
$this->getHeader('Content-Type')->shouldReturn('application/xml');
74+
$this->getHeader('Content-Type')->shouldReturn(['application/xml']);
7575
}
7676

7777
public function it_has_a_header_line(MessageInterface $message)

‎spec/Decorator/StreamDecoratorSpec.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ public function it_checks_whether_the_stream_is_writable(StreamInterface $stream
9595

9696
public function it_writes_to_the_stream(StreamInterface $stream)
9797
{
98-
$stream->write('body')->shouldBeCalled();
98+
$stream->write('body')->shouldBeCalled()->willReturn(4);
9999

100100
$this->write('body');
101101
}

‎spec/Encoding/FilteredStreamStubSpec.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -57,12 +57,12 @@ public function it_does_not_know_the_content_size()
5757

5858
class FilteredStreamStub extends FilteredStream
5959
{
60-
protected function readFilter()
60+
protected function readFilter(): string
6161
{
6262
return 'convert.base64-encode';
6363
}
6464

65-
protected function writeFilter()
65+
protected function writeFilter(): string
6666
{
6767
return 'convert.base64-encode';
6868
}

‎spec/Encoding/MemoryStream.php

+15-15
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,12 @@ public function __construct($body = '', $chunkSize = 8192)
2828
$this->chunkSize = $chunkSize;
2929
}
3030

31-
public function __toString()
31+
public function __toString(): string
3232
{
3333
return $this->getContents();
3434
}
3535

36-
public function close()
36+
public function close(): void
3737
{
3838
fclose($this->resource);
3939
}
@@ -46,64 +46,64 @@ public function detach()
4646
return $resource;
4747
}
4848

49-
public function getSize()
49+
public function getSize(): int
5050
{
5151
return $this->size;
5252
}
5353

54-
public function tell()
54+
public function tell(): int
5555
{
5656
return ftell($this->resource);
5757
}
5858

59-
public function eof()
59+
public function eof(): bool
6060
{
6161
return feof($this->resource);
6262
}
6363

64-
public function isSeekable()
64+
public function isSeekable(): bool
6565
{
6666
return true;
6767
}
6868

69-
public function seek($offset, $whence = SEEK_SET)
69+
public function seek(int $offset, int $whence = SEEK_SET): void
7070
{
7171
fseek($this->resource, $offset, $whence);
7272
}
7373

74-
public function rewind()
74+
public function rewind(): void
7575
{
7676
$this->seek(0);
7777
}
7878

79-
public function isWritable()
79+
public function isWritable(): bool
8080
{
8181
return true;
8282
}
8383

84-
public function write($string)
84+
public function write(string $string): int
8585
{
86-
fwrite($this->resource, $string);
86+
return fwrite($this->resource, $string);
8787
}
8888

89-
public function isReadable()
89+
public function isReadable(): bool
9090
{
9191
return true;
9292
}
9393

94-
public function read($length)
94+
public function read(int $length): string
9595
{
9696
return fread($this->resource, min($this->chunkSize, $length));
9797
}
9898

99-
public function getContents()
99+
public function getContents(): string
100100
{
101101
$this->rewind();
102102

103103
return $this->read($this->size);
104104
}
105105

106-
public function getMetadata($key = null)
106+
public function getMetadata(string $key = null)
107107
{
108108
$metadata = stream_get_meta_data($this->resource);
109109

0 commit comments

Comments
 (0)
Please sign in to comment.