Civcraft / CBanManagement

BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Multiple bans #8

Closed suirad closed 9 years ago

suirad commented 9 years ago

Right now, i have it preventing duplicate and overwriting previous bans with new ones. Both internally and in the DB. Should i have it store bans for duplicate uuids? That way if one is removed the other(s) stay in effect? @rourke750 @erocs

erocs commented 9 years ago

The private key would be (UUID, PluginName), so yes, bans from other plugins would stay in effect.

suirad commented 9 years ago

I am going to make it store one ban per plugin name. Any addition bans by the same plugin will overwrite the previous.

Should i make it ignore new bans, with the same plugin name, but of lower ban level?

erocs commented 9 years ago

Sounds fine to me.