OutlawByteStudios / Persistent-Kingdoms

Persistent Worlds Overhaul Mod for Mount and Blade Warband
Other
7 stars 7 forks source link

"player_set_admin_permissions" Improvements #477

Closed Thomas-Smyth closed 6 years ago

Thomas-Smyth commented 6 years ago

Since new admin permissions slots have been added this function is outdated as not all options can be updated using it.

Either needs to be updated or removed.

Saptortacus commented 6 years ago

I say we remove it because it should be left to server owners to handle permission stuff

Thomas-Smyth commented 6 years ago

@Saptortacus the purpose of this function is to use a set of bits to assign permissions on the user's client so menu options can be hidden.

As ModSys only allows you to send 4 ints this makes it easier to update all the permission slots as it can be done with a single server message.

Server owners still manage the permissions, this just makes it easier for them. We do not use it on Phoenix's Scripts as I don't think ModSys can format the bits correctly and the permissions are returned from the database separately anyway.