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

Update to 1.20.5-1.21 #912

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Aurelien30000
Copy link
Contributor

@Aurelien30000 Aurelien30000 commented Jul 7, 2024

  • Update maven plugins dependencies
  • Rework a bit packet accessors to support both legacy craft bukkit package versions & modern paper mapping

You can try it with the following provided .jar.
Please be cautions using it!

1.21.4 supported

https://github.com/Aurelien30000/NametagEdit/actions/runs/12456443965

- Update maven plugins dependencies
- Rework a bit packet accessors to support both legacy craft bukkit package versions & modern paper mapping
@fleivinho
Copy link

Nice!

@riqvip
Copy link

riqvip commented Jul 11, 2024

Thanks so much man. Honestly you're a life saver.

@Shadow80456
Copy link

upload the plugin to, 1.20.6, PLS

@JustOneMoreBlock
Copy link
Collaborator

Same file

@Shadow80456
Copy link

Shadow80456 commented Jul 14, 2024 via email

@JustOneMoreBlock
Copy link
Collaborator

  • Update maven plugins dependencies
  • Rework a bit packet accessors to support both legacy craft bukkit package versions & modern paper mapping

You can try it with the following provided .jar. Please be cautions using it!

NametagEdit.zip

@JustSimplyGaming
Copy link

  • Update maven plugins dependencies
  • Rework a bit packet accessors to support both legacy craft bukkit package versions & modern paper mapping

You can try it with the following provided .jar. Please be cautions using it!
NametagEdit.zip

Hey sorry for the late reply.
At the moment i think it works, but in combination with Ultrapermissions it dosnt work as intended i think.
image

But when its "active":

image

^^'

@Aurelien30000
Copy link
Contributor Author

  • Update maven plugins dependencies
  • Rework a bit packet accessors to support both legacy craft bukkit package versions & modern paper mapping

You can try it with the following provided .jar. Please be cautions using it!
NametagEdit.zip

Hey sorry for the late reply. At the moment i think it works, but in combination with Ultrapermissions it dosnt work as intended i think. image

But when its "active":

image

^^'

As far as I know this plugin may not be directly supported. I don't think it is related to the 1.21 update, please tell me if it is not the case. Note that NameTagEdit has its own handling and probably overrides all other changes made from other plugins such as Ultrapermissions (that are not aware of NTE).

@JustSimplyGaming
Copy link

Thats strange cause UltraPermission names just your Plugin as functional. If i remove it, UltraPermission says that for this function "NameTagEdit" is needed.

@gosuper2
Copy link

  • Обновление зависимостей плагинов maven
  • Немного переработаны методы доступа к пакетам для поддержки как устаревших версий пакета Craft bukkit, так и современного бумажного сопоставления

Вы можете попробовать это с помощью следующих предоставленных . Пожалуйста, будьте осторожны при его использовании!.jar

NametagEdit.zip
Hi, your plugin works perfectly on version 1.21, could you post it on the spigot website, I just use aternos for playing with friends, and there are no more normal plugins for prefixes in the tab

@Aurelien30000
Copy link
Contributor Author

Thats strange cause UltraPermission names just your Plugin as functional. If i remove it, UltraPermission says that for this function "NameTagEdit" is needed.

Right, sorry for the confusion. I can't really see what would cause that on our side. Can you reproduce it on older versions (e.g. 1.20.4)?

@Erlpil
Copy link

Erlpil commented Aug 10, 2024

Hello! Thanks for providing this build. It worked great on 1.21!
For info: This breaks on the new 1.21.1 just released. I ran their build nr. 2 for 1.21.1
Here is the error: pastebin.com/raw/reyaJp9m

@Aurelien30000
Copy link
Contributor Author

@Erlpil Thanks for the feedback. The provided plugin has been updated, you can try it out!

@mxz7
Copy link

mxz7 commented Aug 21, 2024

@Aurelien30000 hex colours don't seem to work for usernames, it uses the last digit of the hex code as the username colour

image

@Aurelien30000
Copy link
Contributor Author

Aurelien30000 commented Aug 21, 2024

@mxz7 That's a weird issue but you are right, it seems that combined color codes/non standard colors (as last) no longer work. I have also noticed that simple formatting such as &4&l is affected for player names.

I will investigate further & try to fix the issue.

@Aurelien30000
Copy link
Contributor Author

