fix(server/player): Missing bankAuth in SetPlayerPrimaryJob -> toPlayerJob

[stone9k]

Description

bankAuth was not set when setting job with SetPlayerPrimaryJob -> toPlayerJob

Checklist

• I have personally loaded this code into an updated Qbox project and checked all of its functionality.
• My pull request fits the contribution guidelines & code conventions.

[Frowmza]

imo, static data shouldn't be in player data at all, bankAuth, isboss, payment, label. We could always access them thru core shared config/export by key which is player job name, so I think adding this is useless.

[stone9k]

imo, static data shouldn't be in player data at all, bankAuth, isboss, payment, label. We could always access them thru core shared config/export by key which is player job name, so I think adding this is useless.

This is clearly a bug: the values are set correctly on login etc, but in this specific case bankAuth isn’t being applied.
Removing it as you suggest would break compatibility with existing resources that already rely on these fields.