Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

add some missing null types in phpdoc #1016

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

julienbornstein
Copy link

@julienbornstein julienbornstein commented Feb 26, 2021

  • In ServiceBusRestProxy: fix phpdoc @return : receiveMessage() receiveQueueMessage() and receiveSubscriptionMessage() can return null.

  • In BrokerProperties, by default, when initiate an instance of the class, all properties are unset, so they are null. Fixing Phpdoc for properties, getters and setters
  • In BrokeredMessage, also some phpdoc type fixes

@ghost
Copy link

ghost commented Feb 26, 2021

CLA assistant check
All CLA requirements met.

@julienbornstein julienbornstein changed the title set return type nullable for ServiceBusRestProxy::receiveMessage() add some missing null types in phpdoc Feb 26, 2021
- In ServiceBusRestProxy/IServiceBus, fix phpdoc @return : receiveMessage() receiveQueueMessage() and receiveSubscriptionMessage() can return null
- In BrokerProperties, by default, when initiate an instance of the class, all properties are unset, so they are null. Fixing Phpdoc for properties, getters and setters
- In BrokeredMessage, also some phpdoc type fixes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants