Git Repositories View: allow tag check-out

Tags can now be checked out properly. This fix
enables the check-out on tag nodes in the Git
Repositories View (also on double-click on a tag
node).

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Merge "Ignore secondary mouse clicks in the 'History' view's hyperlinks"

Merge "Improve error validation in the patch wizard"

Merge "'Git' property page does not need 'Default' and 'Apply' buttons"

Improve error validation in the patch wizard

Currently, no error messages are presented to the user for invalid
input into the wizard. This is very unhelpful as the user may not
know what is wrong with their input. Switching to the clipboard
mode may also not clear the error message due to an error in the
validation logic. To also help prevent the user from entering data
that may not end up being processed, controls are now enabled and
disabled based on which radio button has been selected.

Change-Id: I50998266c213deabec4a01a44024d95015a56b55

'Git' property page does not need 'Default' and 'Apply' buttons

The 'Git' property page currently only shows read-only information
to the user. As there is no state information on this property
page, the 'Default' and 'Apply' buttons should be removed.

Change-Id: Id706859173a6bd9f11f4adebf78d1e03579621a1

Improve the patch wizard's text and image enhance its presentation

The wording of certain messages has been corrected and mnemonics
has been introduced to enhance accessibility. The wizard now also
uses the same title area image that the CVS plug-in uses.

Change-Id: Ia4777f18c5aec73983ceebb9ff12553cd24a2c20

Merge "Relax constructor parameters of EGit operations"

Precheck selected files in the commit dialog

The commit dialog presents all local modifications of files but
does not respect the user's selection. The dialog should precheck
any files that the user has selected and any files that are in any
folders that the user has selected.

Bug: 304131
Change-Id: I1a0f73641b5bd5603b9fbde8fc25eb310976b3e0

Ignore secondary mouse clicks in the 'History' view's hyperlinks

Only the primary mouse click should be acknowledged as a request
to jump to another commit when browsing commits in the 'History'
view. At the moment, users that want to select the commit id and
copy the id to the clipboard through the context menu might
actually end up jumping to the selected commit as the secondary
mouse click is processed as a hyperlink request.

Change-Id: I8351cbda039419160e6731e307fa7243a3be90d4

Relax constructor parameters of EGit operations

Certain EGit operations takes a Collection<IResource> as a
parameter in its constructor. The problem with this type is that
one cannot pass in a Collection<IFile> or a Collection<IFolder>.
Instead, you are forced to alter your existing collection into one
for IResources. The fix is to alter these constructors so that they
take a Collection<? extends IResource> instead.

Change-Id: I93acf875107596b7eb08d76aaf69f13ed0cab524

Add tagging UI support

Implements UI for unsigned and annotated tags. This implementation
allows the user to select which commit object should be associated with
created tag. List of all commits is available in the 'Advanced' section
of the dialog. There is a special combo widget that can suggest commits
based on match of typed-in text with commit SHA-1 or part of first line
of existing commit messages.

User can also edit/overwrite tag message or associated commit of an existing
tag by selecting it from the list of existing tags and marking the
'Force replace existing tag' check box.

By default tag is created for current HEAD, this can be changed in the
'Advanced' section.

The tag dialog is available in project's context menu ('Team->Tag...').

[ms]: rebased to master and resolved conflict

CQ: 4073
Bug: 311262
Change-Id: Icb30655845d1e1198e59992a10148421ab9f9bd5
Signed-off-by: Dariusz Luksza <dariusz@luksza.org>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Git Repositories View: Bare Repositoy Support

The following actions are disabled for a "bare" repository:
- checkout of branches
- browsing the working directory
- import of projects

The "bare"-ness of a Repository is visualized in the "Working Directoy" node
which displays "This is  a bare Repository" instead of the working directory.

Bug: 311919
Change-Id: Ib698326233dee88b7a25bda7484b76b068c33a6c
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Merge "Git RepositoriesView: fix Label Provider"

Fix NullPointer in "Compare with each other"

The Exception was thrown in the HistoryView,
"Compare with each other":
When the file did not exist in the newer one of the
two commits there was a NullPointer Exception.

Bug: 312644
Signed-off-by: Stefan Lay <stefan.lay@sap.com>

