// Copyright (C) 2009 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 com.google.gerrit.reviewdb.client.Project;
import com.google.gerrit.server.project.NoSuchProjectException;
import com.google.gerrit.server.project.ProjectControl;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;

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 ProjectControlHandler extends OptionHandler<ProjectControl> {
  private final ProjectControl.Factory projectControlFactory;

  @Inject
  public ProjectControlHandler(
      final ProjectControl.Factory projectControlFactory,
      @Assisted final CmdLineParser parser, @Assisted final OptionDef option,
      @Assisted final Setter<ProjectControl> setter) {
    super(parser, option, setter);
    this.projectControlFactory = projectControlFactory;
  }

  @Override
  public final int parseArguments(final Parameters params)
      throws CmdLineException {
    final String token = params.getParameter(0);
    String projectName = token;

    while (projectName.endsWith("/")) {
      projectName = projectName.substring(0, projectName.length() - 1);
    }

    if (projectName.endsWith(".git")) {
      // Be nice and drop the trailing ".git" suffix, which we never keep
      // in our database, but clients might mistakenly provide anyway.
      //
      projectName = projectName.substring(0, projectName.length() - 4);
      while (projectName.endsWith("/")) {
        projectName = projectName.substring(0, projectName.length() - 1);
      }
    }

    while (projectName.startsWith("/")) {
      // Be nice and drop the leading "/" if supplied by an absolute path.
      // We don't have a file system hierarchy, just a flat namespace in
      // the database's Project entities. We never encode these with a
      // leading '/' but users might accidentally include them in Git URLs.
      //
      projectName = projectName.substring(1);
    }

    final ProjectControl control;
    try {
      Project.NameKey nameKey = new Project.NameKey(projectName);
      control = projectControlFactory.validateFor(nameKey, ProjectControl.OWNER | ProjectControl.VISIBLE);
    } catch (NoSuchProjectException e) {
      throw new CmdLineException(owner, "'" + token + "': not a Gerrit project");
    }

    setter.addValue(control);
    return 1;
  }

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

@@ -898,16 +898,55 @@
 isted;%0A%0A
+import org.eclipse.jgit.lib.Constants;%0A
 import o
@@ -1763,26 +1763,26 @@
     
-final 
 String 
-token
+projectName
  = p
@@ -1807,40 +1807,8 @@
 (0);
-%0A    String projectName = token;
 %0A%0A  
@@ -1928,405 +1928,8 @@
  %7D%0A%0A
-    if (projectName.endsWith(%22.git%22)) %7B%0A      // Be nice and drop the trailing %22.git%22 suffix, which we never keep%0A      // in our database, but clients might mistakenly provide anyway.%0A      //%0A      projectName = projectName.substring(0, projectName.length() - 4);%0A      while (projectName.endsWith(%22/%22)) %7B%0A        projectName = projectName.substring(0, projectName.length() - 1);%0A      %7D%0A    %7D%0A%0A
     
@@ -2319,24 +2319,582 @@
 (1);%0A    %7D%0A%0A
+    String nameWithoutSuffix = projectName;%0A    if (nameWithoutSuffix.endsWith(Constants.DOT_GIT_EXT)) %7B%0A      // Be nice and drop the trailing %22.git%22 suffix, which we never keep%0A      // in our database, but clients might mistakenly provide anyway.%0A      //%0A      nameWithoutSuffix = nameWithoutSuffix.substring(0, //%0A          nameWithoutSuffix.length() - Constants.DOT_GIT_EXT.length());%0A      while (nameWithoutSuffix.endsWith(%22/%22)) %7B%0A        nameWithoutSuffix =%0A            nameWithoutSuffix.substring(0, nameWithoutSuffix.length() - 1);%0A      %7D%0A    %7D%0A%0A
     final Pr
@@ -2977,27 +2977,33 @@
 NameKey(
-projectName
+nameWithoutSuffix
 );%0A     
@@ -3195,19 +3195,28 @@
 %22 + 
-token
+projectName
  + %22':
+ is
  not

Polish the error message about wrong project name