search

MonoliYoda / evve_dashboard_dev

0 stars 2 forks source link

Parsing character names assumes all characters have 2 names #1

Closed SoCariEVE closed 4 years ago

SoCariEVE commented 4 years ago

dashboard/views.py tries to destructure a character's name into first_name and last_name. This fails for characters that have only one name.

MonoliYoda commented 4 years ago

Wrapping the .split() function in a try-catch block with a failover to last_name ="" should do the trick.

MonoliYoda commented 4 years ago

I pushed this change to the dev server. If it works all good, will update the repo.

dgrif commented 4 years ago

tested with my main and alt, both single word usernames. prior to the fix I got the server error, but now it works. 👍

MonoliYoda commented 4 years ago

Fixed.