[ 
https://issues.apache.org/jira/browse/ISIS-2255?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17008372#comment-17008372
 ] 

ASF subversion and git services commented on ISIS-2255:
-------------------------------------------------------

Commit 7de533afd1a6a55ef1337d05c27b17eda40fdd83 in isis's branch 
refs/heads/master from danhaywood
[ https://gitbox.apache.org/repos/asf?p=isis.git;h=7de533a ]

ISIS-2255: sync-adoc.sh


> In WrapperFactory, lookup mixin properties and collections (as well as 
> actions).
> --------------------------------------------------------------------------------
>
>                 Key: ISIS-2255
>                 URL: https://issues.apache.org/jira/browse/ISIS-2255
>             Project: Isis
>          Issue Type: Improvement
>            Reporter: Daniel Keir Haywood
>            Assignee: Daniel Keir Haywood
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> 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.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to