Skip to content

Commit

Permalink
Merge pull request #324 from rogerb87/bugfix/php8-parameter-order-dep…
Browse files Browse the repository at this point in the history
…recation

Fix Deprecated Notice for PHP8
  • Loading branch information
goetas authored Dec 18, 2021
2 parents ddbba22 + a1acb92 commit 2f37b88
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/Serializer/ExclusionManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,15 +41,15 @@ public function shouldSkipEmbedded(object $object, Relation $relation, Serializa
return $this->shouldSkipRelation($relation, $context);
}

return $this->shouldSkip($relation, $relation->getEmbedded()->getExclusion(), $context);
return $this->shouldSkip($relation, $context, $relation->getEmbedded()->getExclusion());
}

private function shouldSkipRelation(Relation $relation, SerializationContext $context): bool
{
return $this->shouldSkip($relation, $relation->getExclusion(), $context);
return $this->shouldSkip($relation, $context, $relation->getExclusion());
}

private function shouldSkip(Relation $relation, ?Exclusion $exclusion = null, SerializationContext $context): bool
private function shouldSkip(Relation $relation, SerializationContext $context, ?Exclusion $exclusion = null): bool
{
$propertyMetadata = new RelationPropertyMetadata($exclusion, $relation);
if ($context->getExclusionStrategy()) {
Expand Down

0 comments on commit 2f37b88

Please sign in to comment.