Merge pull request #11248 from gytis/rest-data-fail-hal-without-seria…

…liser Fail build with HAL resource without serializer #11246
quarkusio · Mar 15, 2021 · 087b300 · 087b300
2 parents 7a2b463 + bda2e56
commit 087b300
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 0 deletions.
diff --git a/core/deployment/src/main/java/io/quarkus/deployment/Capability.java b/core/deployment/src/main/java/io/quarkus/deployment/Capability.java
@@ -30,6 +30,8 @@ public enum Capability {
     RESTEASY_JSON,
     RESTEASY_MUTINY,
     RESTEASY_REACTIVE,
+    RESTEASY_JACKSON,
+    RESTEASY_JSONB,
     JWT,
     TIKA,
     MONGODB_PANACHE,

diff --git a/...e/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/RestDataProcessor.java b/...e/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/RestDataProcessor.java
@@ -50,6 +50,10 @@ void implementResources(CombinedIndexBuildItem index, List<RestDataResourceBuild
             ResourceMetadata resourceMetadata = resourceBuildItem.getResourceMetadata();
             ResourceProperties resourceProperties = getResourceProperties(resourcePropertiesProvider,
                     resourceMetadata, resourcePropertiesBuildItems);
+            if (resourceProperties.isHal() && !hasHalCapability(capabilities)) {
+                throw new IllegalStateException(
+                        "Cannot generate HAL endpoints without a RESTEasy JSON-B or Jackson capability");
+            }
             if (resourceProperties.isExposed()) {
                 jaxRsResourceImplementor.implement(classOutput, resourceMetadata, resourceProperties);
             }
@@ -93,4 +97,8 @@ private ResourceProperties getResourceProperties(ResourcePropertiesProvider reso
     private boolean hasValidatorCapability(Capabilities capabilities) {
         return capabilities.isPresent(Capability.HIBERNATE_VALIDATOR);
     }
+
+    private boolean hasHalCapability(Capabilities capabilities) {
+        return capabilities.isPresent(Capability.RESTEASY_JSONB) || capabilities.isPresent(Capability.RESTEASY_JACKSON);
+    }
 }
diff --git a/...oyment/src/main/java/io/quarkus/resteasy/jackson/deployment/ResteasyJacksonProcessor.java b/...oyment/src/main/java/io/quarkus/resteasy/jackson/deployment/ResteasyJacksonProcessor.java
@@ -18,5 +18,6 @@ void feature(BuildProducer<FeatureBuildItem> feature) {
     void capabilities(BuildProducer<CapabilityBuildItem> capability) {
         capability.produce(new CapabilityBuildItem(Capability.RESTEASY_JSON));
         capability.produce(new CapabilityBuildItem(Capability.REST_JACKSON));
+        capability.produce(new CapabilityBuildItem(Capability.RESTEASY_JACKSON));
     }
 }
diff --git a/...deployment/src/main/java/io/quarkus/resteasy/jsonb/deployment/ResteasyJsonbProcessor.java b/...deployment/src/main/java/io/quarkus/resteasy/jsonb/deployment/ResteasyJsonbProcessor.java
@@ -32,6 +32,7 @@ void feature(BuildProducer<FeatureBuildItem> feature) {
     void capabilities(BuildProducer<CapabilityBuildItem> capability) {
         capability.produce(new CapabilityBuildItem(Capability.RESTEASY_JSON));
         capability.produce(new CapabilityBuildItem(Capability.REST_JSONB));
+        capability.produce(new CapabilityBuildItem(Capability.RESTEASY_JSONB));
     }
 
     @BuildStep