Incorrect Virtue Level calculation about servuo

[dominicrusso]

In talking to one other developer, they suggested the intent may have been to make it 0, 4k, 10k, Max. if that's the case, we should probably just make it a static point instead of a calculated midpoint.

[dominicrusso]

Also in a related note, it looks like the Virtue gump uses hardcoded values instead of GetMaxAmount(...)

private int GetHueFor(int index)
        {
            if (m_Beheld.Virtues.GetValue(index) == 0)
                return 2402;

            int value = m_Beheld.Virtues.GetValue(index);

            if (value < 4000)
                return 2402;

            if (value >= 30000)
                value = 30000; //Sanity

            int vl;

            if (value < 10000)
                vl = 0;
            else if (value >= 20000 && index == 5)
                vl = 2;
            else if (value >= 21000 && index != 1)
                vl = 2;
            else if (value >= 22000 && index == 1)
                vl = 2;
            else
                vl = 1;

            return m_Table[(index * 3) + vl];
        }

[kamronbatman]

The calculation is wrong because the value was changed from 9999 to 10000. The original RunUO calculation is a hack that actually works, albeit virtually unreadable.

If your value is 20,000, and you use the SUO calculation:
(v + 10000) / 10000 -> (20000 + 10000) / 10000 -> 3 - which is wrong.
But if you use the RunUO calculation:
(v + 9999) / 10000 -> (20000 + 9999) / 10000 -> 2.99 -> 2 - which is correct, because the max value check was done before this one.

So the code simply needs to be reverted back to what RunUO had.