Refactor GitCloneWizard

Extract reusable code to an abstract super class.

Bug: 361251
Change-Id: I94056fc75acde4e29b72dd5284cac97101b64e86
Signed-off-by: Stefan Lay <stefan.lay@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

2 files changed, 394 insertions, 295 deletions

diff --git a/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/clone/AbstractGitCloneWizard.java b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/clone/AbstractGitCloneWizard.java
new file mode 100644
index 0000000000..34c3b57eb1
--- /dev/null
+++ b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/clone/AbstractGitCloneWizard.java
@@ -0,0 +1,374 @@
+/*******************************************************************************
+ * Copyright (C) 2008, Roger C. Soares <rogersoares@intelinet.com.br>
+ * Copyright (C) 2008, Shawn O. Pearce <spearce@spearce.org>
+ * Copyright (C) 2008, Marek Zawirski <marek.zawirski@gmail.com>
+ * Copyright (C) 2008, Robin Rosenberg <robin.rosenberg@dewire.com>
+ * Copyright (C) 2010, Mathias Kinzler <mathias.kinzler@sap.com>
+ * Copyright (C) 2010, Benjamin Muskalla <bmuskalla@eclipsesource.com>
+ * Copyright (C) 2011, Stefan Lay <stefan.lay@sap.com>
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *******************************************************************************/
+package org.eclipse.egit.ui.internal.clone;
+
+import java.io.File;
+import java.io.IOException;
+import java.lang.reflect.InvocationTargetException;
+import java.text.MessageFormat;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.LinkedHashSet;
+import java.util.List;
+import java.util.Set;
+
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.IProgressMonitor;
+import org.eclipse.core.runtime.IStatus;
+import org.eclipse.core.runtime.Status;
+import org.eclipse.core.runtime.jobs.Job;
+import org.eclipse.egit.core.RepositoryUtil;
+import org.eclipse.egit.core.internal.util.ProjectUtil;
+import org.eclipse.egit.core.op.CloneOperation;
+import org.eclipse.egit.core.op.CloneOperation.PostCloneTask;
+import org.eclipse.egit.core.op.ConfigureFetchAfterCloneTask;
+import org.eclipse.egit.core.op.ConfigurePushAfterCloneTask;
+import org.eclipse.egit.core.op.ListRemoteOperation;
+import org.eclipse.egit.core.op.SetChangeIdTask;
+import org.eclipse.egit.core.securestorage.UserPasswordCredentials;
+import org.eclipse.egit.ui.Activator;
+import org.eclipse.egit.ui.JobFamilies;
+import org.eclipse.egit.ui.UIPreferences;
+import org.eclipse.egit.ui.UIText;
+import org.eclipse.egit.ui.internal.components.RepositorySelection;
+import org.eclipse.egit.ui.internal.credentials.EGitCredentialsProvider;
+import org.eclipse.jface.dialogs.ErrorDialog;
+import org.eclipse.jface.dialogs.MessageDialog;
+import org.eclipse.jface.operation.IRunnableWithProgress;
+import org.eclipse.jface.wizard.IWizardContainer;
+import org.eclipse.jface.wizard.Wizard;
+import org.eclipse.jgit.lib.Constants;
+import org.eclipse.jgit.lib.Ref;
+import org.eclipse.jgit.lib.Repository;
+import org.eclipse.jgit.storage.file.FileRepository;
+import org.eclipse.jgit.transport.URIish;
+import org.eclipse.jgit.transport.UsernamePasswordCredentialsProvider;
+import org.eclipse.osgi.util.NLS;
+import org.eclipse.ui.IWorkingSet;
+
+/**
+ * Implements the basic functionality of a clone wizard
+ */
+public abstract class AbstractGitCloneWizard extends Wizard {
+
+	/**
+	 * a page for branch selection
+	 */
+	protected SourceBranchPage validSource;
+
+	/**
+	 * a page for selection of the clone destination
+	 */
+	protected CloneDestinationPage cloneDestination;
+
+	/**
+	 * a page for some gerrit related parameters
+	 */
+	protected GerritConfigurationPage gerritConfiguration;
+
+	/**
+	 * the path where a clone has been created in
+	 */
+	protected String alreadyClonedInto;
+
+	/**
+	 * whether the clone operation is done later on by the caller of the wizard
+	 */
+	protected boolean callerRunsCloneOperation;
+
+	private CloneOperation cloneOperation;
+
+	/**
+	 * adds some basic functionality
+	 */
+	public AbstractGitCloneWizard() {
+		setNeedsProgressMonitor(true);
+		validSource = new SourceBranchPage() {
+
+			@Override
+			public void setVisible(boolean visible) {
+				if (visible) {
+					setSelection(getRepositorySelection());
+					setCredentials(getCredentials());
+				}
+				super.setVisible(visible);
+			}
+		};
+		cloneDestination = new CloneDestinationPage() {
+			@Override
+			public void setVisible(boolean visible) {
+				if (visible)
+					setSelection(getRepositorySelection(),
+							validSource.getAvailableBranches(),
+							validSource.getSelectedBranches(),
+							validSource.getHEAD());
+				super.setVisible(visible);
+			}
+		};
+	}
+
+	/**
+	 * subclasses may add pages to the Wizard which will be shown before the clone step
+	 */
+	protected abstract void addPreClonePages();
+
+	/**
+	 * subclasses may add pages to the Wizard which will be shown after the clone step
+	 */
+	protected abstract void addPostClonePages();
+
+	/**
+	 * @return the currently selected repository to clone
+	 */
+	protected abstract RepositorySelection getRepositorySelection();
+
+	/**
+	 * @return credentials
+	 */
+	protected abstract UserPasswordCredentials getCredentials();
+
+	@Override
+	final public void addPages() {
+		addPreClonePages();
+		addPage(validSource);
+		addPage(cloneDestination);
+		addPostClonePages();
+	}
+
+	/**
+	 * Do the clone using data which were collected on the pages
+	 * {@code validSource} and {@code cloneDestination}
+	 *
+	 * @param uri
+	 * @param credentials
+	 * @return if clone was successful
+	 */
+	protected boolean performClone(URIish uri,
+			UserPasswordCredentials credentials) {
+		setWindowTitle(NLS.bind(UIText.GitCloneWizard_jobName, uri.toString()));
+		final boolean allSelected;
+		final Collection<Ref> selectedBranches;
+		if (validSource.isSourceRepoEmpty()) {
+			// fetch all branches of empty repo
+			allSelected = true;
+			selectedBranches = Collections.emptyList();
+		} else {
+			allSelected = validSource.isAllSelected();
+			selectedBranches = validSource.getSelectedBranches();
+		}
+		final File workdir = cloneDestination.getDestinationFile();
+		final Ref ref = cloneDestination.getInitialBranch();
+		final String remoteName = cloneDestination.getRemote();
+
+		boolean created = workdir.exists();
+		if (!created)
+			created = workdir.mkdirs();
+
+		if (!created || !workdir.isDirectory()) {
+			final String errorMessage = NLS.bind(
+					UIText.GitCloneWizard_errorCannotCreate, workdir.getPath());
+			ErrorDialog.openError(getShell(), getWindowTitle(),
+					UIText.GitCloneWizard_failed, new Status(IStatus.ERROR,
+							Activator.getPluginId(), 0, errorMessage, null));
+			// let's give user a chance to fix this minor problem
+			return false;
+		}
+
+		int timeout = Activator.getDefault().getPreferenceStore()
+				.getInt(UIPreferences.REMOTE_CONNECTION_TIMEOUT);
+		final CloneOperation op = new CloneOperation(uri, allSelected,
+				selectedBranches, workdir, ref != null ? ref.getName() : null,
+				remoteName, timeout);
+		if (credentials != null)
+			op.setCredentialsProvider(new UsernamePasswordCredentialsProvider(
+					credentials.getUser(), credentials.getPassword()));
+		op.setCloneSubmodules(cloneDestination.isCloneSubmodules());
+
+		if (gerritConfiguration != null
+				&& gerritConfiguration.configureGerrit()) {
+			boolean hasReviewNotes = hasReviewNotes(uri, timeout, credentials);
+			if (!hasReviewNotes)
+				MessageDialog.openInformation(getShell(),
+						UIText.GitCloneWizard_MissingNotesTitle,
+						UIText.GitCloneWizard_MissingNotesMessage);
+			doGerritConfiguration(remoteName, op, hasReviewNotes);
+		}
+
+		if (cloneDestination.isImportProjects()) {
+			final IWorkingSet[] sets = cloneDestination.getWorkingSets();
+			op.addPostCloneTask(new PostCloneTask() {
+				public void execute(Repository repository,
+						IProgressMonitor monitor) throws CoreException {
+					importProjects(repository, sets);
+				}
+			});
+		}
+
+		alreadyClonedInto = workdir.getPath();
+
+		if (!callerRunsCloneOperation)
+			runAsJob(uri, op);
+		else
+			cloneOperation = op;
+		return true;
+	}
+
+	private void importProjects(final Repository repository,
+			final IWorkingSet[] sets) {
+		String repoName = Activator.getDefault().getRepositoryUtil()
+				.getRepositoryName(repository);
+		Job importJob = new Job(MessageFormat.format(
+				UIText.GitCloneWizard_jobImportProjects, repoName)) {
+
+			protected IStatus run(IProgressMonitor monitor) {
+				List<File> files = new ArrayList<File>();
+				ProjectUtil.findProjectFiles(files, repository.getWorkTree(),
+						null, monitor);
+				if (files.isEmpty())
+					return Status.OK_STATUS;
+
+				Set<ProjectRecord> records = new LinkedHashSet<ProjectRecord>();
+				for (File file : files)
+					records.add(new ProjectRecord(file));
+				try {
+					ProjectUtils.createProjects(records, repository, sets,
+							monitor);
+				} catch (InvocationTargetException e) {
+					Activator.logError(e.getLocalizedMessage(), e);
+				} catch (InterruptedException e) {
+					Activator.logError(e.getLocalizedMessage(), e);
+				}
+				return Status.OK_STATUS;
+			}
+		};
+		importJob.schedule();
+	}
+
+	private boolean hasReviewNotes(final URIish uri, int timeout,
+			UserPasswordCredentials credentials) {
+		boolean hasNotes = false;
+		try {
+			final Repository db = new FileRepository(new File("/tmp")); //$NON-NLS-1$
+			final ListRemoteOperation listRemoteOp = new ListRemoteOperation(
+					db, uri, timeout);
+			if (credentials != null)
+				listRemoteOp
+						.setCredentialsProvider(new EGitCredentialsProvider(
+								credentials.getUser(), credentials
+										.getPassword()));
+			getContainer().run(true, true, new IRunnableWithProgress() {
+				public void run(IProgressMonitor monitor)
+						throws InvocationTargetException, InterruptedException {
+					listRemoteOp.run(monitor);
+				}
+			});
+			String notesRef = Constants.R_NOTES + "review"; //$NON-NLS-1$
+			hasNotes = listRemoteOp.getRemoteRef(notesRef) != null;
+		} catch (IOException e) {
+			Activator.handleError(UIText.GitCloneWizard_failed, e, true);
+		} catch (InvocationTargetException e) {
+			Activator.handleError(UIText.GitCloneWizard_failed, e.getCause(),
+					true);
+		} catch (InterruptedException e) {
+			// nothing to do
+		}
+		return hasNotes;
+	}
+
+	private void doGerritConfiguration(final String remoteName,
+			final CloneOperation op, final boolean hasNotes) {
+		String gerritBranch = gerritConfiguration.getBranch();
+		URIish pushURI = gerritConfiguration.getURI();
+		if (hasNotes) {
+			String notesRef = Constants.R_NOTES + "review"; //$NON-NLS-1$
+			op.addPostCloneTask(new ConfigureFetchAfterCloneTask(remoteName,
+					notesRef + ":" + notesRef)); //$NON-NLS-1$
+		}
+		if (gerritBranch != null && gerritBranch.length() > 0) {
+			ConfigurePushAfterCloneTask push = new ConfigurePushAfterCloneTask(
+					remoteName, "HEAD:refs/for/" + gerritBranch, pushURI); //$NON-NLS-1$
+			op.addPostCloneTask(push);
+		}
+		op.addPostCloneTask(new SetChangeIdTask(true));
+	}
+
+	/**
+	 * @param container
+	 */
+	public void runCloneOperation(IWizardContainer container) {
+		try {
+			container.run(true, true, new IRunnableWithProgress() {
+				public void run(IProgressMonitor monitor)
+						throws InvocationTargetException, InterruptedException {
+					executeCloneOperation(cloneOperation, monitor);
+				}
+			});
+		} catch (InvocationTargetException e) {
+			Activator.handleError(UIText.GitCloneWizard_failed, e.getCause(),
+					true);
+		} catch (InterruptedException e) {
+			// nothing to do
+		}
+	}
+
+	private void runAsJob(final URIish uri, final CloneOperation op) {
+		final Job job = new Job(NLS.bind(UIText.GitCloneWizard_jobName,
+				uri.toString())) {
+			@Override
+			protected IStatus run(final IProgressMonitor monitor) {
+				try {
+					return executeCloneOperation(op, monitor);
+				} catch (InterruptedException e) {
+					return Status.CANCEL_STATUS;
+				} catch (InvocationTargetException e) {
+					Throwable thr = e.getCause();
+					return new Status(IStatus.ERROR, Activator.getPluginId(),
+							0, thr.getMessage(), thr);
+				}
+			}
+
+			@Override
+			public boolean belongsTo(Object family) {
+				if (family.equals(JobFamilies.CLONE))
+					return true;
+				return super.belongsTo(family);
+			}
+		};
+		job.setUser(true);
+		job.schedule();
+	}
+
+	private IStatus executeCloneOperation(final CloneOperation op,
+			final IProgressMonitor monitor) throws InvocationTargetException,
+			InterruptedException {
+
+		final RepositoryUtil util = Activator.getDefault().getRepositoryUtil();
+
+		op.run(monitor);
+		util.addConfiguredRepository(op.getGitDir());
+		return Status.OK_STATUS;
+	}
+
+	/**
+	 * @param newValue
+	 *            if true the clone wizard just creates a clone operation. The
+	 *            caller has to run this operation using runCloneOperation. If
+	 *            false the clone operation is performed using a job.
+	 */
+	public void setCallerRunsCloneOperation(boolean newValue) {
+		callerRunsCloneOperation = newValue;
+	}
+
+}
diff --git a/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/clone/GitCloneWizard.java b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/clone/GitCloneWizard.java
index 3dba04520e..df4bda8a46 100644
--- a/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/clone/GitCloneWizard.java
+++ b/org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/clone/GitCloneWizard.java
@@ -15,72 +15,24 @@ package org.eclipse.egit.ui.internal.clone;
 
 import java.io.File;
 import java.io.IOException;
