split getFieldDefinitionSettings() into field and storage settings

This is maybe overkill. Pondering.

We should at least document that it's fine to mix both in one array, because core does that too.

Automatically closed - issue fixed for 2 weeks with no activity.