cyberjunk / meridian59-dotnet

New 3D client and tools for Meridian 59
GNU General Public License v3.0
34 stars 36 forks source link

Sort char creation ability lists, raise points to 125. #307

Closed skittles1 closed 5 years ago

skittles1 commented 5 years ago

Logic changes to ability selection:

Also as an aside, I considered combining as much as I could into 'Ability' stuff rather than Spell/Skill duplicates, but there's some differences in what gets sent for AvatarCreatorSpellObject and AvatarCreatorSkillObject, and if we combine it then it'd be tricky to have it work on other servers (possible but clunky).

skittles1 commented 5 years ago

This should be ready to review/merge now.

cyberjunk commented 5 years ago

I think I would prefer if you do this without LINQ. I don't remember exactly but there was a reason that I did not use it so far and I would prefer if it stays that way.

skittles1 commented 5 years ago

Replaced the LINQ calls with loops.