Sonar

Incorrect breadcrumb link on contextualized review page

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.13.1
  • Fix Version/s: 3.0
  • Component/s: Issues, Web
  • Labels:
    None
  • Number of attachments :
    1

Description

Example: http://nemo.sonarsource.org/project_reviews/view/112

If you click on the "All Sonar plugins" link in the header breadcrumb, you get nowhere. This is because only the "id" parameter is overriden by the breadcrumb component.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Simon Brandhof added a comment -

Works well but :

  • "Review1" must be replaced by "Review #1" (see attached screenshot)
  • the permalink must not contain the parameter title=Permalink
Show
Simon Brandhof added a comment - Works well but : "Review1" must be replaced by "Review #1" (see attached screenshot) the permalink must not contain the parameter title=Permalink
Hide
Permalink
Fabrice Bellingard added a comment -

Fixed on master.
FI, I kept the "Review" name in the breadcrumb as on "Home - My Project > #45" was quite weird to see (you can make a test by modifying the label with Firebug).

Show
Fabrice Bellingard added a comment - Fixed on master. FI, I kept the "Review" name in the breadcrumb as on "Home - My Project > #45" was quite weird to see (you can make a test by modifying the label with Firebug).
Hide
Permalink
Freddy Mallet added a comment -

Hi Fabrice, I'm reopening this ticket as the "reviews" web service doesn't allow anymore to search a review by "id". Thanks

Show
Freddy Mallet added a comment - Hi Fabrice, I'm reopening this ticket as the "reviews" web service doesn't allow anymore to search a review by "id". Thanks
Hide
Permalink
Fabrice Bellingard added a comment -

The search by 'id' for the review WS has been added back.

Show
Fabrice Bellingard added a comment - The search by 'id' for the review WS has been added back.

People

  • Assignee:
    Fabrice Bellingard
    Reporter:
    Fabrice Bellingard
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: