opendev/gerrit
Code Issues Proposed changes

Merge "Drop the user provided project name's trailing '.git' suffix"

Browse Source
This commit is contained in:
David Pursehouse
2013-01-07 20:58:04 -08:00
committed by Gerrit Code Review
parent be59d97cad 3e5aba87e8
commit 4a0b044a83
1 changed files with 17 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/CreateProjectScreen.java
View File

@@ -46,6 +46,8 @@ import com.google.gwt.user.client.ui.VerticalPanel;
import com.google.gwtexpui.globalkey.client.NpTextBox;
import com.google.gwtjsonrpc.common.VoidResult;
import org.eclipse.jgit.lib.Constants;
public class CreateProjectScreen extends Screen {
private Grid grid;
private NpTextBox project;
@@ -235,8 +237,22 @@ public class CreateProjectScreen extends Screen {
new GerritCallback<VoidResult>() {
@Override
public void onSuccess(VoidResult result) {
String nameWithoutSuffix = projectName;
if (nameWithoutSuffix.endsWith(Constants.DOT_GIT_EXT)) {
// Be nice and drop the trailing ".git" suffix, which we never
// keep in our database, but clients might mistakenly provide
// anyway.
//
nameWithoutSuffix = nameWithoutSuffix.substring(0, //
nameWithoutSuffix.length() - Constants.DOT_GIT_EXT.length());
while (nameWithoutSuffix.endsWith("/")) {
nameWithoutSuffix = nameWithoutSuffix.substring(//
0, nameWithoutSuffix.length() - 1);
}
}
History.newItem(Dispatcher.toProjectAdmin(new Project.NameKey(
projectName), ProjectScreen.INFO));
nameWithoutSuffix), ProjectScreen.INFO));
}
@Override