MauroDataMapper / mdm-ui

Web front-end for the Mauro Data Mapper
Apache License 2.0
7 stars 5 forks source link

Server error response message is displayed when a Data element linked to a comment is selected #329

Open Jayafo opened 3 years ago

Jayafo commented 3 years ago

Describe the bug When a user selects a data element link in a comment, server error response message is displayed on the page. To ReproduceServer error message Perquisite: Ensure Data element is linked to the comment

  1. Go to 'Mauro'
  2. Click on 'a Model
  3. Select the Menu
  4. Keep Focus on the menu from the dropdown down menu list
  5. Select the data element link on comment
  6. Verify the error page displayed Server response message { "headers": { "normalizedNames": {}, "lazyUpdate": null }, "status": 404, "statusText": "OK", "url": "https://rost6-mauro-james-2021-09.rost6.dev.dataproducts.nhs.uk/nhsd_mauro_test/api/dataModels/dm:Sub%20Testing%2028.09/profiles/used", "ok": false, "name": "HttpErrorResponse", "message": "Http failure response for https://rost6-mauro-james-2021-09.rost6.dev.dataproducts.nhs.uk/nhsd_mauro_test/api/dataModels/dm:Sub%20Testing%2028.09/profiles/used: 404 OK", "error": { "path": "/api/dataModels/dm:Sub%20Testing%2028.09/profiles/used", "resource": "DataModel", "id": "null" } } Expected behaviour The page should display the linked data element in the comment. Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

olliefreeman commented 3 years ago

@Jayafo can you provide the text of the comment which you have please

olliefreeman commented 3 years ago

The failing url is not a url that’s ever been used in links to things, this is in fact a url to the used profiles of a datamodel, attempting to use pathing in the url which has never been allowed for that url. Which is why we’d like to see the comment text

Jayafo commented 3 years ago

@olliefreeman below is the text of the com image ment is on the screenshot attached