Doctor Who - Regeneration

Doctor Who - Regeneration

318k Downloads

[SUGGESTION]Implement whitelist/blacklist for player UUID who can/can't use regeneration

JackCarterSmith opened this issue ยท 10 comments

commented

Describe the feature you'd like to see implemented:
I think you can implement a perms/blacklist/whitelist in config file for specify who can regenerate or not. (By default you set to any for the lambda player)
It's can be usefull for server owner with ranked or perms controlled features.

commented

For my personnal use permission node can do the job! I thought of an UUID whitelist because other mods that implemented it this way.

commented

Any reason for a config value? Would a permission node for permission mods not suffice?

commented

Permissions is already implemented ?

commented

No, it is not
It will be next release

commented

Maybe I can help for the code? Up to you ^^

commented

I've already started ๐Ÿ˜…
But you can help suggest what permission nodes you need :)

commented

Okay okay good news!
I will explain how do I see it:

  • max_regeneration (for a group/user)
  • can_regenerate (for a group/user/ or delimited area like "regen inhibition area")
  • can_keep_previous_skin (for a group/user)
  • can_be_kill_onregen (for a group/user)
  • a perm node/add command to add (not set) regen to a specific player (for challenge/event reward)

For now it's what I think I need ^^
Thank you for your great job!

commented

All seem pretty doable, what do you mean by can_keep_previous_skin though?

commented

I think It's can be better to make 3 sub-nodes to it:

  • Main node: randomSkins-> if it's true, the player regenerate with a new randoms skins, but depend on the node "canSelectSkin" to tell if a player can or can't select is next regen. If false the player keep it's current skin.
  • subnode: canSelectSkin -> the player CAN select is next skin "regen" in the select GUI
  • subnode: canKeepSkin -> the player can choose to keep is current skin for is next regen.

Better explained?

commented

I will think about adding these three permissions! If I do, expect them next version!