CDR Tickets

Issue Number 3823
Summary Two reports not working on DEV
Created 2014-11-05 13:09:08
Issue Type Bug
Submitted By Osei-Poku, William (NIH/NCI) [C]
Assigned To Englisch, Volker (NIH/NCI) [C]
Status Closed
Resolved 2014-11-05 16:32:19
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.141193
Description

The "General Reports" and "Citations" links are not working on the report menu on DEV.
I get a python script error each time I click on the link.

"A problem occurred in a Python script.

D:\cdr\Log\tmprvcp7x.html contains the description of this error.
"

"A problem occurred in a Python script.

D:\cdr\Log\tmpx0is6y.html contains the description of this error.
"

Comment entered 2014-11-05 14:23:07 by Kline, Bob (NIH/NCI) [C]

I need to merge trunk into Ampere, and sweep through to make sure all the changes that were made for the security holes are running on DEV. I won't do that until I hear back from both Volker and Alan that it's OK for me to proceed with that merge.

Comment entered 2014-11-05 14:37:05 by Englisch, Volker (NIH/NCI) [C]

I have never done a merge myself so I'm not exactly sure how safe this merge would be. Currently I have only two files in my (Ampere) sandbox that are not versioned:

M       Filters\CDR0000000157.xml
M       etc\cdrapphosts.rc

Hopefully that answers your concerns.

Comment entered 2014-11-05 14:43:52 by Kline, Bob (NIH/NCI) [C]

I think we're going to be OK. I'm going to proceed with the merge.

Comment entered 2014-11-05 16:32:19 by Kline, Bob (NIH/NCI) [C]

I think the report links should be OK now, William. Please give them a shot.

Volker and Alan:

The merge of trunk changes into Ampere is done and has been committed. You should do an "svn up" in your Ampere sandboxes. There was only one spot I had to resolve manually. Volker had added a missing argument to a string interpolation for the code that explains that we don't have a filter for the requested document type, and I had added cgi.escape() calls around the arguments that were already there (as part of the security fixes). Since those happened on the same line of code, there was no was Subversion was going to figure out what to do. Fortunately, it was obvious how to merge the two changes and all should be well. Let me know if you have any problems in your sandboxes.

I have also brought the code in d:\Inetpub\wwwroot\cgi-bin\cdr and in d:\cdr\lib\Python in sync with what's in the Ampere branch of Subversion (except that I didn't remove any of the "extras" - like the DADA files 🙂 ). If anything I replaced needs to be retrieved, look in d:\cdr\backups\20141105.

Comment entered 2014-11-05 17:48:25 by Osei-Poku, William (NIH/NCI) [C]

I think the report links should be OK now, William. Please give them a shot.

Yes, the reports are working now. Thank you!

Comment entered 2014-11-05 17:49:58 by Osei-Poku, William (NIH/NCI) [C]

Verified on DEV. Since Bob addressed some of his comments to Alan and Volker, I will not close this issue until you tell me to.

Comment entered 2014-11-05 18:20:22 by Kline, Bob (NIH/NCI) [C]

You can close the ticket. They have the information they need.

Elapsed: 0:00:00.001930