protobuf icon indicating copy to clipboard operation
protobuf copied to clipboard

[php][ext] Support NULL values conversion

Open FabioBatSilva opened this issue 1 year ago • 6 comments

  • Handle NULL values in the PHP extension's conversion functions.
  • Tests that verify the behavior of converting supported types.

Should fix : https://github.com/protocolbuffers/protobuf/issues/20000

FabioBatSilva avatar Apr 11 '25 15:04 FabioBatSilva

I just left a comment on https://github.com/protocolbuffers/protobuf/issues/20000, I think we generally don't want to support $msg->setFoo(null).

haberman avatar May 12 '25 05:05 haberman

We triage inactive PRs and issues in order to make it easier to find active work. If this PR should remain active, please add a comment.

This PR is labeled inactive because the last activity was over 90 days ago. This PR will be closed and archived after 14 additional days without activity.

github-actions[bot] avatar Sep 05 '25 10:09 github-actions[bot]

Please don't dismiss it. PR is still relevant, waiting for the maintainers.

FabioBatSilva avatar Sep 08 '25 14:09 FabioBatSilva

I just left a comment on #20000, I think we generally don't want to support $msg->setFoo(null).

https://github.com/protocolbuffers/protobuf/pull/20649#issuecomment-2873223983

FabioBatSilva avatar Sep 08 '25 14:09 FabioBatSilva

We triage inactive PRs and issues in order to make it easier to find active work. If this PR should remain active, please add a comment.

This PR is labeled inactive because the last activity was over 90 days ago. This PR will be closed and archived after 14 additional days without activity.

github-actions[bot] avatar Jan 02 '26 10:01 github-actions[bot]

PR is still relevant, waiting for the maintainers. :disappointed:

FabioBatSilva avatar Jan 02 '26 17:01 FabioBatSilva