History View: sub menus for reset and quick diff

Add sub menus to the context menu of the history
view to improve the usability of the menu.

Reset
    Hard
    Mixed
    Soft

Quick Diff
    Reset Baseline to first Parent of HEAD
    Reset Baseline to HEAD
    Set as Baseline

Change-Id: I0cd088b3ea3d39be087607a33b627129e5b04cba
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>

Git RepositoriesView: fix Label Provider

The label provider must be fixed with respect to the "checked out"-decoration.
Currently, this only works if a local branch is checked out. The fix makes the
decoration work also if a tag or remote branch is checked out.

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Removed checkedItems field from GitProjectsImportPage

The checkedItems field has been removed from GitProjectsImportPage,
and a getCheckedProjects method has been added instead, which computes
the set of checked projects based on information available from the UI
widgets.

This fixes at least one bug that was affecting my work: unchecking projects
by using the spacebar didn't work, because there was no KeyListener in place
to update the checkedItems field. This lead to the project being unchecked
in the UI, but still considered checked by the wizard page, and upon
clicking Finish, the wizard would try to import all of those unchecked
projects.

[ms]:
- split off extraction of ProjectRecord into separate change
- ported change to current master
- added Ryan's copyright

Bug: 308284
Change-Id: I5c0889e9f6e95c2c62d96195623499802ce701a1
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Signed-off-by: Ryan Schmitt <ryan.schmitt@boeing.com>

Remove dead code from ProjectRecord and GitProjectsImportPage

The IImportStructureProvider passed to ProjectRecord was always null and
hence had no effect. The constructor ProjectRecord(Object, int) is
obsolete and no longer used. Therefore removing this dead code.

Change-Id: I9937c06217c855e3f8cf2391de4849636fa3e4af
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Refactored GitProjectsImportPage

Extract inner class ProjectRecord to improve readability.

[ms] : rebased to current master

Change-Id: Iec631ae27610ccf147c4ac3445aead9d017b07ef
Signed-off-by: Ryan Schmitt <ryan.schmitt@boeing.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Branch Selection Dialog UI improvements

The dialog shown for Team->Branch... and Team->Reset To... was
reworked to re-use the same tree as the Git Repositories view.

Change-Id: I68eb12f7c6d7b46488650e635639772a95a88e90
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Show "Create Patch..." context menu entry in history view

The menu entry is shown in history view and not in the package
explorer as in CVS. The reason is that when you use a distributed
versioning system you normally commit your local changes
to a local branch before you create a patch.

When one commit is selected in the commit list, the diff to
its parent can be saved to a file or stored in the clipboard.

The file filter of the history view is used.

It is not possible to create patch files for the first commit and
for merge commits.

Per default the wizard creates a patch with a format which can
be applied with the eclipse apply patch wizard:  The paths to the
files have no prefixes (as with git format-patch --no-prefix) and
they are relative to the eclipse project and not to the repository.

There is an option to create a patch which can be applied with
"git apply" on the command line (Not yet in eclipse.).
There may be more options in the future on the Options page.
Before more options are offered the diff rendering code should
be moved to jgit.

[ms]
- fixed some style nits
- reduced initial wizard height

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

Merge "BranchOperation: fix checkout for tags and remote branches"

Merge "LabelDecoration should show ellipsis if HEAD is on a commit"

Merge "Use jgit p2 repository to resolve jgit dependencies"

BranchOperation: fix checkout for tags and remote branches

The current implementation wrongly changes the HEAD
ref to point to the remote branch instead of to the underlying
commit. Support for tags is not working at all.
This fixes both issues.

Bug: 312053

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Use jgit p2 repository to resolve jgit dependencies

EGit Tycho builds on build.eclipse.org frequently hit corrupted artifacts
which leads to broken builds. Cleaning up these corrupted files is tedious
since it requires file system access on the build server. Hence we want to
switch to use job-local m2 repositories. This requires that build artifacts
are shared between the jgit and egit build jobs via p2. Therefore we switch
from pom-first to p2 build dependencies between the egit and the jgit build.

For local builds the egit build assumes that jgit and egit are located
under a common parent folder. If this is not the case the URL of the jgit
update site has to be injected via system property "jgit-site".

