CAUSEWAY-3859: flattens PromptStyleFacet

Author: andi-huber

core/metamodel/src/main/java/org/apache/causeway/core/metamodel/facets/actions/layout/ActionLayoutFacetFactory.java

@@ -18,16 +18,22 @@
  */
 package org.apache.causeway.core.metamodel.facets.actions.layout;
 
+import java.util.Optional;
+
 import jakarta.inject.Inject;
 
 import org.apache.causeway.applib.annotation.ActionLayout;
+import org.apache.causeway.applib.annotation.PromptStyle;
+import org.apache.causeway.core.config.CausewayConfiguration;
 import org.apache.causeway.core.metamodel.context.MetaModelContext;
+import org.apache.causeway.core.metamodel.facetapi.FacetHolder;
 import org.apache.causeway.core.metamodel.facetapi.FeatureType;
 import org.apache.causeway.core.metamodel.facets.FacetFactoryAbstract;
 import org.apache.causeway.core.metamodel.facets.actions.position.ActionPositionFacetFallback;
 import org.apache.causeway.core.metamodel.facets.actions.redirect.RedirectFacetFallback;
 import org.apache.causeway.core.metamodel.facets.members.layout.group.LayoutGroupFacetFromActionLayoutAnnotation;
 import org.apache.causeway.core.metamodel.facets.members.layout.order.LayoutOrderFacetFromActionLayoutAnnotation;
+import org.apache.causeway.core.metamodel.facets.object.promptStyle.PromptStyleFacet;
 import org.apache.causeway.core.metamodel.specloader.validator.ValidationFailureUtils;
 
 public class ActionLayoutFacetFactory
@@ -79,8 +85,8 @@ public void process(final ProcessMethodContext processMethodContext) {
                 .create(actionLayoutIfAny, facetHolder));
 
         // promptStyle
-        addFacetIfPresent(PromptStyleFacetForActionLayoutAnnotation
-                .create(actionLayoutIfAny, getConfiguration(), facetHolder));
+        addFacetIfPresent(
+            createPromptStyleFacetForActionLayoutAnnotation(actionLayoutIfAny, getConfiguration(), facetHolder));
 
         // position
         var actionPositionFacet = ActionPositionFacetForActionLayoutAnnotation
@@ -102,4 +108,32 @@ public void process(final ProcessMethodContext processMethodContext) {
 
     }
 
+    // -- HELPER
+
+    private static Optional<PromptStyleFacet> createPromptStyleFacetForActionLayoutAnnotation(
+        final Optional<ActionLayout> actionLayoutIfAny,
+        final CausewayConfiguration configuration,
+        final FacetHolder holder) {
+
+        return Optional.ofNullable(
+            actionLayoutIfAny
+                .map(ActionLayout::promptStyle)
+                .filter(promptStyle -> promptStyle != PromptStyle.NOT_SPECIFIED)
+                .map(promptStyle -> switch (promptStyle) {
+                    case DIALOG, DIALOG_MODAL, DIALOG_SIDEBAR, INLINE, INLINE_AS_IF_EDIT->
+                        new PromptStyleFacet("ActionLayoutAnnotation", promptStyle, holder);
+                    case AS_CONFIGURED->
+                        holder.containsNonFallbackFacet(PromptStyleFacet.class)
+                            ? null // do not replace
+                            : PromptStyleFacet.asConfgured(configuration, holder);
+                    case NOT_SPECIFIED -> null; // unexpected code reach
+                })
+                .orElseGet(() ->
+                    // do not replace
+                    holder.containsNonFallbackFacet(PromptStyleFacet.class)
+                        ? null
+                        : PromptStyleFacet.asConfgured(configuration, holder))
+        );
+    }
+
 }

core/metamodel/src/main/java/org/apache/causeway/core/metamodel/facets/actions/layout/PromptStyleFacetForActionLayoutAnnotation.java

@@ -49,7 +49,7 @@ public ActionSemanticsFacet(final String origin, final SemanticsOf of, final Fac
     @Override
     public void visitAttributes(final BiConsumer<String, Object> visitor) {
         visitor.accept("origin", origin());
-        visitor.accept("precedence", getPrecedence().name());
+        visitor.accept("precedence", precedence().name());
         visitor.accept("value", value);
     }
 

core/metamodel/src/main/java/org/apache/causeway/core/metamodel/facets/actions/layout/PromptStyleFacetForActionLayoutXml.java

@@ -18,15 +18,50 @@
  */
 package org.apache.causeway.core.metamodel.facets.object.promptStyle;
 
+import java.util.function.BiConsumer;
+
+import org.jspecify.annotations.NonNull;
+
 import org.apache.causeway.applib.annotation.PromptStyle;
-import org.apache.causeway.core.metamodel.facets.SingleValueFacet;
+import org.apache.causeway.core.config.CausewayConfiguration;
+import org.apache.causeway.core.metamodel.facetapi.Facet;
+import org.apache.causeway.core.metamodel.facetapi.FacetHolder;
 
 /**
- * Indicates that the editing of the property should be published.
+ * Provides the prompt style for editing of a property.
  */
-public interface PromptStyleFacet extends SingleValueFacet<PromptStyle> {
+public record PromptStyleFacet(
+       @NonNull String origin,
+       @NonNull PromptStyle value,
+       @NonNull FacetHolder facetHolder,
+       Facet.@NonNull Precedence precedence,
+       boolean isObjectTypeSpecific
+   ) implements Facet {
+
+   public static PromptStyleFacet compositeValueEdit(final FacetHolder facetHolder) {
+       return new PromptStyleFacet("CompositeValueEdit", PromptStyle.INLINE_AS_IF_EDIT, facetHolder, Precedence.SYNTHESIZED, false);
+   }
+   public static PromptStyleFacet asConfgured(final CausewayConfiguration configuration, final FacetHolder facetHolder) {
+       return new PromptStyleFacet("Configuration", configuration.getViewer().getWicket().getPromptStyle(), facetHolder, Precedence.DEFAULT, false);
+   }
+
+   @Override public Class<? extends Facet> facetType() { return getClass(); }
+   @Override public Precedence getPrecedence() { return precedence(); }
+   @Override public FacetHolder getFacetHolder() { return facetHolder(); }
+
+   public PromptStyleFacet(final String origin, final PromptStyle of, final FacetHolder holder) {
+       this(origin, of, holder, Precedence.DEFAULT, false);
+   }
+
+   public PromptStyleFacet(final String origin, final PromptStyle of, final FacetHolder holder, final boolean isObjectTypeSpecific) {
+       this(origin, of, holder, Precedence.DEFAULT, isObjectTypeSpecific);
+   }
 
-    @Override
-    PromptStyle value();
+   @Override
+   public void visitAttributes(final BiConsumer<String, Object> visitor) {
+       visitor.accept("origin", origin());
+       visitor.accept("precedence", getPrecedence().name());
+       visitor.accept("promptStyle", value);
+   }
 
 }