toInt() should only return int

[Sebbo94BY]

When the function is called to int, I would expect, that this function only returns int and not also float.

Relates to #191 and maybe should be in one commit.

[ronindesign]

Hmm, my intuition tells me this may be more complicated. There's seems to be a reason/scenario for when a float should be returned (i.e. greater than 32bit int implies string is actually a float value I believe, but I may misunderstand)

Due to the explicitely designed nature of this, I'm not certain the ts3server doesn't return float values in some cases... I'll need to revew this further. Maybe we can test if there is a case that results in an icon ID being a float (or greater than a 32bit Int)?

[Sebbo94BY]

Yeah, some more PHPUnit tests would be very helpful - also for future PHP upgrades and code changes.

I've never really checked these icon IDs, but as far as I know, those IDs are only integers, never floats. I've never seen any icon ID, which was a float. But no gurantee on that. 😅

[Sebbo94BY]

Ok, the function Node::iconGetName() currently returns for example 3525880661.0. Also see #191 for further information.

[Sebbo94BY]

@ronindesign how do we want to proceed with this topic?

[ronindesign]

Merging this and closing #190 as this should resolve.