// Copyright (C) 2012 The Android Open Source Project
//
// 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 com.google.gerrit.server.args4j;

import static com.google.gerrit.util.cli.Localizable.localizable;

import com.google.common.base.Splitter;
import com.google.gerrit.reviewdb.client.Branch;
import com.google.gerrit.reviewdb.client.Change;
import com.google.gerrit.reviewdb.client.Project;
import com.google.gerrit.server.query.change.ChangeData;
import com.google.gerrit.server.query.change.InternalChangeQuery;
import com.google.gwtorm.server.OrmException;
import com.google.inject.Inject;
import com.google.inject.Provider;
import com.google.inject.assistedinject.Assisted;
import java.util.List;
import org.kohsuke.args4j.CmdLineException;
import org.kohsuke.args4j.CmdLineParser;
import org.kohsuke.args4j.OptionDef;
import org.kohsuke.args4j.spi.OptionHandler;
import org.kohsuke.args4j.spi.Parameters;
import org.kohsuke.args4j.spi.Setter;

public class ChangeIdHandler extends OptionHandler<Change.Id> {
  private final Provider<InternalChangeQuery> queryProvider;

  @Inject
  public ChangeIdHandler(
      // TODO(dborowitz): Not sure whether this is injectable here.
      Provider<InternalChangeQuery> queryProvider,
      @Assisted CmdLineParser parser,
      @Assisted OptionDef option,
      @Assisted Setter<Change.Id> setter) {
    super(parser, option, setter);
    this.queryProvider = queryProvider;
  }

  @Override
  public final int parseArguments(Parameters params) throws CmdLineException {
    String token = params.getParameter(0);
    List<String> tokens = Splitter.on(',').splitToList(token);
    if (tokens.size() != 3) {
      throw new CmdLineException(
          owner, localizable("change should be specified as <project>,<branch>,<change-id>"));
    }

    try {
      Change.Key key = Change.Key.parse(tokens.get(2));
      Project.NameKey project = new Project.NameKey(tokens.get(0));
      Branch.NameKey branch = new Branch.NameKey(project, tokens.get(1));
      for (ChangeData cd : queryProvider.get().byBranchKey(branch, key)) {
        setter.addValue(cd.getId());
        return 1;
      }
    } catch (IllegalArgumentException e) {
      throw new CmdLineException(owner, localizable("Change-Id is not valid: %s"), e.getMessage());
    } catch (OrmException e) {
      throw new CmdLineException(owner, localizable("Database error: %s"), e.getMessage());
    }

    throw new CmdLineException(owner, localizable("\"%s\": change not found"), token);
  }

  @Override
  public final String getDefaultMetaVariable() {
    return "CHANGE";
  }
}

@@ -747,24 +747,71 @@
 e.Splitter;%0A
+import com.google.common.flogger.FluentLogger;%0A
 import com.g
@@ -847,16 +847,16 @@
 Branch;%0A
-
 import c
@@ -1568,16 +1568,96 @@
 e.Id%3E %7B%0A
+  private static final FluentLogger logger = FluentLogger.forEnclosingClass();%0A%0A
   privat
@@ -2642,13 +2642,13 @@
     
-for (
+List%3C
 Chan
@@ -2657,13 +2657,19 @@
 Data
+%3E
  c
-d :
+hanges =
  que
@@ -2709,19 +2709,271 @@
 ch, key)
-) %7B
+;%0A      if (!changes.isEmpty()) %7B%0A        if (changes.size() %3E 1) %7B%0A          String msg = %22%5C%22%25s%5C%22: resolves to multiple changes%22;%0A          logger.atSevere().log(msg, token);%0A          throw new CmdLineException(owner, localizable(msg), token);%0A        %7D
 %0A       
@@ -2990,17 +2990,29 @@
 dValue(c
-d
+hanges.get(0)
 .getId()

Make assumption on number of query results explicit