Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add console commands to manipulate ranks / groups / permissions #87

Open
madmaxoft opened this issue Sep 25, 2014 · 6 comments
Open

Add console commands to manipulate ranks / groups / permissions #87

madmaxoft opened this issue Sep 25, 2014 · 6 comments
Labels

Comments

@madmaxoft
Copy link
Member

We should have a full set of commands to manipulate ranks, groups and permissions so that people don't need to use webadmin to edit ranks.
Proposed commands:
addrank <rankname>
delrank <rankname> [<replacementrankname>]
renamerank <oldname> <newname>
addgroup <groupname>
delgroup <groupname>
renamegroup <oldname> <newname>
addpermission <groupname> <permission>
delpermission <groupname> <permission>
addrankgroup <rankname> <groupname>
delrankgroup <rankname> <groupname>
listranks
listgroups <rankname>
listpermissions <groupname>
listrankpermissions <rankname>

Marking as easy since most of these map directly to API calls, so the command handlers should be pretty easy to write.
Best place for the handlers would be the ranks.lua file; don't forget to use the Info.lua file to register and document them.

@madmaxoft madmaxoft added the easy label Sep 25, 2014
@bennasar99 bennasar99 assigned bennasar99 and unassigned bennasar99 Dec 29, 2014
@bennasar99
Copy link
Contributor

I'll try to do that.
EDIT: Didn't remember now ranks and groups are managed with SQLite.
Sorry :(

@bennasar99 bennasar99 removed their assignment Dec 29, 2014
@madmaxoft
Copy link
Member Author

They are in SQLite, but the API provides a full set of commands to work with them, so it should be a simple task of mapping the console commands to API calls. Have a look at how the webadmin management does it.

@bennasar99
Copy link
Contributor

Now I see, I may try later.

@Howaner
Copy link
Contributor

Howaner commented Mar 24, 2015

Why not create one permission command instead of many commands?

Example:
/perms group AllPerms create
/perms group AllPerms add *
/perms rank Admin create
/perms rank Admin addgroup AllPerms
/perms rank Admin prefix "&b"
/perms user Howaner setgroup Admin

@madmaxoft
Copy link
Member Author

I'm not against a single command, but I strongly suggest having simple separate commands as well, because almost anyone using the server will need to use those and we want to keep it as simple as possible for the regular people. No-one wants to read a big hunk of text in order to be able to play on a home server.

@Howaner
Copy link
Contributor

Howaner commented Mar 24, 2015

I think one command with the same usage format as PermissionsEx (The most used bukkit permission plugin) is much clearer and simpler than 20+ commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants