ERROR while using namesilo WHMCS module

[corozanu]

WHMCS 8.11.2
PHP 8.1

TypeError: count(): Argument #1 ($value) must be of type Countable|array, null given in /var/www/vhosts/domain.com/subdomains/subdomain.domain.com/vendor/whmcs/whmcs-foundation/lib/Domain/Registrar/Domain.php:0
Stack trace:
#0 /var/www/vhosts/domain.com/subdomains/subdomain.domain.com/vendor/whmcs/whmcs-foundation/lib/Domain/Registrar/Domain.php(0): count()
#1 /var/www/vhosts/domain.com/subdomains/subdomain.domain.com/vendor/whmcs/whmcs-foundation/lib/Domains.php(0): WHMCS\Domain\Registrar\Domain->hasNameservers()
#2 /var/www/vhosts/domain.com/subdomains/subdomain.domain.com/vbit/clientsdomains.php(0): WHMCS\Domains->getDomainInformation()
#3 {main}

[MarkCanada]

Same problem here - but only with .CA domains - one is in transfer, the other one I wanted to transfer through WHMCS.
.COM work well.

[MarkCanada]

The domain transfer completed and it shows up now in WHMCS without error.

It seems that maybe this problem comes up if the domain is not available in NameSilo yet?
Like during a transfer or when we want to switch it to NameSilo to start a transfer.

So far I have only tested it with .CA

[jstebenne]

I have the same problem with .COM domains that are being transferred

[Sufiyanshaikh123]

Try this fix:

File name: namesilo.php

$response = [];
$nameservers = $domain->nameservers;
$i = 0;
foreach ($nameservers->nameserver as $ns) {
$response['nameservers']['ns' . ++$i] = (string)$ns;

Replace with:

$response = [];
$response['nameservers'] = [];
$nameservers = $domain->nameservers;
if ($nameservers && isset($nameservers->nameserver)) {
$i = 0;
foreach ($nameservers->nameserver as $ns) {
$response['nameservers']['ns' . ++$i] = (string)$ns;
}