Removed the AttributeService
This wasn't really acting like a service, was only calling on the Character Service to do the actual persisting of entity changes.
Believe most of this code was more like a Facade layer action.
Made Character Getters Unmodifiable
These should only be modified by services, preventing any badly coded skills from changing these values
Added a few more modifier to support modification to the maps within the Character class
Renamed BaseAttributes to CharacterAttributes
Character = Leveled up attributes
base = default + character
Removed the AttributeService This wasn't really acting like a service, was only calling on the Character Service to do the actual persisting of entity changes. Believe most of this code was more like a Facade layer action.
Made Character Getters Unmodifiable These should only be modified by services, preventing any badly coded skills from changing these values Added a few more modifier to support modification to the maps within the Character class
Renamed BaseAttributes to CharacterAttributes Character = Leveled up attributes base = default + character