Skip to content

Commit 3d8bb7a

Browse files
committed
Cast to correct datatypes
To avoid strict type PHP fatal errors to occur
1 parent e284220 commit 3d8bb7a

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

src/Entity/Item.php

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,8 @@ public function getAvailable(): bool
8484
public function unserialize(?array $data): self
8585
{
8686
if (!empty($data['id'])) {
87-
$this->setSequentialId($data['id']);
87+
$sequentialId = (int)$data['id'];
88+
$this->setSequentialId($sequentialId);
8889
}
8990

9091
if (!empty($data['item_uuid'])) {
@@ -96,13 +97,15 @@ public function unserialize(?array $data): self
9697
}
9798

9899
if (!empty($data['price'])) {
99-
$this->setPrice($data['price']);
100+
$price = (float)$data['price'];
101+
$this->setPrice($price);
100102
}
101103

102104
if (!empty($data['available'])) {
103-
$this->setAvailable($data['available']);
105+
$isAvailable = (bool)$data['available'];
106+
$this->setAvailable($isAvailable);
104107
}
105108

106109
return $this;
107110
}
108-
}
111+
}

src/Entity/User.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,8 @@ public function getCreationDate(): ?string
125125
public function unserialize(?array $data): self
126126
{
127127
if (!empty($data['id'])) {
128-
$this->setSequentialId($data['id']);
128+
$sequentialId = (int)$data['id'];
129+
$this->setSequentialId($sequentialId);
129130
}
130131

131132
if (!empty($data['user_uuid'])) {

0 commit comments

Comments
 (0)