Skip to content

Commit

Permalink
Merge pull request #73 from cultuurnet/feature/PUBLIQ-952
Browse files Browse the repository at this point in the history
Lock jms/serializer at 3.24.0 + Rewind body before fetching contents
  • Loading branch information
zuuperman authored Nov 24, 2023
2 parents c99ed34 + 55cc934 commit 36d82ba
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
"require": {
"php": "^8",
"guzzlehttp/guzzle": "^6.5.8|^7.4.5",
"jms/serializer": "^3.28",
"jms/serializer": "3.24.0",
"simple-bus/jms-serializer-bridge": "^6.3.1"
},
"require-dev": {
Expand Down
5 changes: 4 additions & 1 deletion src/SearchClient.php
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,9 @@ private function search(SearchQueryInterface $searchQuery, $type): PagedCollecti

$result = $this->client->request('GET', $type, $options);

return $this->serializer->deserialize($result->getBody()->getContents());
$body = $result->getBody();
$body->rewind();

return $this->serializer->deserialize($body->getContents());
}
}

0 comments on commit 36d82ba

Please sign in to comment.