node 'PlanetTeamSpeak\TeamSpeak3Framework\Node\Server' has no property named 'virtualserver_icon_id'

[MajorOli]

Description and goal

node 'PlanetTeamSpeak\TeamSpeak3Framework\Node\Server' has no property named 'virtualserver_icon_id'

if usage $v->getInfo(true,true) its activate the convert option at public function getInfo(bool $extend = true, bool $convert = false): array))

The error occurs when the foreach loop cant find _icon_id

} elseif ($key->endsWith('_icon_id')) {
    $info[$key->toString()] = $this->iconGetName($key)->filterDigits();

Thats a good case for a UnitTest

[MajorOli]

memo
at

public function iconGetName(string $key): StringHelper
    {
        $iconid = $this[$key];
        if (! is_int($iconid)) {
            $iconid = $iconid->toInt();
        }

        $iconid = $iconid < 0 ? pow(2, 32) - abs($iconid) : $iconid;

        return new StringHelper('/icon_'.$iconid);
    }

-> Cannot find declaration to go to at $iconid = $iconid->toInt();. Okay, thats the Problem

[MajorOli]

Explain:
filterDigits()return an object so this function is used in other functions. At code analysis i found inconsistent chaining when filterDigits() is used.

So i expect on an id an integer type. Also in my opinion its logical to convert to an int type during the Node class.