Skip to content

Assignee_id must be nullable#112

Merged
LKaemmerling merged 2 commits intoLKDevelopment:masterfrom
tanji:patch-1
Dec 16, 2024
Merged

Assignee_id must be nullable#112
LKaemmerling merged 2 commits intoLKDevelopment:masterfrom
tanji:patch-1

Conversation

@tanji
Copy link
Copy Markdown
Contributor

@tanji tanji commented Jan 3, 2024

A primary IP can be unassigned and therefore assignee_id will be null, which led to an exception.

<b>Fatal error</b>:  Uncaught TypeError: Argument 12 passed to LKDev\HetznerCloud\Models\PrimaryIps\PrimaryIp::__construct() must be of the type integer, null given, called in /opt/ops-ip-lister/vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/PrimaryIps/PrimaryIp.php on line 166 and defined in /opt/ops-ip-lister/vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/PrimaryIps/PrimaryIp.php:96
Stack trace:
#0 /opt/ops-ip-lister/vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/PrimaryIps/PrimaryIp.php(166): LKDev\HetznerCloud\Models\PrimaryIps\PrimaryIp-&gt;__construct(0000, 'base-image', '2023-12-23T02:0...', '1.1.1.1', 'ipv4', Array, false, Object(LKDev\HetznerCloud\Models\Protection), Array, Object(LKDev\HetznerCloud\Models\Datacenters\Datacenter), 'server', NULL, false)
#1 /opt/ops-ip-lister/vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/PrimaryIps/PrimaryIps.php(163): LKDev\HetznerCloud\Models\PrimaryIps\PrimaryIp::parse(Object(stdClass))
#2 [internal function]: LKDev\HetznerCloud\Models\PrimaryIps\Prim in <b>/opt/ops-ip-lister/vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/PrimaryIps/PrimaryIp.php</b> on line <b>96</b><br />

A primary IP can be unassigned and therefore assignee_id will be null, which led to an exception.
@DGINXREAL
Copy link
Copy Markdown

DGINXREAL commented Dec 14, 2024

Please merge this.... @LKaemmerling

Comment thread src/Models/PrimaryIps/PrimaryIp.php Outdated
@LKaemmerling LKaemmerling merged commit c92401f into LKDevelopment:master Dec 16, 2024
@LKaemmerling
Copy link
Copy Markdown
Collaborator

Thank you @tanji (& @DGINXREAL)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants