flying-squid icon indicating copy to clipboard operation
flying-squid copied to clipboard

Implement ops

Open rom1504 opened this issue 10 years ago • 10 comments

Only ops should be able to kick and ban.

rom1504 avatar Sep 06 '15 15:09 rom1504

We should first implement a permissions system.

ghost avatar Sep 27 '15 19:09 ghost

Well this is the permissions system

rom1504 avatar Sep 27 '15 19:09 rom1504

Yes, but I mean not like normal minecraft's permission system. I am talking about a good one

ghost avatar Sep 27 '15 19:09 ghost

Something like, (not real JSON, I think)

[
{
  "name": "mhsjlw",
  "permissions": [
    "*"
  ]
},
{
  "name": "rom1504",
  "permissions": [
    "world-guard.*"
  ]
}
]

ghost avatar Sep 27 '15 19:09 ghost

I think it would be best to simply implement ops (wouldn't be hard, just player.op) and use that ("return" inside a command if the player is not an op). I imagine somebody will make an advanced permissions base-external-plugin that other plugins will build on top of.

Just my personal opinion so we can get as similar to vanilla as possible.

demipixel avatar Nov 06 '15 09:11 demipixel

I'm fine with both solutions.

rom1504 avatar Nov 06 '15 10:11 rom1504

Sorta done in the other PR

demipixel avatar Dec 10 '15 07:12 demipixel

We're still missing the saving/loading part

rom1504 avatar Dec 10 '15 09:12 rom1504

Okay, although that seems like a completely seperate issue considering nothing gets saved yet :P

demipixel avatar Dec 10 '15 15:12 demipixel

well, ops saving/loading is fairly independent from all the rest of the saving if we follow vanilla format (it's just a ops.json file) (https://github.com/rom1504/node-minecraft-wrap/blob/master/lib/wrap.js#L113)

rom1504 avatar Dec 10 '15 15:12 rom1504