- non-default local build:
$ mvn clean install -Djgit-site=file:/path/to/org.eclipse.jgit.updatesite/target/site

- hudson build on build.eclipse.org
$ mvn clean install -Djgit-site=https://build.eclipse.org/hudson/job/jgit/
              lastSuccessfulBuild/artifact/org.eclipse.jgit.packaging/
              org.eclipse.jgit.updatesite/target/site/

Change-Id: I43ee21f23d474a6579df187869c821fc3960583f
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Move UpdateJob to UpdateOperation

UpdateOperation currently schedules UpdateJob.
This is not needed since UpdateOperation itself
is scheduled as Job. Thus UpdateJob was integrated
in UpdateOperation.

Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>

LabelDecoration should show ellipsis if HEAD is on a commit

Bug: 311364

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Replace org.apache.log4j package import by bundle dependency

Tycho picks some org.slf4j bundles to satisfy our package dependency
to org.apache.log4j needed to run swtbot tests. This caused a
NoClassDefFoundError due to some additional dependencies slf4j has.
Switching to a bundle dependency fixes this problem.

Change-Id: Ib9e70312bcdd591ec874cea0d10140300446cf93
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Merge "Fix RepositoriesViewLabelProvider"

Fix RepositoriesViewLabelProvider

If there are two branches with the same shortened name (e. g. a local
and a remote "master" branch), the current implementation wrongly
marks both of them as "checked out" if one of them is checked out.

The fix looks for the full branch name instead of the shortened name
in order to determine whether the icon should be decorated.

Change-Id:  Ib6652732cfecfa762c0aa810cb4a5226b46fc27b
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Action Reactoring (AbstractOperationAction)

AbstractOperationAction currently executes operations in ModalContext.
Furthermore the createOperation method has an untyped parameter
which resulted in operations with untyped parameter.
AbstractOperationAction and AbstractRevObjectAction were replaced by
AbstractResourceOperationAction and AbstractRevCommitOperationAction.
These new action base classes provide type safe operation creation and
execute the operation using a Job.

Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>

RepositoryTreeNode: fix compareTo() implemenation

The current implementation returns the wrong result when sorting a list of RepositoryTreeNodes.

Merge "Prepare EGit for the new State MERGING_RESOLVED"

Prevent leaking of SWT menus in the 'Git Repositories' view

New instances of SWT menus are currently being instantiated and set
on the tree control every time a user brings up the context menu in
the 'Git Repositories' view. When the tree is disposed, the tree's
menu is also disposed. However, as new menus are instantiated and
set every time, those menus that have been replaced are now lying
around and will only be disposed when the shell itself is disposed.

Change-Id: Ia77bf8cbd8cd3ef27e536821dd979e0882926f44

Git Repositories View: Allow to "remove" multiple Repositories

The "Remove" action should also work on a selection with more than one
Repository to improve usability.

Bug: 311366
Change-Id: Ic0ea1245eaf1c2df124c11fded3f4f78b330d7b3
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Prepare EGit for the new State MERGING_RESOLVED

JGit will report a new repository state MERGING_RESOLVED. Currently EGit
doesn't support committing in this state. Missing is the correct handling
of the content of .git/MERGE_MESSAGE and .git/MERGE_HEAD. When committing
in this state these contents have to be used and after a successful commit
these files have to be deleted.

This fix adds a special check for this new state and disallows committing
in this state.

This change depends on JGit change I0a885e2fe8c85049fb23722351ab89cf2c81a431.
This change will break the build of EGit if this JGit change is not merged.
On the other hand: As soon as the JGit change has been merged we'll have to
merge this change soon.

Change-Id: Ie28c2b43d168b8d6a2873bf9d09a1be9cc8bbf15
Signed-off-by: Christian Halstrick <christian.halstrick@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Revert "Prepare EGit for the new State MERGING_RESOLVED"

This reverts commit dd4a8c44bb201fb58c6031ff9324beb817e9bb9d.

That commit came in through a wrong push directly into the master
branch without going through gerrit

Merge "Replace getWorkbenchWindow().run by using a Job"

Prepare EGit for the new State MERGING_RESOLVED

JGit will report a new repository state MERGING_RESOLVED. Currently EGit
doesn't support committing in this state. Missing is the correct handling
of the content of .git/MERGE_MESSAGE and .git/MERGE_HEAD. When committing
in this state these contents have to be used and after a successful commit
these files have to be deleted.

This fix adds a special check for this new state and disallows committing
in this state.

This Change depends on JGit Change http://egit.eclipse.org/r/#change,667.
This change will break the build of EGit if this JGit change is not merged.
On the other hand: As soon as the JGit change has been merged we'll have to
merge this change soon.

Signed-off-by: Christian Halstrick <christian.halstrick@sap.com>

Replace getWorkbenchWindow().run by using a Job

ResetAction and Track now use a Job to execute the
operation.

Change-Id: Ifbad2a09d567a231461b2c33ff03868e1e1c7edc
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>

Add '+' to CloneGitRepository Icon

Bug: 312007

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Cleanup the RepositoriesView toolbar

Change-Id: Ic17a7927b8e19318546804f40962ff6a317b4d7f

Improve DiscardChangesAction

Currently DiscardChangesAction (Trigger: Replace With->File in Git
Index) runs in the UI thread. This might block the UI. Furthermore
the discard operation is located in the action.
Now DiscardChangesAction uses a Job to execute. The discard operation
was moved in a new class DiscardChangesOperation.

Change-Id: I06fe4efa096ae4ca6b1110c2f7259243e1d2c99d
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Git Repositories View: remove "Existing Projects" node

During the corresponding discussion on the egit mailing list,
no requests were made to keep this node.

Change-Id: Ie09d224f10162dff6ec8a48f1c6ef0b47c4aa64a

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Git Repositories View: CloneGit Icon

The currently used "Import" icon in the view's toolbar is ugly and misleading.
I created a "Clone" icon more suitable for this.

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Merge "Import wizard must not open with error"

Merge "Git Repositories View: Copy/Paste support"

Fix UI Tests for Git Clone

Bug: 311838

Change-Id: I13258edb78d0e0359424d0c4aedbca89358bdc68
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Import wizard must not open with error

Bug: 311834

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Git Repositories View: Copy/Paste support

This adds copy/paste support for the keyboard
and introduces some error handling/user feedback
for the paste action

Bug: 311486

Change-Id: Ie49f9ebfeb32d5665a0e424a13c55169f824027d
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Merge "Execute ResetOperation in Workspace Runnable"

Git Import Wizard

This is the new Import wizard for importing Projects from Git.
Similar to CVS, it first shows a list of Git Repositories. The user
may select one from the list (the same list as shown in the
Git Repositories View) and continues from there. Alternatively,
the user can clone or add a Repository to the list.
Then all continues like with the Import Projects... wizard
used in the Git Repositories View.

Bug: 281394

Change-Id: I7647bcbe7870500ef145b0100e5843d5b2930e94
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Execute ResetOperation in Workspace Runnable

ResetOperation changes the working tree if reset
type is HARD. In this case the operation is executed
in a WorkspaceRunnable to protect the changed
resources.

Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>

Git Repositories View: add CollapseAll action

This is helpful to restore the view after clicking around when there are too
many open nodes.

Bug: 311485
Change-Id: Id5bd583eec28978c28e3fa4eb758fb11b8642e0a
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Make clear that "Delete branch..." will delete forcefully

The current implementation doesn't check if commits on the branch
to be deleted have been merged to the currently checked out branch.
Since this may lead to data loss if the user isn't aware of this
the dialog text should state that explicitly.

Bug: 311354
Change-Id: Iefd51f8f98f6cf5633d6c47e314af9e36896a0b8
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Branching in RepositoriesView

This is a very first implementation of adding/deleting
branches in the RepositoriesView. It works on the
branch-related nodes.
This has no extensive input validation or such, and
it also does not take other repository states into
account, so perhaps this might have to be
added, too.
The "Delete Branch" operation uses "force"-mode
to make sure it is not rejected if the branch is not
a parent of the current branch or such.

Bug: 311354

Change-Id: I8d5501086568ff7219f504f8779cd394fa13cda3
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Fix remaining NLS issues

