benini / scid

Other
43 stars 12 forks source link

Pinfo #170

Closed ukscid closed 6 months ago

ukscid commented 7 months ago

Bugfix for splitnames "lname, fname" results in a leading blank to " fname .."

chess24.com has moved to chess.com and a real good player webside. resolvers for chessbase and chess.com added

ukscid commented 6 months ago

Input "Lastname, Firstname" results in list: " Lastname" "Firstname". There is a blank at the start of lastname. This leads to a url like https://chess.com/en/players/%%20firstname-lastname or https://players.chessbase.com/de/player/lastname_%%20firstname Most website ignore this additional blank (%%20) and work correct, but chess.com does not and throws a 404 Page not found error. This fix should remove this blank before the lastname.

benini commented 6 months ago

Do you just want to trim whitespaces?

ukscid commented 6 months ago

You are right. I changed it to use [string trim ..]