CDR Tickets

Issue Number 4686
Summary [Media] Images don't show up in PP until published
Created 2019-10-29 14:49:47
Issue Type Improvement
Submitted By Osei-Poku, William (NIH/NCI) [C]
Assigned To Kline, Bob (NIH/NCI) [C]
Status Closed
Resolved 2019-11-25 15:02:44
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.251770
Description

We've noticed that images don't show up in Summary PP until they have been published to Cancer.gov. The images only display a placeholder in the summary document until the image document has been hot-fixed.

Here is an example on QA:
Media doc - CDR0000752132
Summary doc - CDR0000062923

The MedialLink for this example is the first in one in this summary.

Comment entered 2019-10-31 16:41:55 by Englisch, Volker (NIH/NCI) [C]

It appears that the URL for images is pointing to the URL on the Drupal server

https://www-qa.cancer.gov/images/cdr/live/CDR752132-750.jpg

instead of the image on the CDR server

https://cdr-qa.cancer.gov/cgi-bin/cdr/GetCdrImage.py?pp=Y&id=CDR0000752132-400.jpg

The replacement happens in postprocess() (line 109).

Comment entered 2019-10-31 16:47:49 by Kline, Bob (NIH/NCI) [C]

I suspected something like that.

Comment entered 2019-11-23 13:41:25 by Kline, Bob (NIH/NCI) [C]

Two questions:

  1. would it mess up your testing if I address this as part of Kepler, instead of waiting for Leibniz?

  2. can you confirm that what you want is for all the image links from a publish preview rendering of the summary to https://whatever.cancer.gov/images/cdr/live/CDRwhatever.jpg instead to go to the CDR server to fetch (and possibly scale) the image from the latest publishable version?

Comment entered 2019-11-25 12:56:41 by Osei-Poku, William (NIH/NCI) [C]

1. Would anything change for us to re-test what have been Kepler tested already or this would just be in addition to what we are testing? If it is the latter then, that should not be a problem.

2. Yes, we would want to see the publishable version of the image in the Pub preview.

Comment entered 2019-11-25 15:02:44 by Kline, Bob (NIH/NCI) [C]

The only thing changed is where the image links point, so I think the answer to the first question that nothing else should be affected.

Comment entered 2019-12-02 11:13:50 by Kline, Bob (NIH/NCI) [C]

This enhancement provides an effective workaround to sidestep the GateKeeper image failures reported in OCECDR-4704.

Comment entered 2019-12-09 15:36:01 by Osei-Poku, William (NIH/NCI) [C]

Verified on QA2. Thanks!

Comment entered 2020-02-26 14:29:27 by Osei-Poku, William (NIH/NCI) [C]

Verified on PROD. Thanks!

Elapsed: 0:00:00.001909