To reach complete translatability for Helios externalize the remaining
strings visible to end users and put NLS comments to mark technical strings.
Also change compiler settings to raise an error on NLS problems so that we
stay clean from now on and upgrades other compiler settings to Eclipse 3.6
default.

Change-Id: Iecfa96fc7cb6ea924c9e16876833c082ebdee5c7
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

EGit operation / action refactoring

Replace IWorkspaceRunnable in EGit operations (e.g. BranchOperation)
by a new interface IEGitOperation.
IWorkspaceRunnable is no good interface for an operation. The user
thinks he has to execute the operation using Workspace.run. But that
is not right for all EGit operations. Currently many operations are
used by just calling op.run. Thus a new interface with an execute
method was introduced. Those operations that need a workspace runnable
and a scheduling rule do the workspace runnable in their execute
method (e.g. branch). It's an internal of an operation how it needs
to execute.
BranchOperation now uses a workspace runnable to lock the workspace
while the working tree is modified.
BranchAction executes BranchOperation in a Job. The other actions
will be adopted to this execution pattern in further commits.

Change-Id: Ia3c1bf157d17632a17b7c678ee45717fb7944f0c
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>

Remove usage of unpublished API's in org.eclipse.egit.core

This also flags further such use in this bundle as errors and
upgrades other compiler settings to Eclipse 3.5.2 defaults.

A testcase is introduced to cover error handling pattern, but
it is incomplete and only implemented for BlobStorage.

Bug: 310933 (partial fix)

Change-Id: I40f6b9d457010a0e1a68d1d33edb11f4da7a1321
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Git Repositories View: Import Projects

Add some context menu in the view to allow starting
of different wizards for project import (not only the
Import Existing Projects wizard).
This is also in preparation for a better Git Import wizard.
The bug below is not solved by this, but this should
help in the discussion of it.

Bug: 281394

Change-Id: Ic673e8119158adfbf7147ac3ce925bd7f37fa9e4
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Merge "Enable import of empty Git repositories"

Enable import of empty Git repositories

Currently File->Import->Git Repository does not
allow to import an empty Git repository.
GitCloneWizard was enhanced to enable the import of
an empty repository.

Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>

Remove references to internal types in ShareSingleProjectCommand

There is no reason to use ConfigureProjectWizard because all the
methods it calls is actually public API. One minor discrepancy
introduced here is that we no longer go through Team's
ResizeWizardDialog inner class. However, the WizardDialog class
actually already sets a shell style with SWT.RESIZE so the
ResizeWizardDialog actually gives us nothing of value.

Change-Id: I81fd2d36748512234f3c8187873a65a1bfe1e3d7

Reorganize Git Repositories View tree

1, Split the "Branches" node into a "Local" and "Remote"
part
2. add a "Tags" node to separate them from the branches
3. show the full name of the reference behind HEAD
4. add a "Symbolic References"node to show
other symbolic references
5. cleanup context menus (in particular, do not allow
to checkout tags, as this is currently not supported)
6. Better icons (taken from CVS UI).
7. Exception handling for content provider

Bug: 309441

Change-Id: I623c7c481df669e04dbeca232fa9e6dc021e3abc
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Merge "Make the org.eclipse.core.test bundle a fragment bundle"

Fix broken 'Show In' feature for 'Git Repositories' view

Code was added to make the 'Git Repositories' view show up in the
'Show In' menu but no code was introduced in the view to actually
react to the request. As is, the view will simply be shown without
actually having anything selected. The fix is to implement the
IShowInTarget interface so that the request can be handled.

Change-Id: I64e11f856c25970c07e03dc3365e24f7e1733ede

Improve error handling in EGit UI

Use recently introduced error handling method
Activator.handleError instead of using ErrorDialog.

Change-Id: I97d918c572a141feea1b39c13ff7dcff2d8cc5ca
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>

Make the org.eclipse.core.test bundle a fragment bundle

A fragment bundle allows us direct access to internal packages of the
host bundle that we are testing without the hazzle with export/import.

Since we cannot have an activator for a fragment we now set up the
MockSystemReader in the GitTestCase that our tests derive from anyway.

Change-Id: I6a1ded23d0a3063ae03ef6b8683c04b8dbe0d84e
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>

