search

IntellectualSites / PlotSquared

PlotSquared - Reinventing the plotworld
GNU General Public License v3.0
543 stars 740 forks source link

argument priorisation problem for /p tp #1691

Closed khlfkk closed 6 years ago

khlfkk commented 7 years ago

Bug report

paste link: https://pastebin.com/Wg27wNnq

Description of the problem: when visit the player claim /p tp 7005151678 it caused exeception:( java.lang.NumberFormatException: For input string: "7005151678" )

boy0001 commented 7 years ago

Is that meant to be a player name?

khlfkk commented 7 years ago

yup, it is a player name called "7005151678". May i know is it don't support full number IGN?

RedstoneFuture commented 6 years ago

Yes. I get a error too. ^^ "An error occured: For input string: 005151678"

https://namemc.com/profile/7005151678.1

Opachl commented 6 years ago

I will have a look to this. :)

Bug is reproducible. I try to find a Solution.

I will try change the code in that way that it always asumes this fallbackorder: -> Playername -> Alias -> world -> PlotId -> page (Searchresult)

Additional as the Player is unkown (means never played on the Server) will result assuming it as id. See Fallbackorder. For this special case we will catch the NumberFormatException and convert it to a normal console warning message.

RedstoneFuture commented 6 years ago

https://github.com/IntellectualSites/PlotSquared/issues/1822

RedstoneFuture commented 6 years ago

@boy0001 @Sauilitired What is the currently argument prioritisation of the /home command? Is a rework planed?