Adding support for multiple classifiers per gate

[packetcable.git] / packetcable-policy-server / src / main / java / org / opendaylight / controller / packetcable / provider / validation / impl / validators / qos / GateValidator.java

diff --git a/packetcable-policy-server/src/main/java/org/opendaylight/controller/packetcable/provider/validation/impl/validators/qos/GateValidator.java b/packetcable-policy-server/src/main/java/org/opendaylight/controller/packetcable/provider/validation/impl/validators/qos/GateValidator.java
index e97974cc501c51a187caad107e46ad1e28cc7830..fcfaa6a2e8394a57a2c917cb8f14bc31e2f2bb15 100644 (file)
--- a/packetcable-policy-server/src/main/java/org/opendaylight/controller/packetcable/provider/validation/impl/validators/qos/GateValidator.java
+++ b/packetcable-policy-server/src/main/java/org/opendaylight/controller/packetcable/provider/validation/impl/validators/qos/GateValidator.java
@@ -10,10 +10,8 @@ package org.opendaylight.controller.packetcable.provider.validation.impl.validat
 
 import org.opendaylight.controller.packetcable.provider.validation.ValidationException;
 import org.opendaylight.controller.packetcable.provider.validation.impl.validators.AbstractValidator;
-import org.opendaylight.controller.packetcable.provider.validation.impl.validators.qos.classifier.ClassifierValidator;
-import org.opendaylight.controller.packetcable.provider.validation.impl.validators.qos.classifier.ExtClassifierValidator;
-import org.opendaylight.controller.packetcable.provider.validation.impl.validators.qos.classifier.Ipv6ClassifierValidator;
-import org.opendaylight.yang.gen.v1.urn.packetcable.rev151026.pcmm.qos.gates.apps.app.subscribers.subscriber.gates.Gate;
+import org.opendaylight.controller.packetcable.provider.validation.impl.validators.qos.classifier.ClassifiersValidator;
+import org.opendaylight.yang.gen.v1.urn.packetcable.rev151101.pcmm.qos.gates.apps.app.subscribers.subscriber.gates.Gate;
 
 /**
  * @author rvail
@@ -23,12 +21,11 @@ public class GateValidator extends AbstractValidator<Gate> {
     private static final String GATE_ID = "gate.gateId";
     private static final String GATE_SPEC = "gate.gate-spec";
     private static final String TRAFFIC_PROFILE = "gate.traffic-profile";
+    private static final String CLASSIFIERS = "gate.classifiers";
 
 //    private final GateSpecValidatator gateSpecValidatator = new GateSpecValidatator();
     private final TrafficProfileValidator trafficProfileValidator = new TrafficProfileValidator();
-    private final ClassifierValidator classifierValidator = new ClassifierValidator();
-    private final ExtClassifierValidator extClassifierValidator = new ExtClassifierValidator();
-    private final Ipv6ClassifierValidator ipv6ClassifierValidator = new Ipv6ClassifierValidator();
+    private final ClassifiersValidator classifiersValidator = new ClassifiersValidator();
 
     @Override
     public void validate(final Gate gate, final Extent extent) throws ValidationException {
@@ -41,48 +38,13 @@ public class GateValidator extends AbstractValidator<Gate> {
         // all leafs in GateSpec are optional
         // mustExist(gate.getGateSpec(), GATE_SPEC);
 
+        // Classifiers
+        mustExist(gate.getClassifiers(), CLASSIFIERS);
+
         mustExist(gate.getTrafficProfile(), TRAFFIC_PROFILE);
         if (extent == Extent.NODE_AND_SUBTREE) {
-//            validateChild(gateSpecValidatator, gate.getGateSpec());
             validateChild(trafficProfileValidator, gate.getTrafficProfile());
-        }
-
-        // Classifiers
-
-        if (gate.getClassifier() != null) {
-
-            // classifer is not null, ext and ipv6 must be null
-
-            if (gate.getExtClassifier() != null || gate.getIpv6Classifier() != null) {
-                getErrorMessages().add("Only one type of classifier is allowed");
-            }
-            else if (extent == Extent.NODE_AND_SUBTREE) {
-                validateChild(classifierValidator, gate.getClassifier());
-            }
-
-        }
-        else if (gate.getExtClassifier() != null) {
-
-            // classifer is null; ext is not null and ipv6 must be null
-
-            if (gate.getIpv6Classifier() != null) {
-                getErrorMessages().add("Only one type of classifier is allowed");
-            }
-            else if (extent == Extent.NODE_AND_SUBTREE) {
-                validateChild(extClassifierValidator, gate.getExtClassifier());
-            }
-
-        }
-        else if (gate.getIpv6Classifier() != null) {
-
-            // classifer and ext are null; ipv6 is not
-            if (extent == Extent.NODE_AND_SUBTREE) {
-                validateChild(ipv6ClassifierValidator, gate.getIpv6Classifier());
-            }
-
-        }
-        else {
-            getErrorMessages().add("a classifer is required");
+            validateChild(classifiersValidator, gate.getClassifiers());
         }
 
         throwErrorsIfNeeded();