EventStrategy has a few issues that could be smoothed out, but not without an allowance for breaking change in the API:
- For the creation of new properties, an empty detached property is created to represent it - now that detachment is configurable, that doesn't always make sense. For example, if you configured for reference detachment then you would probably want a ReferenceProperty instead. Not sure how this should be resolved, but it probably needs a change to the eventing API itself
- Detachment is configured a bit strangely with the use of null and passing classes for the appropriate detachment factories....would be nicer to have an interface to represent this stuff.