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)
