0.12.x selector fix

annotations/halkyon-annotations/src/main/java/io/dekorate/halkyon/handler/ComponentHandler.java

@@ -117,7 +117,6 @@ private void addVisitors(ComponentConfig config) {
     Map<String, String> allLabels = new HashMap<>();
     allLabels.put(Labels.NAME, config.getName());
 
-    System.out.println("Component version:" + config.getVersion()+".");
     if (Strings.isNotNullOrEmpty(config.getVersion())) {
         allLabels.put(Labels.VERSION, config.getVersion());
     }

annotations/kubernetes-annotations/src/main/java/io/dekorate/kubernetes/decorator/ApplyDeploymentStrategyDecorator.java

@@ -38,7 +38,6 @@ public ApplyDeploymentStrategyDecorator(String name, DeploymentStrategy strategy
   @Override
   public void andThenVisit(final DeploymentSpecFluent<?> spec, final ObjectMeta resourceMeta) {
     boolean hasCustomRollingUpdate = hasCusomRollingUpdateConfig(rollingUpdate);
-    System.out.println("Custom rolling update:" + hasCustomRollingUpdate);
     if (strategy == DeploymentStrategy.Recreate) {
       if (hasCustomRollingUpdate) {
         throw new IllegalStateException("Detected both Recreate strategy and custom Rolling Update config. Please use one or the other!");
@@ -47,7 +46,6 @@ public void andThenVisit(final DeploymentSpecFluent<?> spec, final ObjectMeta re
         .withType("Recreate")
         .endStrategy();
     } else if (strategy == DeploymentStrategy.RollingUpdate || hasCustomRollingUpdate) {
-      System.out.println("Applying rolling update!");
       spec.withNewStrategy()
         .withType("RollingUpdate")
         .withNewRollingUpdate()

core/src/main/java/io/dekorate/kubernetes/decorator/AddToMatchingLabelsDecorator.java

@@ -8,6 +8,10 @@ public class AddToMatchingLabelsDecorator extends NamedResourceDecorator<LabelSe
 
   private String key;
   private String value;
+
+ 	public AddToMatchingLabelsDecorator(String key, String value) {
+    this(ANY, key, value);
+  }
 
  	public AddToMatchingLabelsDecorator(String name, String key, String value) {
 		super(name);
@@ -21,10 +25,10 @@ public AddToMatchingLabelsDecorator(String kind, String name, String key, String
 		this.value = value;
 	}   
 
-	@Override
-	public void andThenVisit(LabelSelectorFluent<?> selector, ObjectMeta resourceMeta) {
+  @Override
+  public void andThenVisit(LabelSelectorFluent<?> selector, ObjectMeta resourceMeta) {
     selector.addToMatchLabels(key, value);
-	}
+  }
 
   @Override
   public Class<? extends Decorator>[] after() {

core/src/main/java/io/dekorate/kubernetes/decorator/AddToSelectorDecorator.java

@@ -22,6 +22,7 @@
 import io.dekorate.SelectorDecoratorFactories;
 import io.dekorate.SelectorDecoratorFactory;
 import io.dekorate.deps.kubernetes.api.builder.VisitableBuilder;
+import io.dekorate.deps.kubernetes.api.builder.Visitor;
 import io.dekorate.deps.kubernetes.api.model.ObjectMeta;
 
 public class AddToSelectorDecorator extends NamedResourceDecorator<VisitableBuilder> {
@@ -51,11 +52,17 @@ public AddToSelectorDecorator(String kind, String name, String key, String value
 	@Override
 	public void andThenVisit(VisitableBuilder builder ,String kind, ObjectMeta resourceMeta) {
     Optional<SelectorDecoratorFactory> factory = SelectorDecoratorFactories.find(kind);
-    factory.ifPresent(f -> f.createAddToSelectorDecorator(resourceMeta.getName(), key, value));
+    factory.map(f -> f.createAddToSelectorDecorator(resourceMeta.getName(), key, value)).ifPresent(m -> builder.accept((Visitor) m));
 	}
 
-	@Override
+  @Override
 	public void andThenVisit(VisitableBuilder item, ObjectMeta resourceMeta) {
     //Not needed
 	}
+
+	@Override
+  public Class<? extends Decorator>[] before() {
+    return new Class[] {AddToSelectorDecorator.class};
+  }
+
 }

core/src/main/java/io/dekorate/kubernetes/decorator/RemoveFromMatchingLabelsDecorator.java

@@ -7,6 +7,10 @@ public class RemoveFromMatchingLabelsDecorator extends NamedResourceDecorator<La
 
   private String key;
 
+  public RemoveFromMatchingLabelsDecorator(String key) {
+    this(ANY, key);
+  }
+
   public RemoveFromMatchingLabelsDecorator(String name, String key) {
     super(name);
     this.key = key;

core/src/main/java/io/dekorate/kubernetes/decorator/RemoveFromSelectorDecorator.java

@@ -22,12 +22,17 @@
 import io.dekorate.SelectorDecoratorFactories;
 import io.dekorate.SelectorDecoratorFactory;
 import io.dekorate.deps.kubernetes.api.builder.VisitableBuilder;
+import io.dekorate.deps.kubernetes.api.builder.Visitor;
 import io.dekorate.deps.kubernetes.api.model.ObjectMeta;
 
 public class RemoveFromSelectorDecorator extends NamedResourceDecorator<VisitableBuilder> {
 
   private final String key;
 
+	public RemoveFromSelectorDecorator(String key) {
+    this(ANY, key);
+	}
+
 	public RemoveFromSelectorDecorator(String name, String key) {
 		super(name);
 		this.key = key;
@@ -42,11 +47,16 @@ public RemoveFromSelectorDecorator(String kind, String name, String key) {
 	@Override
 	public void andThenVisit(VisitableBuilder builder ,String kind, ObjectMeta resourceMeta) {
     Optional<SelectorDecoratorFactory> factory = SelectorDecoratorFactories.find(kind);
-    factory.ifPresent(f -> f.createRemoveFromSelectorDecorator(resourceMeta.getName(), key));
+    factory.map(f -> f.createRemoveFromSelectorDecorator(resourceMeta.getName(), key)).ifPresent(m -> builder.accept((Visitor) m));
 	}
 
 	@Override
 	public void andThenVisit(VisitableBuilder item, ObjectMeta resourceMeta) {
     //Not needed
 	}
+
+	@Override
+  public Class<? extends Decorator>[] after() {
+    return new Class[] {ResourceProvidingDecorator.class, AddToSelectorDecorator.class};
+  }
 }