search

wovencode / OpenMMO-Other

Ideas, Tasks and others
2 stars 0 forks source link

Allow Whitespace in Character/Account names again #3

Closed francy51 closed 4 years ago

francy51 commented 4 years ago

Its basically all in Tools.IsAllowedName function. BUT: This is a bit tricky with multiple whitespaces in names. there should be just 1 allowed.

DX4D commented 4 years ago

Multiple methods for accomplishing this are located in the WIP folder as well as unit tests for most. Those methods need to all be tested for performance as this can be a very costly method. Do not trust the reported numbers, we need to retest all of the methods and find the best one so we don't have 2 seconds of lag added to character creation submission.

DX4D commented 4 years ago

(Close issue #11 in the other issue tracker once this is completed)

DX4D commented 4 years ago

@francy51 Please close this issue, I will open a new one.

DX4D commented 4 years ago

Finished