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

Dog gets user prefix in nametag #109

Open
Krazytrain opened this issue Sep 4, 2023 · 7 comments
Open

Dog gets user prefix in nametag #109

Krazytrain opened this issue Sep 4, 2023 · 7 comments
Labels
enhancement New feature or request

Comments

@Krazytrain
Copy link

Type of bug

Compatibility issue

Error log (if applicable)

No response

Bug description

The prefix of your nametag goes on the dogs nametag.

Steps to reproduce

  1. Have a prefix on your name with luckperms or nametagedit
  2. Your dog will have this prefix too.

Expected behaviour

Dog not to have the prefix in there name.

Actual behaviour

Dog namename has your prefix in there nametag too.

@Krazytrain Krazytrain added the bug Something isn't working label Sep 4, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in MyDog Sep 4, 2023
@HeroGamers
Copy link
Owner

Hi there,

Thanks for the bug report!

Can you let me know what your chat formatting plugin is? i.e. EssentialsChat, etc.
Do you have any other plugins that might change the names of tameables?

I have had prefixes on multiple testing environments for MyDog and have never run into this issue - so I would need some more info to try and reproduce this issue.

For technicalities, the following is how custom names for dogs are made in MyDog (https://github.com/HeroGamers/MyDog/blob/master/src/main/java/dk/fido2603/mydog/objects/Dog.java#L461):

  • Get dog collar color
  • Get name of dog
  • Create custom name with dog collar color, name of dog, and optionally the current dog mode

None of the code should use prefixes in any way, or have anything to do with the owner.

Could you provide more information, then I can try reproducing it again.

Cheers,
Hero

@HeroGamers HeroGamers added the more info needed This is hard to fix with the current information given label Sep 7, 2023
@HeroGamers HeroGamers changed the title Dog Nametag Dog gets user prefix in nametag Sep 7, 2023
@Krazytrain
Copy link
Author

We have Essentials chat and Name tag edit plugin, we also use luck perms for the permissions.

@Krazytrain
Copy link
Author

Looking into it tab has nametags too but we dont use them. We only use it for the tab lists

@Krazytrain
Copy link
Author

Krazytrain commented Sep 7, 2023

So i have removed alot of the plugins and still happens...

Image

Nothing to do with luckperms too.

@Krazytrain
Copy link
Author

Turns out it was TAB...
Reading through there wiki i found this,
Additional note 2 - Prefix/suffix on pets
Since 1.9 teams affect tamed animals as well, displaying prefix/suffix on them as well and if unlimited nametag mode is enabled, they will be completely invisible. If you want to avoid it, install this plugin.

I had to add another plugin to remove the nametags on pets.

@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in MyDog Sep 7, 2023
@HeroGamers
Copy link
Owner

Hi there!

Thanks so much for the extensive testing!
In the next release, I'll reopen the issue and turn it into a feature to disable owner nametags on dogs, such that others don't have to go and install another plugin for a feature which I feel should also be implemented by MyDog.

Best regards,
Hero

@HeroGamers HeroGamers reopened this Sep 11, 2023
@github-project-automation github-project-automation bot moved this from ✅ Done to 📋 Backlog in MyDog Sep 11, 2023
@HeroGamers HeroGamers added enhancement New feature or request and removed bug Something isn't working more info needed This is hard to fix with the current information given labels Sep 11, 2023
@HeroGamers
Copy link
Owner

Note for future: Probably requires NMS or ProtocolLib

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants