adobeGH-24 add getRule API

npeltier · Sep 7, 2020 · 2f2a1c9 · 2f2a1c9
1 parent ea9b253
commit 2f2a1c9
Show file tree

Hide file tree

Showing 5 changed files with 33 additions and 3 deletions.
diff --git a/bundles/core/src/main/java/com/adobe/dx/style/StyleWorker.java b/bundles/core/src/main/java/com/adobe/dx/style/StyleWorker.java
@@ -39,4 +39,13 @@ public interface StyleWorker {
      */
     @Nullable String getDeclaration(@Nullable Breakpoint breakpoint, SlingHttpServletRequest request);
 
+    /**
+     * Generates a rule specific to that generator
+     *
+     * @param breakpoint if rule is specific to it, null otherwise
+     * @param id rules can reuse in format <code>${id}</code> current
+     * @param request current request
+     * @return one or several rules
+     */
+    @Nullable String getRule(@Nullable Breakpoint breakpoint, @Nullable String id, SlingHttpServletRequest request);
 }
diff --git a/bundles/core/src/main/java/com/adobe/dx/style/internal/Border.java b/bundles/core/src/main/java/com/adobe/dx/style/internal/Border.java
@@ -89,6 +89,12 @@ public String getKey() {
         return null;
     }
 
+    @Override
+    public @Nullable String getRule(@Nullable Breakpoint breakpoint, @Nullable String id,
+                                    SlingHttpServletRequest request) {
+        return null;
+    }
+
     private String buildBorder(ValueMap policy) {
         String borderSides = policy.get(PN_SIDES, String.class);
         if (StringUtils.equals(ALL, borderSides)) {

diff --git a/bundles/core/src/main/java/com/adobe/dx/style/internal/Shadow.java b/bundles/core/src/main/java/com/adobe/dx/style/internal/Shadow.java
@@ -68,4 +68,10 @@ public String getKey() {
         }
         return null;
     }
+
+    @Override
+    public @Nullable String getRule(@Nullable Breakpoint breakpoint, @Nullable String id,
+                                    SlingHttpServletRequest request) {
+        return null;
+    }
 }
diff --git a/bundles/core/src/main/java/com/adobe/dx/style/internal/StyleServiceImpl.java b/bundles/core/src/main/java/com/adobe/dx/style/internal/StyleServiceImpl.java
@@ -20,7 +20,6 @@
 import static com.day.cq.wcm.commons.Constants.EMPTY_STRING_ARRAY;
 import static org.apache.commons.lang3.StringUtils.EMPTY;
 
-import com.adobe.dx.bindings.internal.DxBindingsValueProvider;
 import com.adobe.dx.responsive.Breakpoint;
 import com.adobe.dx.style.StyleWorker;
 import com.adobe.dx.style.StyleService;
@@ -33,10 +32,8 @@
 
 import org.apache.commons.collections.MapUtils;
 import org.apache.commons.lang.StringUtils;
-import org.apache.poi.hssf.record.PageBreakRecord;
 import org.apache.sling.api.SlingHttpServletRequest;
 import org.apache.sling.api.resource.Resource;
-import org.apache.sling.api.scripting.SlingBindings;
 import org.jetbrains.annotations.NotNull;
 import org.osgi.service.component.annotations.Component;
 import org.osgi.service.component.annotations.ConfigurationPolicy;

diff --git a/bundles/core/src/test/java/com/adobe/dx/style/internal/StyleServiceImplTest.java b/bundles/core/src/test/java/com/adobe/dx/style/internal/StyleServiceImplTest.java
@@ -68,6 +68,12 @@ public String getKey() {
                 return getFromRequest("min-height", breakpoint.key(), "minheight", request);
             }
         }
+
+        @Override
+        public @Nullable String getRule(@Nullable Breakpoint breakpoint, @Nullable String id,
+                                        SlingHttpServletRequest request) {
+            return null;
+        }
     };
 
     StyleWorker worker2 = new StyleWorker() {
@@ -84,6 +90,12 @@ public String getKey() {
                 return getFromRequest("min-width", breakpoint.key(), "minwidth", request);
             }
         }
+
+        @Override
+        public @Nullable String getRule(@Nullable Breakpoint breakpoint, @Nullable String id,
+                                        SlingHttpServletRequest request) {
+            return null;
+        }
     };
     StyleServiceImpl service;