-import java.lang.reflect.InvocationTargetException;
-import java.text.MessageFormat;
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.Collections;
-import java.util.LinkedHashSet;
-import java.util.List;
-import java.util.Set;
 
-import org.eclipse.core.runtime.CoreException;
-import org.eclipse.core.runtime.IProgressMonitor;
-import org.eclipse.core.runtime.IStatus;
-import org.eclipse.core.runtime.Status;
-import org.eclipse.core.runtime.jobs.Job;
-import org.eclipse.egit.core.RepositoryUtil;
-import org.eclipse.egit.core.internal.util.ProjectUtil;
-import org.eclipse.egit.core.op.CloneOperation;
-import org.eclipse.egit.core.op.CloneOperation.PostCloneTask;
-import org.eclipse.egit.core.op.ConfigureFetchAfterCloneTask;
-import org.eclipse.egit.core.op.ConfigurePushAfterCloneTask;
-import org.eclipse.egit.core.op.ListRemoteOperation;
-import org.eclipse.egit.core.op.SetChangeIdTask;
 import org.eclipse.egit.core.securestorage.UserPasswordCredentials;
-import org.eclipse.egit.ui.Activator;
-import org.eclipse.egit.ui.JobFamilies;
 import org.eclipse.egit.ui.UIIcons;
-import org.eclipse.egit.ui.UIPreferences;
 import org.eclipse.egit.ui.UIText;
 import org.eclipse.egit.ui.internal.SecureStoreUtils;
+import org.eclipse.egit.ui.internal.components.RepositorySelection;
 import org.eclipse.egit.ui.internal.components.RepositorySelectionPage;
-import org.eclipse.egit.ui.internal.credentials.EGitCredentialsProvider;
-import org.eclipse.jface.dialogs.ErrorDialog;
 import org.eclipse.jface.dialogs.MessageDialog;
-import org.eclipse.jface.operation.IRunnableWithProgress;
-import org.eclipse.jface.wizard.IWizardContainer;
-import org.eclipse.jface.wizard.Wizard;
-import org.eclipse.jgit.lib.Constants;
-import org.eclipse.jgit.lib.Ref;
-import org.eclipse.jgit.lib.Repository;
-import org.eclipse.jgit.storage.file.FileRepository;
-import org.eclipse.jgit.transport.URIish;
-import org.eclipse.jgit.transport.UsernamePasswordCredentialsProvider;
 import org.eclipse.jgit.util.FileUtils;
-import org.eclipse.osgi.util.NLS;
-import org.eclipse.ui.IWorkingSet;
 
 /**
  * Import Git Repository Wizard. A front end to a git clone operation.
  */