Fix EGit history handling code to consider IAdaptables

The history code naively assumes that all of its inputs are of type
IResource. However, this is invalid as objects like an editor's
input or JDT's handling of the model of a Java project are not
represented by an IResource. The history code needs to be corrected
to consider the case where an IAdaptable is handed to it.

Bug: 311249
Change-Id: Ie1e20f68b9023bd98981f42d9b8fa14aae250614

Merge "Exception handling: make sure to retain stack trace"

Merge "Fix NPEs being thrown on tree selection in the reset dialog"

Merge "Remove references to internal classes in GitProjectsImportPage"

Fix NPEs being thrown on tree selection in the reset dialog

New code was introduced to enable/disable buttons based on the
selection in the tree. However, this code did not compensate
for the fact that some of those buttons may not have been
instantiated if the dialog is being used for a reset operation.

Change-Id: I0a61b141acce4d4c778a42467179e190cc67415e

Remove references to internal classes in GitProjectsImportPage

Where possible, internal classes should not be referenced as they
are not API and may be moved, renamed, or deleted at any time.

Bug: 310933
Change-Id: I6d2d8c0ada7a334544f60319f57afbbf00caa509

Missing perspectives for "Show In Git Repositores View"

Only the resource perspective was amended. Java, PDE and
DebugPerspectives should be added, too.

Bug: 309551 (continued)

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Exception handling: make sure to retain stack trace

Error logging with the status taken from a CoreException is
an anti-pattern as the stack trace will be lost.

Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Add repository name to Git Project Label

The repository name is added to the Git Project label decoration by default.
The pattern is [{repository} {branch}], i.e. space as separator.

Change-Id: I119d4d4b0d406b1778cc2da1d3d66b36cc9dde9d
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Merge "Tycho build for org.eclipse.egit.ui.test"

Update manifest metadata to be compliant for Helios

In order to comply with the Helios release train,
we need to update metadata in our manifests. The
fix involves not referring to the providerName
as Eclipse.org anymore and using Eclipse EGit.

Change-Id: I7b0e487a330e694eb3f2f979d3f1694061b21bd7
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Cleanup the EGit feature name

The old feature name contained a redundant word.

Change-Id: Ide49b6acbda751fb2d850e09ac107adf281ff78a
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Branch checkout: add progress monitoring

This adds progress monitoring to the branch
checkout operation for better user feedback.

Change-Id: Ic4360bafc881004e8f1cac2d95912c2e452895bd
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Remote configuration wizards

There should be some UI to allow specifying
remotes instead of having to edit the config
file directly. Some context menus were added
to the Repositories view to allow just that.
In order to re-use the existing wizard pages as
much as possible, there was some refactoring
required for these, too.

Bug: 304182

Change-Id: Ie17241dcf9a3665f899c6a044fc84afd04ceadb1
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Team Context menu cleanup

Team->Show In Git Repositories View should move to
Show In->Git Repositories View

Team->Show In History should be removed altogether

Bug: 309551
Change-Id: Iba8b1f8121084f0647765af72c577c7072dfd9d5
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Clean up error handling

Change-Id: Iafe95e06eccc715180184260d7cb53004874b437
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Fix error handling in some actions

Some missing error logging was added.

Change-Id: Ice7612f46d46b088fe99bef82a851c4191c7fd9e
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

'Repositories' view should use IOpenListener

Using open events to expand/collapse tree nodes or to open the
current selection is very common in the Eclipse workbench. An
IOpenListener has been attached to the 'Repositories' view so
that it will react in the same way as other Eclipse views.

Change-Id: I2e6d622fc96ca4a0f52ac0b462f1320f754dbc1b
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Tycho build for org.eclipse.egit.ui.test

Following approach described in [1].

Fixed dependencies:
- correcting wrong version range for org.eclipse.core.filesystem
- org.eclipse.ui and org.hamcrest need to be referenced via require-bundle

Make GitCloneWizardTest.canCloneARemoteRepo() independent from Eclipse
workspace name. This eliminates the need to configure the workspace used
by the test run from Tycho.

Introduce maven property "ui.test.vmargs" to enable passing VM parameters to the
VM running the tests.

