Fix: Allow isPublishMessageWithNullCharacter to receive messages as null (#162)

DavidCova · web-flow · commit 92e348db89c8 · 2023-08-10T16:18:24.000Z
diff --git a/src/MessageProcessors/Mqtt311MessageProcessor.php b/src/MessageProcessors/Mqtt311MessageProcessor.php
@@ -66,10 +66,10 @@ protected function parseAndValidateSubscribeAcknowledgementMessage(string $data)
     /**
      * Determines if the given message is a PUBLISH message and contains the unicode null character U+0000.
      *
-     * @param Message $message
+     * @param Message|null $message
      * @return bool
      */
-    private function isPublishMessageWithNullCharacter(Message $message): bool
+    private function isPublishMessageWithNullCharacter(?Message $message): bool
     {
         return $message !== null
             && $message->getType()->equals(MessageType::PUBLISH())

Original file line number	Diff line number	Diff line change
`@@ -66,10 +66,10 @@ protected function parseAndValidateSubscribeAcknowledgementMessage(string $data)`
`66`	`66`	`/**`
`67`	`67`	`* Determines if the given message is a PUBLISH message and contains the unicode null character U+0000.`
`68`	`68`	`*`
`69`		`- * @param Message $message`
	`69`	`+ * @param Message\|null $message`
`70`	`70`	`* @return bool`
`71`	`71`	`*/`
`72`		`- private function isPublishMessageWithNullCharacter(Message $message): bool`
	`72`	`+ private function isPublishMessageWithNullCharacter(?Message $message): bool`
`73`	`73`	`{`
`74`	`74`	`return $message !== null`
`75`	`75`	`&& $message->getType()->equals(MessageType::PUBLISH())`