top command should skip inactive...badly

ValkyrjaProject · Jul 10, 2024 · d22edf9 · d22edf9
1 parent bb75cc1
commit d22edf9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Modules/Karma.cs b/Modules/Karma.cs
@@ -33,7 +33,7 @@ public List<Command> Init(IValkyrjaClient iClient)
 // !top
 			Command newCommand = new Command("top");
 			newCommand.Type = CommandType.Standard;
-			newCommand.Description = "Check how many cookies you've got.";
+			newCommand.Description = "List the top karma holders.";
 			newCommand.ManPage = new ManPage("[n]", "`[n]` - Optional argument specifying how many members with the highest count you would like to fetch. Default 5.");
 			newCommand.IsPremiumServerwideCommand = true;
 			newCommand.RequiredPermissions = PermissionType.Everyone;
@@ -63,7 +63,7 @@ public List<Command> Init(IValkyrjaClient iClient)
 				foreach( UserData userData in users )
 				{
 					if( i++ == n )
-						return;
+						break;
 
 					SocketGuildUser user = null;
 					if( (user = e.Server.Guild.GetUser(userData.UserId)) == null )