Skip to content

Commit

Permalink
Sometimes the succesful flag on the message payload is not set, in th…
Browse files Browse the repository at this point in the history
…ese situations, successful should be treated as true (#50) (#51)
  • Loading branch information
Alex Boyce authored May 14, 2019
1 parent 79f4443 commit 6f29f5b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/Bayeux/Message.php
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ class Message
* @JMS\Type("bool")
* @JMS\Exclude(if="context.getDirection() === 1")
*/
private $successful;
private $successful = true;

/**
* @var bool|null
Expand Down Expand Up @@ -326,7 +326,7 @@ public function setTimestamp(?\DateTimeImmutable $timestamp): Message
/**
* @return bool|null
*/
public function isSuccessful(): ?bool
public function isSuccessful(): bool
{
return $this->successful;
}
Expand All @@ -336,7 +336,7 @@ public function isSuccessful(): ?bool
*
* @return Message
*/
public function setSuccessful(?bool $successful): Message
public function setSuccessful(bool $successful): Message
{
$this->successful = $successful;

Expand Down

0 comments on commit 6f29f5b

Please sign in to comment.