CDR Tickets

Issue Number 4206
Summary [Summaries] Broken Patient/HP Toggle When Linked to a Blocked Doc
Created 2016-12-22 16:17:59
Issue Type Bug
Submitted By Juthe, Robin (NIH/NCI) [E]
Assigned To Englisch, Volker (NIH/NCI) [C]
Status Closed
Resolved 2017-08-16 16:47:42
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.200654
Description

Sharon reported that the Thymoma Treatment HP summary (CDR62912) contains a broken link to the patient summary in the "toggle" on Cancer.gov. Volker noticed that this is caused by the fact that there are two patient versions of the summary--one active and one blocked. When the link gets created the HP toggle URL picks up the URL from the blocked instead of the active summary. We are fixing this problem manually, but I'm adding an issue so we can fix this problem in the future.

Comment entered 2017-01-20 11:41:56 by Englisch, Volker (NIH/NCI) [C]

Adding my findings from an email I had submitted and grabbing the ticket from .
This is a SQL query fix within a filter if I remember correctly.

I found the problem. The link for the toggle get’s created by looking at the PatientVersionOf element in the database. It’s assumed – incorrectly as it turns out – that there can only exist a single patient summary with this element pointing to one HP summary. Unfortunately, there do exist two summaries both pointing to the HP summary, one of which is a blocked summary. So, when the link gets created the HP toggle URL picks up the URL from the blocked instead of the active summary.

Comment entered 2017-03-21 14:04:58 by Englisch, Volker (NIH/NCI) [C]

The SQL query to pick up the SummaryToggleURL has been updated in the following filter:

  • CDR000157.xml: Vendor Filter: Summary

This is ready for review on DEV.

Comment entered 2017-03-21 14:26:19 by Englisch, Volker (NIH/NCI) [C]

I think this can only truly be tested by either looking at the XML output or by publishing the document and look at the link on Cancer.gov.
The toggles are currently missing on DEV. I'm not allowed - due to some other testing - to publish to QA, so we may need to publish to STAGE if that's desired.

Comment entered 2017-06-14 15:29:39 by Englisch, Volker (NIH/NCI) [C]

, the fix for this toggle problem is ready for testing on DEV.
Should we move it to QA and run diff reports?

Comment entered 2017-07-28 13:07:36 by Englisch, Volker (NIH/NCI) [C]

This ticket is a duplicate of CDECDR-4271.

Comment entered 2017-08-10 15:03:38 by Juthe, Robin (NIH/NCI) [E]

This is a duplicate of OCECDR-4271.

Comment entered 2017-08-14 15:15:00 by Englisch, Volker (NIH/NCI) [C]

I'm re-opening this ticket. OCECDR-4271 addresses the PP report while this ticket is addressing the vendor output that needs to be handled separately.

Comment entered 2017-08-14 15:16:29 by Englisch, Volker (NIH/NCI) [C]

Adding as a watcher. The fix for this ticket can be done as a filter change.

Comment entered 2017-08-16 16:47:32 by Englisch, Volker (NIH/NCI) [C]

The following filter has been updated to fix the HP/Patient toggle issue:

  • R14889: Vendor Filter: Summary (CDR000157)

The filter had been updated on QA and a diff report showed the updated SummaryToggleURL as the only changes.
The changes have been copied to STAGE and PROD.

Please verify on PROD and close this ticket.

Comment entered 2017-09-14 16:23:23 by Englisch, Volker (NIH/NCI) [C]

, is there anything that needs to be tested on PROD or can we close this ticket?

Comment entered 2017-10-27 13:02:06 by Englisch, Volker (NIH/NCI) [C]

Could I ask again if this ticket could be closed? It's in production since August.

Comment entered 2017-10-27 13:05:52 by Osei-Poku, William (NIH/NCI) [C]

Yes. I'll close it right away. Sorry for the delay.

Elapsed: 0:00:00.001312