Aurelien30000 commented Aug 24, 2024

Apparently, from the first checks I have done, the method used by NTE (team) is no longer suitable for "coloring" the player name as before (bold, previous colors from the prefix, etc.). Only basic colors may be supported. The issue could be fixed by using an alternative method or another plugin.

Anyways, I will make further investigations and maybe try to make a Mojang report.

@mxz7
Copy link

mxz7 commented Aug 28, 2024

Yeah I moved to TAB and seems to work better for my needs, a bit more complicated but way more features

@Erlpil
Copy link

Erlpil commented Nov 24, 2024

We are back to the grind with the 1.21.3 stable release with paper. Tried to run the provided build in the original post on papers build 59 for 1.21.3 and got this error: https://pastebin.com/raw/XV3diQgh

Are you planning on keeping this project alive @Aurelien30000 , or is it time for us to start to look elsewhere?
(I really appreciate all the work you have put into the plugin keeping it alive so far <3 )

@Aurelien30000
Copy link
Contributor Author

We are back to the grind with the 1.21.3 stable release with paper. Tried to run the provided build in the original post on papers build 59 for 1.21.3 and got this error: https://pastebin.com/raw/XV3diQgh

Are you planning on keeping this project alive @Aurelien30000 , or is it time for us to start to look elsewhere? (I really appreciate all the work you have put into the plugin keeping it alive so far <3 )

@Erlpil Sure, sorry for the delay, NameTagEdit has been updated to support 1.21.2+.

You can download artifacts from here: https://github.com/Aurelien30000/NametagEdit/actions/runs/12076523603

@Erlpil
Copy link

Erlpil commented Dec 1, 2024

Thanks for the patch! Worked like a charm on paper 1.21.3 build 76! We really appreciate you keeping this plugin alive!

@Aurelien30000
Copy link
Contributor Author

@Aurelien30000

https://pastebin.com/L3permZe

paper/spigot 1.20.4/1.21.4 with your latest release

Hi, thanks for reaching me out. I able currently unable to reproduce your issue. Can you provide more details about your server setup, plugins and NTE configuration? Thanks !

@Aurelien30000
Copy link
Contributor Author

@Nandi0813 Please could you provide the requested additional information? Thanks

@Brasil150
Copy link

Brasil150 commented Feb 4, 2025

Hi @Aurelien30000 The plugin is breaking in 1.21.4, can you fix it?

[23:22:03 INFO]: [NametagEdit] Enabling NametagEdit v4.5.24
[23:22:03 INFO]: [NametagEdit] Disabling NametagEdit v4.5.24
[23:22:03 ERROR]: Error occurred while disabling NametagEdit v4.5.24
java.lang.NullPointerException: Cannot invoke "com.nametagedit.plugin.NametagManager.reset()" because "this.manager" is null
        at NametagEdit.jar/com.nametagedit.plugin.NametagEdit.onDisable(NametagEdit.java:86) ~[NametagEdit.jar:?]
        at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:290) ~[purpur-api-1.21.4-R0.1-SNAPSHOT.jar:?]
        at io.papermc.paper.plugin.manager.PaperPluginInstanceManager.disablePlugin(PaperPluginInstanceManager.java:237) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.disablePlugin(PaperPluginManagerImpl.java:114) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at org.bukkit.plugin.SimplePluginManager.disablePlugin(SimplePluginManager.java:550) ~[purpur-api-1.21.4-R0.1-SNAPSHOT.jar:?]
        at NametagEdit.jar/com.nametagedit.plugin.NametagEdit.testCompat(NametagEdit.java:106) ~[NametagEdit.jar:?]
        at NametagEdit.jar/com.nametagedit.plugin.NametagEdit.onEnable(NametagEdit.java:39) ~[NametagEdit.jar:?]
        at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:284) ~[purpur-api-1.21.4-R0.1-SNAPSHOT.jar:?]
        at io.papermc.paper.plugin.manager.PaperPluginInstanceManager.enablePlugin(PaperPluginInstanceManager.java:202) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.enablePlugin(PaperPluginManagerImpl.java:109) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:520) ~[purpur-api-1.21.4-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.craftbukkit.CraftServer.enablePlugin(CraftServer.java:670) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at org.bukkit.craftbukkit.CraftServer.enablePlugins(CraftServer.java:619) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at net.minecraft.server.MinecraftServer.loadWorld0(MinecraftServer.java:746) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at net.minecraft.server.MinecraftServer.loadLevel(MinecraftServer.java:491) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at net.minecraft.server.dedicated.DedicatedServer.initServer(DedicatedServer.java:365) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1178) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at net.minecraft.server.MinecraftServer.lambda$spin$2(MinecraftServer.java:313) ~[purpur-1.21.4.jar:1.21.4-2387-4f48185]
        at java.base/java.lang.Thread.run(Thread.java:1583) ~[?:?]
[23:22:03 ERROR]: [NametagEdit]
------------------------------------------------------
[WARNING] NametagEdit v4.5.24 Failed to load! [WARNING]
------------------------------------------------------
This might be an issue with reflection. REPORT this:
> Cannot invoke "java.lang.reflect.Field.set(Object, Object)" because "com.nametagedit.plugin.packets.PacketAccessor.PARAMS" is null
The plugin will now self destruct.
------------------------------------------------------

@WilliamsJack
Copy link

@Brasil150 Try the latest build at https://github.com/Aurelien30000/NametagEdit/actions/workflows/maven.yml - it looks like you might be running NametagEdit 1.21.2/1.21.3

@Erlpil
Copy link

Erlpil commented Feb 7, 2025

Nametagedit breaks login on BungeeCord build 1899 and later.

Hello again @Aurelien30000 !
Not sure where you want this issue reported, writing it here for now. Please direct me if you want it somewhere else.

We are running tour latest build for NametagEdit downloaded from here: https://github.com/Aurelien30000/NametagEdit/actions/runs/12456443965

During routine maintenance we found that this causes issues when running BungeeCord build 1899 or later. This is the error in the bungee console: pastebin.com/raw/YBCAEXnM

I have tried to run BungeeCord without any plugins as well as ripping out all plugins on our hub server. The issue only appeared when NametagEdit was installed.

Here is the BungeeCord commit that causes the issue: SpigotMC/BungeeCord@508c2f7

Is there a way you can solve this problem?
Please let me know if you need anything from me to be able to solve this, if you have time of course.

@Maaattqc
Copy link

Maaattqc commented Feb 9, 2025

Thanks I'm running it on 1.21.4 and it works

@Aurelien30000
Copy link
Contributor Author

Nametagedit breaks login on BungeeCord build 1899 and later.

Hello again @Aurelien30000 ! Not sure where you want this issue reported, writing it here for now. Please direct me if you want it somewhere else.

We are running tour latest build for NametagEdit downloaded from here: https://github.com/Aurelien30000/NametagEdit/actions/runs/12456443965

During routine maintenance we found that this causes issues when running BungeeCord build 1899 or later. This is the error in the bungee console: pastebin.com/raw/YBCAEXnM

I have tried to run BungeeCord without any plugins as well as ripping out all plugins on our hub server. The issue only appeared when NametagEdit was installed.

Here is the BungeeCord commit that causes the issue: SpigotMC/BungeeCord@508c2f7

Is there a way you can solve this problem? Please let me know if you need anything from me to be able to solve this, if you have time of course.

Hi, thanks for the report! This is likely caused by a change in packet internal values that need to be handled in our side. As 1.21.5 is not yet release, I can't guarantee it will be fixed. I'll take a look next week anyways!

@Brasil150
Copy link

Hi @Aurelien30000 any news about Bungeecord changes?

@Erlpil
Copy link

Erlpil commented Mar 5, 2025

Just a heads up here. Geyser (bedrock proxy solution for java servers) are now forcing people to use an updated version of BungeeCord due to some internal changes.

As Bedrock versions are released more frequently than java, there wont be long before server owners will be forced to upgrade their BungeeCord instance, and then remove NameTagEdit due to the error mentioned above. I have tested that the issue with NametagEdit still presist on the current latest BungeeCord build 1918.

@Brasil150
Copy link

Yes @Erlpil Unfortunately I had to remove NameTagEdit to be able to update my Proxy and consequently Geyser and Floodgate as both are requiring the Proxy to be updated.

How updating the Proxy causes problems in NametagEdit was the way. Fortunately this problem only happens on servers running higher versions of the game like 1.21.

I have a Lobby on 1.12.2 and have had no problems.

@Alexgamer179097
Copy link

@Aurelien30000 como lo descargo para la 1.21.4 porfavor

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

Successfully merging this pull request may close these issues.