[jira] [Updated] (ISIS-2232) Contributed properties (from service) throws NPE, due to mismatch in push/pop on the interaction execution graph.

["Daniel Keir Haywood (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ISIS-2232?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Daniel Keir Haywood updated ISIS-2232:
--------------------------------------
    Summary: Contributed properties (from service) throws NPE, due to mismatch in push/pop on the interaction execution graph.  (was: Interaction's executionGraph should not be appended to during the render phase.)

> Contributed properties (from service) throws NPE, due to mismatch in push/pop on the interaction execution graph.
> -----------------------------------------------------------------------------------------------------------------
>
>                 Key: ISIS-2232
>                 URL: https://issues.apache.org/jira/browse/ISIS-2232
>             Project: Isis
>          Issue Type: Improvement
>            Reporter: Daniel Keir Haywood
>            Assignee: Daniel Keir Haywood
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> in the current HEAD (2.0.0-M3-SNAPSHOT), I am getting an NPE as the result of the presence of this contributed action (nb: contributed from a service):
>  
> {code:java}
> public class SimpleObjects {
>     @Action(semantics = SemanticsOf.SAFE)
>     public SimpleObject search(final SimpleObject simpleObject) {
>         return simpleObject;
>     }
>     public List<SimpleObject> autoComplete0Search(final String search) {
>         return findByName(search);
>     }
>     ...
> }{code}
>  
> The NPE occurs in the stuff around Interaction's execution:
> h3. Stack trace:
>  * java.lang.NullPointerException
>  * 
>  * org.apache.isis.applib.util.schema.MemberExecutionDtoUtils#metricsFor(MemberExecutionDtoUtils.java:77)
>  * org.apache.isis.applib.services.iactn.Interaction$Execution$When#metricsFor(Interaction.java:640)
>  * org.apache.isis.applib.services.iactn.Interaction$Execution$When#access$200(Interaction.java:582)
>  * org.apache.isis.applib.services.iactn.Interaction$Execution$When$2#syncMetrics(Interaction.java:613)
>  * org.apache.isis.applib.services.iactn.Interaction$Execution#syncMetrics(Interaction.java:660)
>  * org.apache.isis.applib.services.iactn.Interaction$Execution#setCompletedAt(Interaction.java:503)
>  * org.apache.isis.applib.services.iactn.Interaction#pop(Interaction.java:246)
>  * org.apache.isis.applib.services.iactn.Interaction#executeInternal(Interaction.java:193)
>  * org.apache.isis.applib.services.iactn.Interaction#execute(Interaction.java:142)
>  * org.apache.isis.metamodel.facets.actions.action.invocation.ActionInvocationFacetForDomainEventAbstract#doInvoke(ActionInvocationFacetForDomainEventAbstract.java:208)
>  * org.apache.isis.metamodel.facets.actions.action.invocation.ActionInvocationFacetForDomainEventAbstract#lambda$invoke$0(ActionInvocationFacetForDomainEventAbstract.java:129)
>  * org.apache.isis.runtime.services.xactn.TransactionServiceSpring#executeWithinTransaction(TransactionServiceSpring.java:149)
>  * org.apache.isis.metamodel.facets.actions.action.invocation.ActionInvocationFacetForDomainEventAbstract#invoke(ActionInvocationFacetForDomainEventAbstract.java:128)
>  * org.apache.isis.metamodel.specloader.specimpl.ObjectActionDefault#executeInternal(ObjectActionDefault.java:423)
>  * org.apache.isis.metamodel.specloader.specimpl.ObjectActionDefault#execute(ObjectActionDefault.java:409)
>  * org.apache.isis.metamodel.specloader.specimpl.OneToOneAssociationContributee#get(OneToOneAssociationContributee.java:117)
>  * org.apache.isis.viewer.wicket.model.models.ScalarModel#setObjectFromPropertyIfVisible(ScalarModel.java:761)
>  * org.apache.isis.viewer.wicket.model.models.ScalarModel#access$200(ScalarModel.java:83)
>  * org.apache.isis.viewer.wicket.model.models.ScalarModel$Kind$1#reset(ScalarModel.java:315)
>  * org.apache.isis.viewer.wicket.model.models.ScalarModel$Kind$1#init(ScalarModel.java:305)
>  * org.apache.isis.viewer.wicket.model.models.ScalarModel#init(ScalarModel.java:729)
>  * org.apache.isis.viewer.wicket.model.models.ScalarModel#<init>(ScalarModel.java:724)
>  * org.apache.isis.viewer.wicket.model.models.EntityModel#getPropertyModel(EntityModel.java:367)
>  * org.apache.isis.viewer.wicket.ui.components.entity.fieldset.PropertyGroup#addPropertyToForm(PropertyGroup.java:215)
>  * org.apache.isis.viewer.wicket.ui.components.entity.fieldset.PropertyGroup#buildGui(PropertyGroup.java:117)
>  * org.apache.isis.viewer.wicket.ui.components.entity.fieldset.PropertyGroup#<init>(PropertyGroup.java:81)
>  
> we can see that 'pop' is called but there's no execution to work upon; the stack trace tells us that we're in the render phase and evaluating this contributed property (as an action) to reset the model of the rendered entity.
>  



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