Also introduce maven property "platform-version-name" to simplify building
against different Eclipse platform releases. E.g. to build and test
against helios platform release run

mvn -Dplatform-version-name=helios clean install

The following platform versions are supported:
- ganymede
- galileo
- helios

The tests are executed in the integration-test phase of the default Maven
lifecycle [2].

Since the feature org.eclipse.sdk.ide is not available in the ganymede p2
repository the feature org.eclipse.pde.feature.group is used to setup the
test harness.

[1] https://docs.sonatype.org/display/TYCHO/How+to+run+SWTBot+tests+with+Tycho
[2] http://maven.apache.org/guides/introduction/introduction-to-the-lifecycle.html

Change-Id: I06e94589589ad18b50daf641d89fd65a8c46f694
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

BranchSelectionDialog fixes

Currently, hitting "Rename" on a tag
will do some renaming inappropriately: the result
is a new local branch.
The same goes when renaming a remote
branch using this dialog: the result is a local
branch.
This fix will disable renaming of tags
altogether and use the appropriate prefixes
when renaming branches.
In addition, the different buttons will be
enabled/disabled depending on the selection
of the tag/branch.

Change-Id: I38b2f04f71ba7784506891fb22bc2e72bfaec4ec
Bug: 308457
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

RepositoriesView: use the GitImportProject wizard

The generic ImportExistingProject wizard from the
org.eclipse.ui.ide plug-in does not allow to specify
a directory in 3.4.
Instead of the generic ImportExistingProject wizard,
the RepositoriesView should use the Git wizard which
allows to limit the scope to a directory in the file system
and also adds project filtering capabilities as well as
the option to add the imported projects to Team support.

Change-Id: I4b66358e98a960af6c88e8d6656d24a6ee5279bb
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Fix selection handling in AbstractOperationAction

Team actions based on AbstractOperationAction
were disabled in the Java edtior due to wrong selection
handling.
The selection is now retrieved in selectionChanged.
The operation is now created in the run method.

Change-Id: Ia0c961c3f8adde2877d962ee1ff1f6de97776faf
Bug: 310440
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

NPE upon Team->Show in Git Repositories View

If this action also opens the view for the first time,
the refresh won't be finished by the time the action
is executed, so there is the NPE described in the bug.

Change-Id: I8af99c7689153d1663c039c43eb16f538fcaed19
Bug: 309602
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Editor tab switch should update GitRepostoriesView

When switching editors, the GitRepositoriesView
should update the tree selection if the "Link WithSelection"
toggle is on.

Change-Id: I02adb324696e6bf454cce4ebfd7ffbeae65f5e48
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

NullPointerException during Push

This is a bug in the RefSpecPage that leads
to a NPE being logged.

Change-Id: Ic02ea5c557fde959c6d20e0ba76bab16df350213
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>
Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Merge "Cleanup: remove methods from Activator"

Cleanup: remove methods from Activator

Some methods in the Activator were
moved to the appropriate classes that
need them or dropped altogether
as they were not used anywhere.

Change-Id: Ic7ec277e2dc66fbe4093a81a04a40c317807bc37
Signed-off-by: Mathias Kinzler <mathias.kinzler@sap.com>

Merge "Update license to be consistent with Helios requirements"

Graphical tags in history view were not correctly aligned

This was only a problem when the history graph is not in the first column
of the table

Change-Id: Ieb3f4edce31858def42fc34676f4a7213b86f65e
Signed-off-by: Ruth Alkema <ruth@diasoft.nl>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

'Repositories' view should not show scroll bars unless it has to

The view's code was arbitrarily introducing a TreeColumn to the
view's tree which had a width of 700. This meant that when the
view's width became less than 700, it would have to show scroll
bars to accomodate for this. However, it should actually only be
showing scroll bars if it has to and not because of some arbitrary
value has been surpassed.

The worst part is that using columns means that the text can only
go as far as what is dictated by its width. Since the view does not
have any column headers displayed, the user actually has no way of
resizing the column to view all of the content. The fix is to just
not use any columns and to let the tree itself determine whether it
needs to show scroll bars or not based on whether it is currently
displaying content that exceeds its own width.

Change-Id: Ib1da786859e0368816e229f03d9d6d1d30ec962b