Essentials icon indicating copy to clipboard operation
Essentials copied to clipboard

Add option for Social Spy to use player names

Open Bobcat00 opened this issue 2 years ago • 1 comments

Information

This PR closes #1995 from 5 years and 8 months ago.

Details

Proposed feature:

Add a config file option for Social Spy to use either display names (the default) or player names in its output.

~~Also fix a bug which has been present for 6+ years where socialMutedSpyPrefix should have been socialSpyMutedPrefix.~~ (This was fixed in 2.x with the Adventure changes.)

Environments tested:

OS: Linux 4.15.0-213-generic

Java version: Oracle 17.0.8+9-LTS-211

  • [x] Most recent Paper version git-Paper-381 (MC: 1.20.4)
  • [ ] CraftBukkit/Spigot/Paper 1.12.2
  • [ ] CraftBukkit 1.8.8

Demonstration:

Sample output with socialspy-uses-displaynames: true (the current behavior and default):

SocialSpy1

Sample output with socialspy-uses-displaynames: false:

SocialSpy2

I'll note that I find the second version to be much more readable.

Bobcat00 avatar Jan 09 '24 15:01 Bobcat00

Rebased and retested successfully.

Bobcat00 avatar Feb 21 '24 22:02 Bobcat00