/*
 * Copyright 2014 Open Networking Laboratory
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.onlab.onos.cli.net;

import java.util.LinkedList;
import java.util.List;

import org.apache.karaf.shell.commands.Option;
import org.onlab.onos.cli.AbstractShellCommand;
import org.onlab.onos.net.flow.DefaultTrafficSelector;
import org.onlab.onos.net.flow.TrafficSelector;
import org.onlab.onos.net.intent.Constraint;
import org.onlab.onos.net.intent.constraint.BandwidthConstraint;
import org.onlab.onos.net.intent.constraint.LambdaConstraint;
import org.onlab.onos.net.resource.Bandwidth;
import org.onlab.packet.Ethernet;
import org.onlab.packet.MacAddress;

import static com.google.common.base.Strings.isNullOrEmpty;

/**
 * Base class for command line operations for connectivity based intents.
 */
public abstract class ConnectivityIntentCommand extends AbstractShellCommand {

    @Option(name = "-s", aliases = "--ethSrc", description = "Source MAC Address",
            required = false, multiValued = false)
    private String srcMacString = null;

    @Option(name = "-d", aliases = "--ethDst", description = "Destination MAC Address",
            required = false, multiValued = false)
    private String dstMacString = null;

    @Option(name = "-t", aliases = "--ethType", description = "Ethernet Type",
            required = false, multiValued = false)
    private String ethTypeString = "";

    @Option(name = "-b", aliases = "--bandwidth", description = "Bandwidth",
            required = false, multiValued = false)
    private String bandwidthString = "";

    @Option(name = "-l", aliases = "--lambda", description = "Lambda",
            required = false, multiValued = false)
    private boolean lambda = false;

    /**
     * Constructs a traffic selector based on the command line arguments
     * presented to the command.
     * @return traffic selector
     */
    protected TrafficSelector buildTrafficSelector() {
        TrafficSelector.Builder selectorBuilder = DefaultTrafficSelector.builder();
        Short ethType = Ethernet.TYPE_IPV4;

        if (!isNullOrEmpty(ethTypeString)) {
            EthType ethTypeParameter = EthType.valueOf(ethTypeString);
            ethType = ethTypeParameter.value();
        }
        selectorBuilder.matchEthType(ethType);

        if (!isNullOrEmpty(srcMacString)) {
            selectorBuilder.matchEthSrc(MacAddress.valueOf(srcMacString));
        }

        if (!isNullOrEmpty(dstMacString)) {
            selectorBuilder.matchEthDst(MacAddress.valueOf(dstMacString));
        }

        return selectorBuilder.build();
    }

    /**
     * Builds the constraint list for this command based on the command line
     * parameters.
     *
     * @return List of constraint objects describing the constraints requested
     */
    protected List<Constraint> buildConstraints() {
        final List<Constraint> constraints = new LinkedList<>();

        // Check for a bandwidth specification
        if (!isNullOrEmpty(bandwidthString)) {
            final double bandwidthValue = Double.parseDouble(bandwidthString);
            constraints.add(new BandwidthConstraint(Bandwidth.valueOf(bandwidthValue)));
        }

        // Check for a lambda specification
        if (lambda) {
            constraints.add(new LambdaConstraint(null));
        }

        return constraints;
    }
}

@@ -1139,16 +1139,50 @@
 hernet;%0A
+import org.onlab.packet.IpPrefix;%0A
 import o
@@ -1209,16 +1209,16 @@
 ddress;%0A
-
 %0Aimport 
@@ -1950,32 +1950,820 @@
 peString = %22%22;%0A%0A
+    @Option(name = %22--ipProto%22, description = %22IP Protocol%22,%0A            required = false, multiValued = false)%0A    private String ipProtoString = null;%0A%0A    @Option(name = %22--ipSrc%22, description = %22Source IP Address%22,%0A            required = false, multiValued = false)%0A    private String srcIpString = null;%0A%0A    @Option(name = %22--ipDst%22, description = %22Destination IP Address%22,%0A            required = false, multiValued = false)%0A    private String dstIpString = null;%0A%0A    @Option(name = %22--tcpSrc%22, description = %22Source TCP Port%22,%0A            required = false, multiValued = false)%0A    private String srcTcpString = null;%0A%0A    @Option(name = %22--tcpDst%22, description = %22Destination TCP Port%22,%0A            required = false, multiValued = false)%0A    private String dstTcpString = null;%0A%0A
     @Option(name
@@ -3801,32 +3801,32 @@
 stMacString)) %7B%0A
-
             sele
@@ -3887,32 +3887,695 @@
 g));%0A        %7D%0A%0A
+        if (!isNullOrEmpty(ipProtoString)) %7B%0A            selectorBuilder.matchIPProtocol((byte) Short.parseShort(ipProtoString));%0A        %7D%0A%0A        if (!isNullOrEmpty(srcIpString)) %7B%0A            selectorBuilder.matchIPSrc(IpPrefix.valueOf(srcIpString));%0A        %7D%0A%0A        if (!isNullOrEmpty(dstIpString)) %7B%0A            selectorBuilder.matchIPDst(IpPrefix.valueOf(dstIpString));%0A        %7D%0A%0A        if (!isNullOrEmpty(srcTcpString)) %7B%0A            selectorBuilder.matchTcpSrc((short) Integer.parseInt(srcTcpString));%0A        %7D%0A%0A        if (!isNullOrEmpty(dstTcpString)) %7B%0A            selectorBuilder.matchTcpSrc((short) Integer.parseInt(dstTcpString));%0A        %7D%0A%0A
         return s

Add some more match conditions to connectivity intent CLI commands