Sonar Eclipse

Ability to create a review directly from Eclipse

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.2
  • Component/s: Review
  • Labels:
    None
  • Number of attachments :
    0

Issue Links

is duplicated by

Improvement - An improvement or enhancement to an existing feature or task. SONARIDE-228 Let the user create/edit some reviews directly from Eclipse

  • 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
Evgeny Mandrikov added a comment -

Implemented in https://github.com/SonarSource/sonar-eclipse/commit/a6d7c80073a0973bd2c6e2309e3eb4601d1edb6c

Show
Evgeny Mandrikov added a comment - Implemented in https://github.com/SonarSource/sonar-eclipse/commit/a6d7c80073a0973bd2c6e2309e3eb4601d1edb6c
Hide
Permalink
Freddy Mallet added a comment -

Evgeny, when I :

  • Create a review from the source code editor
  • Submit the review
  • I still have the ability to create a new review on the same violation in the source code editor (and so I then get an error when trying to submit this new review)

This is not a blocker issue but we must perfectly describe/understand what is the expected behavior.

Show
Freddy Mallet added a comment - Evgeny, when I : Create a review from the source code editor Submit the review I still have the ability to create a new review on the same violation in the source code editor (and so I then get an error when trying to submit this new review) This is not a blocker issue but we must perfectly describe/understand what is the expected behavior.
Hide
Permalink
Evgeny Mandrikov added a comment -

Freddy, this happens because marker for Violation wasn't refreshed, so in editor we don't know that review was created.

Show
Evgeny Mandrikov added a comment - Freddy, this happens because marker for Violation wasn't refreshed, so in editor we don't know that review was created.
Hide
Permalink
Freddy Mallet added a comment -

Ok, I see what you mean, the source file needs to be close and then reopened in oder to refresh the markers.

Show
Freddy Mallet added a comment - Ok, I see what you mean, the source file needs to be close and then reopened in oder to refresh the markers.

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Freddy Mallet
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: