[bug] nametags don't work without color code

I had player nametags stop working after some configuration changes. After a bit of fiddling, I discovered the issue: it appears using just %nametag% or &r%nametag% makes nametags invisible, but &f%nametag% makes them visible.

I originally set my config to look like this because I didn't want VelociTab changing nametags:

nametags:
  default: '%username%'

I tried adding &r to see if that would make nametags visible, but that did not work:

nametags:
  default: '&r%username%'

Adding &f finally fixed the issue:

nametags:
  default: '&f%username%'

we actually do handle not setting a color here:

Velocitab/src/main/java/net/william278/velocitab/packet/UpdateTeamsPacket.java

Lines 127 to 133 in 7005cec

    
           if (text == null) { 
        
               return 15; 
        
           } 
        
           int lastFormatIndex = text.lastIndexOf("§"); 
        
           if (lastFormatIndex == -1 || lastFormatIndex == text.length() - 1) { 
        
               return 15; 
        
           }

but I think something might be messing up here before that point:

Velocitab/src/main/java/net/william278/velocitab/packet/ScoreboardManager.java

Line 148 in 7005cec

tabPlayer.getNametag(plugin).thenAccept(nametag -> {

I also note we store nametags per role, which I think will probably break if player-specific placeholders are used (should prob be per-user? maybe not though).

Curious. In my experience setting [&r]%username% and running velocitab reload made usernames disappear, then setting &f%username% and reloading made them come back.

I'm running the following plugins:

Velocity

Advanced-Portals-0.9.3.jar
floodgate-velocity.jar
Geyser-Velocity.jar
LiteBans-2.12.5.jar
LuckPerms-Velocity-5.4.104.jar
MapModCompanion-0.8.1.jar
minimotd-velocity-2.0.14.jar
PAPIProxyBridge-1.4.2.jar
Plan-5.5-build-2461.jar
send-3.0.0.jar
SkinsRestorer.jar
Velocitab-1.5.1.jar
VelocityResourcepacks-b529.jar
VentureChat-3.6.0.jar

Back-end server

AdvancedPortals, BlockLocker, BuycraftX, Celeste, Chunky, ChunkyBorder, 
CMILib, CoreProtect, DiscordSRV, Essentials EssentialsAntiBuild, EssentialsGeoIP, 
EssentialsProtect, FarmControl, FastLeafDecay, floodgate, GameModeInventories, 
HiddenLorax, HyperDrive, LiteBans LuckPerms, MapModCompanion, OpenInv, 
PAPIProxyBridge, PlaceholderAPI, Plan, PlugManX, ProtocolLib, PvPManager, 
SelectionVisualizer Shopkeepers, SkinsRestorer, TimeControl, Vault, VentureChat, 
ViaBackwards, ViaVersion, VillagerModifications, WorldEdit, WorldGuard 
WorldGuardExtraFlags

If you want the default nametags just remove the default value and make the section empty. nametags: {}
If you use the latest version this problem is fixed without changing the section, you only need to recreate the config. I added an option to allow this feature or not.

we actually do handle not setting a color here:

Velocitab/src/main/java/net/william278/velocitab/packet/UpdateTeamsPacket.java

Lines 127 to 133 in 7005cec

if (text == null) {

return 15;

}

int lastFormatIndex = text.lastIndexOf("§");

if (lastFormatIndex == -1 || lastFormatIndex == text.length() - 1) {

return 15;

}

but I think something might be messing up here before that point:

Velocitab/src/main/java/net/william278/velocitab/packet/ScoreboardManager.java

Line 148 in 7005cec

tabPlayer.getNametag(plugin).thenAccept(nametag -> {

I also note we store nametags per role, which I think will probably break if player-specific placeholders are used (should prob be per-user? maybe not though).

Each user is assigned a specific and unique role, which mitigates any potential issues.

news @RalphORama ?

Could you try with the latest Alpha version? https://modrinth.com/plugin/velocitab/version/

closing as this should have been fixed in the latest alpha via #108

	if (text == null) {
	return 15;
	}
	int lastFormatIndex = text.lastIndexOf("§");
	if (lastFormatIndex == -1 \|\| lastFormatIndex == text.length() - 1) {
	return 15;
	}