CDR Tickets

Issue Number 3912
Summary Linked Docs macro issue
Created 2015-05-21 12:50:35
Issue Type Inquiry
Submitted By Osei-Poku, William (NIH/NCI) [C]
Assigned To alan
Status Closed
Resolved 2015-06-09 16:56:43
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.161728
Description

The Linked Docs macro on the Organization and Person toolbars no longer retrieves the linked docs automatically. Rather, it brings up a form that you need to enter the CDR ID and/or additional information in order to get the linked docs report. Can this be changed so that it will automatically retrieve the linked docs?

Comment entered 2015-06-09 15:16:50 by alan

The problem was caused by a code variable name cleanup that wasn't aware of the XMetal macro sending in the old variable name.

It was an easy fix involving only one line of code that could not possibly break anything (he he he).. I've made it on DEV. If we include it in Curie it can go out with that release. Otherwise it has to go with Darwin.

I'll put it on QA for testing but not check it in to the svn Curie release (to be created) unless and until someone says it should deploy with Curie.

Comment entered 2015-06-09 16:56:43 by alan

Issue is resolved, release deployment version not yet set.

Comment entered 2015-06-09 17:00:42 by alan

I installed this on QA and it works for me. It will likely disappear when the changes for Egyptian Mau are deployed to QA but, in the meantime, it can be tested.

On my system the program launched Internet Explorer but it came up minimized rather than visibly in the foreground. But that has been a general problem not specific to this issue.

I'm marking it as resolved-fixed, ready for QA.

Comment entered 2015-06-09 17:10:13 by Kline, Bob (NIH/NCI) [C]

The CDR changes for Egyptian Mau were deployed to QA a while ago, so you fix is safe there.

Comment entered 2015-06-09 18:26:13 by Osei-Poku, William (NIH/NCI) [C]

Verified on QA. Thank you!

Comment entered 2015-07-09 17:53:17 by alan

For the record, the change to implement this is in the file:

\Inetpub\wwwroot\cgi-bin\cdr\LinkedDocs.py

The change was:

From: doc_id = fields.getvalue("doc_id")
  To: doc_id = fields.getvalue("doc_id") or fields.getvalue("DocId")
Comment entered 2016-02-23 10:10:05 by alan

It looks like this fix got included at the last minute in the Curie release. It's already in production.

Comment entered 2016-03-17 11:29:56 by Osei-Poku, William (NIH/NCI) [C]

Verified on PROD. Thanks!

Elapsed: 0:00:00.001532