Fix: Small Fixes 1/10

[RyanYappert]

Weekly PR for various bugfixes as they're reported.

• Prevent JobGetJobChangeListHandler from hitting the packet size limit by removing unneeded information from the packet.
• Stop ItemConsumeStorageItemHandler from squelching errors.
• Fix the underlying issue with the above, which was BBM "Equip from Chest" still being broken. We're reverting back to before I tried to optimize it and broke it.
• More tweaks to server info tracking to prevent it from reading 0s for player counts when under pressure.
• Prevent NRE when running in NaiveLobbyHandling mode.
• Better exceptions for ItemManager.
• Fix bad constructor for C2SCraftGetCraftProductRes that was breaking error handling (when trying to collect a finished craft with a full storage box or bag).

Checklist:

• The project compiles
• The PR targets develop branch

[alborrajo]

The index is done after the filtering, so if you got 3 pawns, A, B and C and only two of them, A and C in the party, they would be reported as slots 1 and 2 when they should be slots 1 and 3

[RyanYappert]

This is actually expecting the order in the party, which is the order used by the UI. If you have three pawns A/B/C, but have them in the party A/C/B, you should report them in the order with indices A-1/C-2/B-3.