/*
 * Copyright 2012-2014 Continuuity, Inc.
 *
 * 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 co.cask.cdap.shell.completer.reactor;

import co.cask.cdap.api.dataset.DatasetSpecification;
import co.cask.cdap.client.DatasetClient;
import co.cask.cdap.shell.completer.StringsCompleter;
import com.google.common.base.Function;
import com.google.common.base.Supplier;
import com.google.common.collect.Iterables;
import com.google.common.collect.Lists;

import java.io.IOException;
import java.util.Collection;
import java.util.List;
import javax.inject.Inject;

/**
 * Completer for dataset names.
 */
public class DatasetNameCompleter extends StringsCompleter {

  @Inject
  public DatasetNameCompleter(final DatasetClient datasetClient) {
    super(new Supplier<Collection<String>>() {
      @Override
      public Collection<String> get() {
        try {
          List<DatasetSpecification> list = datasetClient.list();
          return Lists.newArrayList(
            Iterables.transform(list, new Function<DatasetSpecification, String>() {
              @Override
              public String apply(DatasetSpecification input) {
                // TODO: hack to handle namespaced dataset names
                if (input.getName().startsWith("continuuity.user.")) {
                  return input.getName().substring("continuuity.user.".length());
                } else {
                  return input.getName();
                }
              }
            })
          );
        } catch (IOException e) {
          return Lists.newArrayList();
        }
      }
    });
  }
}

@@ -1707,246 +1707,171 @@
 ames
-%0A                if (input.getName().startsWith(%22continuuity.user.%22)) %7B%0A                  return input.getName().substring(%22continuuity.user.%22.length());%0A                %7D else %7B%0A                  return input.getName();%0A                %7D
+ -- assumes there are no periods in dataset names%0A                String%5B%5D tokens = input.getName().split(%22%5C%5C.%22);%0A                return tokens%5Btokens.length - 1%5D;
 %0A   

Change DatasetNameCompleter to handle namespaced and non-namespaced dataset names better