No global chat message on level ups - except Crafting

Everything is working great since the last update, the only thing I see as an issue right now is that the global chat mesage for all levels are not being displayed.  Only Crafting works. 

ya it does it for every level for crafting but only once every 10 levels for gathering. thats just how the author wrote it, and I kept it that way during the rewrite.

private long[] levelAnnounce = {10, 20, 30, 40, 50, 60, 70, 80, 90, 100, 110, 120, 130, 140, 150, 160, 170, 180, 190, 200};​

if (enableLevelupBroadcast)
{
	
	foreach (var target in BasePlayer.activePlayerList.Where(x => x.userID != player.userID))
	{
		if (levelAnnounce.Contains(Level))
			continue;

		if (hasRights(target.UserIDString) && playerPrefs.PlayerInfo[target.userID].ONOFF)
			Print(target, string.Format(msg("LevelUpTextBroadcast", target.UserIDString), player.displayName, Level, colors[skill], msg(skill + "Skill", target.UserIDString)));
	}
}

versus mine

if (config.generic.enableLevelupBroadcast && (int)Level % 10 == 0)
{
    foreach (var target in BasePlayer.activePlayerList)
    {
        if (target.userID == player.userID || !hasRights(target.UserIDString) || !GetPlayerInfo(target.userID).ENABLED)
            continue;

        Message(target, "LevelUpTextBroadcast", player.displayName, Level, color, _(skill + "Skill", target.UserIDString));
    }
}
nivex

ya it does it for every level for crafting but only once every 10 levels for gathering. thats just how the author wrote it, and I kept it that way during the rewrite.

private long[] levelAnnounce = {10, 20, 30, 40, 50, 60, 70, 80, 90, 100, 110, 120, 130, 140, 150, 160, 170, 180, 190, 200};​

if (enableLevelupBroadcast)
{
	
	foreach (var target in BasePlayer.activePlayerList.Where(x => x.userID != player.userID))
	{
		if (levelAnnounce.Contains(Level))
			continue;

		if (hasRights(target.UserIDString) && playerPrefs.PlayerInfo[target.userID].ONOFF)
			Print(target, string.Format(msg("LevelUpTextBroadcast", target.UserIDString), player.displayName, Level, colors[skill], msg(skill + "Skill", target.UserIDString)));
	}
}

versus mine

if (config.generic.enableLevelupBroadcast && (int)Level % 10 == 0)
{
    foreach (var target in BasePlayer.activePlayerList)
    {
        if (target.userID == player.userID || !hasRights(target.UserIDString) || !GetPlayerInfo(target.userID).ENABLED)
            continue;

        Message(target, "LevelUpTextBroadcast", player.displayName, Level, color, _(skill + "Skill", target.UserIDString));
    }
}

For quite a while there was a problem having broadcasting of leveling "on" as it caused problems with leveling up, some levels would stick at 99% so it was recommended to have the setting set to false, You would still get the chat message that your levels had gone up but not global,
I have not used the plugin with broadcasting enabled since then and find that everything else is working great (after plugin rewrite)

That makes no sense as this is just a notification.  I will remove the modulus check. Thanks

I believe he's saying that

(int)Level % 10 == 0​

makes no sense as it's just a notification and therefore should be shown for all levels.

but I didn't write it originally so I just left it there.

I still havent seen any global chat messages other than crafting, even at level 10, 20, 30 etc.