Sonar Eclipse

When using branches, the resource cannot be reached from review editor

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.1
  • Fix Version/s: 2.4
  • Component/s: None
  • Labels:
  • Number of attachments :
    0

Description

I've tested the review integration in Eclipse. I imported 2 projects and associated them with Sonar with the correct key.

One of them doesn't have the branch attribute.
The other has a branch attribute.

I created 2 reviews, one for each project.
For the project w/o branch, I can click on the review and get directly to the violation in the source code.
For the project with branch, it tells me that the ressource cannot be located.

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. SONARIDE-299 Sonar review resource link in Mylyn not working when sonarResourceKey contains "branch" information

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
relates to

Improvement - An improvement or enhancement to an existing feature or task. SONARIDE-241 Display a more meaningful message when a resource can't be open from a review

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Vincent Latombe added a comment -

Probably caused by this FIXME https://github.com/SonarSource/sonar-eclipse/blob/master/org.sonar.ide.eclipse.jdt/src/org/sonar/ide/eclipse/internal/jdt/JavaElementsAdapterFactory.java#L72

Show
Vincent Latombe added a comment - Probably caused by this FIXME https://github.com/SonarSource/sonar-eclipse/blob/master/org.sonar.ide.eclipse.jdt/src/org/sonar/ide/eclipse/internal/jdt/JavaElementsAdapterFactory.java#L72
Hide
Permalink
Freddy Mallet added a comment -

Manually tested !

Show
Freddy Mallet added a comment - Manually tested !

People

  • Assignee:
    David Gageot
    Reporter:
    Vincent Latombe
Vote (2)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: