Rework module removal

[tornac1234]

Describe the issue

As of now, as pointed out by @SpaceMonkeyy86, module removal works pretty weirdly (e.g. it has a NewParentId which points to its old container) and ModuleRemovedProcessor.cs (server-side) has weird functionning towards it.

I think we should list the different causes of module removal and either treat those as a module reparent (when a module is switched in another slot) or as a proper module unequip (I like this wording better) in which case there'd be both an ParentId and a NewParentId, which would make it clear that the module is becoming a simple item in someone's inventory