Dan Haywood created ISIS-2118: --------------------------------- Summary: Don't redundantly add additional DisabledFacet when disabled at class-level. Key: ISIS-2118 URL: https://issues.apache.org/jira/browse/ISIS-2118 Project: Isis Issue Type: Bug Affects Versions: 1.17.0 Reporter: Dan Haywood Assignee: Dan Haywood Fix For: 1.18.0
using the ISIS-2015 mixin, I see the following for kitchensink's ReferenceObject when annotated as @DomainObject(editing=DISABLED) {code} <mml:prop id="otherName"> <mml:facets> ... <mml:facet id="org.apache.isis.core.metamodel.facets.members.disabled.DisabledFacet" fqcn="org.apache.isis.core.metamodel.facets.properties.disabled.fromimmutable.DisabledFacetOnPropertyDerivedFromImmutable"> <mml:attr name="disabling">true</mml:attr> <mml:attr name="inverted">false</mml:attr> <mml:attr name="semantics">DISABLED</mml:attr> <mml:attr name="when">ALWAYS</mml:attr> <mml:attr name="where">ANYWHERE</mml:attr> </mml:facet> ... <mml:facet id="org.apache.isis.core.metamodel.facets.object.immutable.ImmutableFacet" fqcn="org.apache.isis.core.metamodel.facets.object.domainobject.editing.ImmutableFacetForDomainObjectAnnotation"> <mml:attr name="disabling">true</mml:attr> <mml:attr name="reason">Disabled</mml:attr> <mml:attr name="when">ALWAYS</mml:attr> </mml:facet> {code} The ImmutableFacet isn't required, because the DisabledFacetOnPropertyDerivedFromImmutable should suffice. -- This message was sent by Atlassian JIRA (v7.6.3#76005)