search

taylorkelly / BigBrother

GNU General Public License v3.0
29 stars 20 forks source link

/bb here <radius> Prints waaay too many users. #57

Closed Albuca closed 13 years ago

Albuca commented 13 years ago

A /bb here 25 ; returns 396 player(s) have edited, but only one player name (which is true). It is returning the actual number of modifications made by any player, instead of the number of players. (A /bb here 25 l returns 296 edits by that player.)

Albuca commented 13 years ago

/src/main/java/me/taylorkelly/bigbrother/finder/Finder.java Line 131 size++; //You increment size with every pass. Line 148 player.sendMessage(BigBrother.premessage + size + " player(s) have modified this area:"); //You print out the value of players (which is too large).

Possible solution: Remove the counter at Line 131 and above. Use: playerList.length() //to get the size of the player list.

Cheers! -A

Ps. Im going to go learn how to do a pull request, and well see who gets here first =P

Albuca commented 13 years ago

Created Pull Request