-public class GitCloneWizard extends Wizard {
+public class GitCloneWizard extends AbstractGitCloneWizard {
 
 	private static final String HELP_CONTEXT = "org.eclipse.egit.ui.GitCloneWizard"; //$NON-NLS-1$
 
-	private RepositorySelectionPage cloneSource;
-
-	private SourceBranchPage validSource;
-
-	private CloneDestinationPage cloneDestination;
-
-	private GerritConfigurationPage gerritConfiguration;
-
-	private String alreadyClonedInto;
-
-	private boolean callerRunsCloneOperation;
-
-	private CloneOperation cloneOperation;
+	RepositorySelectionPage cloneSource;
 
 	/**
 	 * The default constructor
@@ -97,33 +49,13 @@ public class GitCloneWizard extends Wizard {
 	 *            with.
 	 */
 	public GitCloneWizard(String presetURI) {
+		super();
 		setWindowTitle(UIText.GitCloneWizard_title);
 		setDefaultPageImageDescriptor(UIIcons.WIZBAN_IMPORT_REPO);
 		setNeedsProgressMonitor(true);
 		cloneSource = new RepositorySelectionPage(true, presetURI);
 		cloneSource.setHelpContext(HELP_CONTEXT);
-		validSource = new SourceBranchPage() {
-
-			@Override
-			public void setVisible(boolean visible) {
-				if (visible) {
-					setSelection(cloneSource.getSelection());
-					setCredentials(cloneSource.getCredentials());
-				}
-				super.setVisible(visible);
-			}
-		};
 		validSource.setHelpContext(HELP_CONTEXT);
-		cloneDestination = new CloneDestinationPage() {
-			@Override
-			public void setVisible(boolean visible) {
-				if (visible)
-					setSelection(cloneSource.getSelection(), validSource
-							.getAvailableBranches(), validSource
-							.getSelectedBranches(), validSource.getHEAD());
-				super.setVisible(visible);
-			}
-		};
 		cloneDestination.setHelpContext(HELP_CONTEXT);
 		gerritConfiguration = new GerritConfigurationPage() {
 
@@ -137,14 +69,14 @@ public class GitCloneWizard extends Wizard {
 		gerritConfiguration.setHelpContext(HELP_CONTEXT);
 	}
 
-	/**
-	 * @param newValue
-	 *            if true the clone wizard just creates a clone operation. The
-	 *            caller has to run this operation using runCloneOperation. If
-	 *            false the clone operation is performed using a job.
-	 */
-	public void setCallerRunsCloneOperation(boolean newValue) {
-		callerRunsCloneOperation = newValue;
+	@Override
+	protected RepositorySelection getRepositorySelection() {
+		return cloneSource.getSelection();
+	}
+
+	@Override
+	protected UserPasswordCredentials getCredentials() {
+		return cloneSource.getCredentials();
 	}
 
 	/**
@@ -153,7 +85,7 @@ public class GitCloneWizard extends Wizard {
 	 * @param show
 	 * @return this wizard
 	 */
-	public GitCloneWizard setShowProjectImport(boolean show) {
+	public AbstractGitCloneWizard setShowProjectImport(boolean show) {
 		cloneDestination.setShowProjectImport(show);
 		return this;
 	}
@@ -177,10 +109,12 @@ public class GitCloneWizard extends Wizard {
 	}
 
 	@Override
-	public void addPages() {
+	protected void addPreClonePages() {
 		addPage(cloneSource);
-		addPage(validSource);
-		addPage(cloneDestination);
+	}
+
+	@Override
+	protected void addPostClonePages() {
 		addPage(gerritConfiguration);
 	}
 
@@ -198,219 +132,10 @@ public class GitCloneWizard extends Wizard {
 						.getCredentials(), cloneSource.getSelection().getURI()))
 					return false;
 			}
-			return performClone();
+			cloneSource.saveUriInPrefs();
+			return performClone(cloneSource.getSelection().getURI(), cloneSource.getCredentials());
 		} finally {
 			setWindowTitle(UIText.GitCloneWizard_title);
 		}
 	}
-
-	boolean performClone() {
-		final URIish uri = cloneSource.getSelection().getURI();
-		setWindowTitle(NLS.bind(UIText.GitCloneWizard_jobName, uri.toString()));
-		final boolean allSelected;
-		final Collection<Ref> selectedBranches;
-		if (validSource.isSourceRepoEmpty()) {
-			// fetch all branches of empty repo
-			allSelected = true;
-			selectedBranches = Collections.emptyList();
-		} else {
-			allSelected = validSource.isAllSelected();
-			selectedBranches = validSource.getSelectedBranches();
-		}
-		final File workdir = cloneDestination.getDestinationFile();
-		final Ref ref = cloneDestination.getInitialBranch();
-		final String remoteName = cloneDestination.getRemote();
-
-		boolean created = workdir.exists();
-		if (!created)
-			created = workdir.mkdirs();
-
-		if (!created || !workdir.isDirectory()) {
-			final String errorMessage = NLS.bind(
-					UIText.GitCloneWizard_errorCannotCreate, workdir.getPath());
-			ErrorDialog.openError(getShell(), getWindowTitle(),
-					UIText.GitCloneWizard_failed, new Status(IStatus.ERROR,
-							Activator.getPluginId(), 0, errorMessage, null));
-			// let's give user a chance to fix this minor problem
-			return false;
-		}
-
-		int timeout = Activator.getDefault().getPreferenceStore().getInt(
-				UIPreferences.REMOTE_CONNECTION_TIMEOUT);
-		final CloneOperation op = new CloneOperation(uri, allSelected,
-				selectedBranches, workdir, ref != null ? ref.getName() : null,
-				remoteName, timeout);
-		UserPasswordCredentials credentials = cloneSource.getCredentials();
-		if (credentials != null)
-			op.setCredentialsProvider(new UsernamePasswordCredentialsProvider(
-					credentials.getUser(), credentials.getPassword()));
-		op.setCloneSubmodules(cloneDestination.isCloneSubmodules());
-
-		if (gerritConfiguration.configureGerrit()) {
-			boolean hasReviewNotes = hasReviewNotes(uri, timeout, credentials);
-			if (!hasReviewNotes)
-				MessageDialog.openInformation(getShell(),
-						UIText.GitCloneWizard_MissingNotesTitle,
-						UIText.GitCloneWizard_MissingNotesMessage);
-			doGerritConfiguration(remoteName, op, hasReviewNotes);
-		}
-
-		if (cloneDestination.isImportProjects()) {
-			final IWorkingSet[] sets = cloneDestination.getWorkingSets();
-			op.addPostCloneTask(new PostCloneTask() {
-				public void execute(Repository repository,
-						IProgressMonitor monitor) throws CoreException {
-					importProjects(repository, sets);
-				}
-			});
-		}
-
-		alreadyClonedInto = workdir.getPath();
-
-		cloneSource.saveUriInPrefs();
-		if (!callerRunsCloneOperation)
-			runAsJob(uri, op);
-		else
-			cloneOperation = op;
-		return true;
-	}
-
-	private void importProjects(final Repository repository,
-			final IWorkingSet[] sets) {
-		String repoName = Activator.getDefault().getRepositoryUtil()
-				.getRepositoryName(repository);
-		Job importJob = new Job(MessageFormat.format(
-				UIText.GitCloneWizard_jobImportProjects, repoName)) {
-
-			protected IStatus run(IProgressMonitor monitor) {
-				List<File> files = new ArrayList<File>();
-				ProjectUtil.findProjectFiles(files, repository.getWorkTree(),
-						null, monitor);
-				if (files.isEmpty())
-					return Status.OK_STATUS;
-
-				Set<ProjectRecord> records = new LinkedHashSet<ProjectRecord>();
-				for (File file : files)
-					records.add(new ProjectRecord(file));
-				try {
-					ProjectUtils.createProjects(records, repository, sets,
-							monitor);
-				} catch (InvocationTargetException e) {
-					Activator.logError(e.getLocalizedMessage(), e);
-				} catch (InterruptedException e) {
-					Activator.logError(e.getLocalizedMessage(), e);
-				}
-				return Status.OK_STATUS;
-			}
-		};
-		importJob.schedule();
-	}
-
-	private boolean hasReviewNotes(final URIish uri, int timeout,
-			UserPasswordCredentials credentials) {
-		boolean hasNotes = false;
-		try {
-			final Repository db = new FileRepository(new File("/tmp")); //$NON-NLS-1$
-			final ListRemoteOperation listRemoteOp = new ListRemoteOperation(db, uri, timeout);
-			if (credentials != null)
-				listRemoteOp
-				.setCredentialsProvider(new EGitCredentialsProvider(
-						credentials.getUser(), credentials
-						.getPassword()));
-			getContainer().run(true, true, new IRunnableWithProgress() {
-				public void run(IProgressMonitor monitor)
-						throws InvocationTargetException, InterruptedException {
-					listRemoteOp.run(monitor);
-				}
-			});
-			String notesRef = Constants.R_NOTES + "review"; //$NON-NLS-1$
-			hasNotes = listRemoteOp.getRemoteRef(notesRef) != null;
-		} catch (IOException e) {
-			Activator.handleError(UIText.GitCloneWizard_failed,
-					e, true);
-		} catch (InvocationTargetException e) {
-			Activator.handleError(UIText.GitCloneWizard_failed,
-					e.getCause(), true);
-		} catch (InterruptedException e) {
-			// nothing to do
-		}
-		return hasNotes;
-	}
-
-	private void doGerritConfiguration(final String remoteName,
-			final CloneOperation op, final boolean hasNotes) {
-		String gerritBranch = gerritConfiguration.getBranch();
-		URIish pushURI = gerritConfiguration.getURI();
-		if (hasNotes) {
-			String notesRef = Constants.R_NOTES + "review"; //$NON-NLS-1$
-			op.addPostCloneTask(new ConfigureFetchAfterCloneTask(remoteName,
-					notesRef + ":" + notesRef)); //$NON-NLS-1$
-		}
-		if (gerritBranch != null && gerritBranch.length() > 0) {
-			ConfigurePushAfterCloneTask push = new ConfigurePushAfterCloneTask(remoteName,
-					"HEAD:refs/for/" + gerritBranch, pushURI); //$NON-NLS-1$
-			op.addPostCloneTask(push);
-		}
-		op.addPostCloneTask(new SetChangeIdTask(true));
-	}
-
-	/**
-	 * @param container
-	 */
-	public void runCloneOperation(IWizardContainer container) {
-		try {
-				container.run(true, true,
-						new IRunnableWithProgress() {
-							public void run(IProgressMonitor monitor)
-									throws InvocationTargetException,
-									InterruptedException {
-								executeCloneOperation(cloneOperation, monitor);
-							}
-						});
-			} catch (InvocationTargetException e) {
-				Activator.handleError(UIText.GitCloneWizard_failed,
-						e.getCause(), true);
-			} catch (InterruptedException e) {
-				// nothing to do
-			}
-	}
-
-	private void runAsJob(final URIish uri, final CloneOperation op) {
-		final Job job = new Job(NLS.bind(UIText.GitCloneWizard_jobName,
-				uri.toString())) {
-			@Override
-			protected IStatus run(final IProgressMonitor monitor) {
-				try {
-					return executeCloneOperation(op, monitor);
-				} catch (InterruptedException e) {
-					return Status.CANCEL_STATUS;
-				} catch (InvocationTargetException e) {
-					Throwable thr = e.getCause();
-					return new Status(IStatus.ERROR, Activator.getPluginId(),
-							0, thr.getMessage(), thr);
-				}
-			}
-
-			@Override
-			public boolean belongsTo(Object family) {
-				if (family.equals(JobFamilies.CLONE))
-					return true;
-				return super.belongsTo(family);
-			}
-		};
-		job.setUser(true);
-		job.schedule();
-	}
-
-	private IStatus executeCloneOperation(final CloneOperation op,
-			final IProgressMonitor monitor) throws InvocationTargetException,
-			InterruptedException {
-
-		final RepositoryUtil util = Activator.getDefault()
-				.getRepositoryUtil();
-
-		op.run(monitor);
-		util.addConfiguredRepository(op.getGitDir());
-		return Status.OK_STATUS;
-	}
 }