search

CivClassic / ExilePearl

For when you really just don't want that dude HERE but don't care if he plays - Paper plugin built for 1.16.5 Minecraft - Anti-grief tool built for players, not mods.
MIT License
0 stars 18 forks source link

Feature request: /ep broadcast only works on groups, should also work for individuals #35

Closed GavJenks closed 6 years ago

GavJenks commented 6 years ago

This is especially relevant on servers such as civclassic, where there's a low limit on how many namelayer groups you can create (10, there). So if I already created 10 groups, and am pearled, I just can't broadcast easily on my own. I need to go find some poor shmuck to borrow their group or make one for me, which is very awkward.

PrisonPearl already allowed individual broadcasting, so there should be a coded solution already available from that plugin.

Lazersmoke commented 6 years ago

What error message do you get? It should be one of the Lang. messages in here. The actual strings are here.

GavJenks commented 6 years ago

/ep broadcast gavjenks: "You don't have permission to chat in that group

Lazersmoke commented 6 years ago

It means exactly what it says. You attempted to broadcast into Gavjenk's personal group, but you don't have permission to chat in that group, so you couldn't. Fix is to check for player names if you can't chat in the group. Slightly ugly but informative messages will ensue.

GavJenks commented 6 years ago

I indicated "feature request" not "bug" because the error makes perfect sense for groups only. But groups only is jut annoying. It should also let you do individuals, like PP does.

Again the code already exists in PP, and is much more user friendly

GavJenks commented 6 years ago

Ohhh did you already merge a fix? If so, cool cool thanks

Lazersmoke commented 6 years ago

Individuals are already implemented, it just gave up after finding the group and realizing you couldn't chat in it. Now, it will also check for players. Putting it on the test server now.