I agree that making database changes should be avoided unless needed.
My argument for adding that as it seems to be universal to any device that supports the LLDP-MIB.
As an example this is what a cisco says:
lldpLocPortId[1] = “Fa0/1”
lldpLocPortId[2] = “Fa0/2”
lldpLocPortId[3] = “Fa0/3”
lldpLocPortId[4] = “Fa0/4”
Now I agree that you should be able to get the mapping with ifDescr or ifName, but this seems to be more universal and should be able to make the mapping process more straight forward.
As an example of someone else having the same problem:
I would think this method would fix their issue as well.
I have started my own fork to implement it.