[CAUSEWAY-2255] In WrapperFactory, lookup mixin properties and collections (as well as actions).

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.0.0-M3
• Assignee: Daniel Keir Haywood
• Reporter: Daniel Keir Haywood
• Created: 05/Jan/20 17:19
• Updated: 07/Jan/20 17:31
• Resolved: 07/Jan/20 17:31

Description

Because we are now more strict about the bookmarks that can be created (in particular, because we do not allow bookmarks to be created on transient objects such as mixins), then the previous code was failing when attempting to create an InteractionDto with a bookmark to a mixin.

Tracking this down, this was because of the invocation of a contributed collection via the WrapperFactory.  The WrapperFactory (more precisely, DomainObjectInvocationHandler) attempts to do a "reverse-looking" to find the contributed mixed member  from the original action on the mixin, but only searched for contributed actions, and did not consider contributed properties or collections.

This change is to fix this.  It should mean that the targetAdapter for InteractionDto's is always correct, and that memberIdentifier == logicalMemberIdentifier in all cases.