Add Tagged VLAN, work at least with HPE Switch

[PR-gh]

Take in charge Vlan for switch HPE (and other switchs) with SNMP :
dot1qVlanStaticName, dot1qVlanStaticRowStatus, dot1qVlanCurrentEgressPorts and dot1qVlanCurrentUntaggedPorts )
Add Information if the Vlan is Tagged or not.
fusioninventory/fusioninventory.github.io#66
fusioninventory/fusioninventory-for-glpi#2852

[g-bougard]

I see mostly cosmetic issues here. By the way, I'm against introducing a new dependency on Math::BigInt because of just this feature. Please try to do without it.

[g-bougard]

I'm okay with last commits, but there's just a last minor change needed ;-)

[g-bougard]

Hi @PR-gh
I found finally some problems with this PR. Sometime your code fails to handle dot1qVlanStaticName, dot1qVlanStaticRowStatus, dot1qVlanCurrentEgressPorts and dot1qVlanCurrentUntaggedPorts in the right way. Then it simply removes VLAN information in that case. The problem is you made some wrong assumption about the suffix to use for dot1qVlanCurrentEgressPorts and dot1qVlanCurrentUntaggedPorts. The assumption works on HPE. You also make the assumption the port id is an ordered port number and in fact this is not always true with some other manufacturers.
I can fall back to use .1.0.8802.1.1.2.1 based method when no result is found by your method, but I still see few problems.
I'll push commit soon and I hope you'll be able to test them and validate they don't break anything in your case.

[abmteam]

Function _getVlans (in Hardware.pm) doesn't bring all results
#366

[g-bougard]

@PR-gh can't validate that b9c5c4b and dad2096 commits do not have any wrong side effect for you ?
@abmteam I checked your issue again. I think This PR plus the last 2 commits may fix your issue. Did you try it ?

[PR-gh]

Hi @g-bougard,
sorry to listen side effect of my PR for other switchs.

I try on many HPE switchs with dad2096 and no side effect, I have make a diff of inventory xml before/after, only difference it's ordered VLAN in VLANS.