CDR Tickets

Issue Number 3354
Summary Modify Publishing Status Report to Display All Error Messages
Created 2011-05-06 10:00:07
Issue Type Improvement
Submitted By Osei-Poku, William (NIH/NCI) [C]
Assigned To Englisch, Volker (NIH/NCI) [C]
Status Closed
Resolved 2011-06-07 15:50:22
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.107682
Description

BZISSUE::5047
BZDATETIME::2011-05-06 10:00:07
BZCREATOR::William Osei-Poku
BZASSIGNEE::Volker Englisch
BZQACONTACT::William Osei-Poku

Kim reported three InScope trials that did not get sent to ctgov because they had sites in the protocol that were not linked to the organization records. This is a known issue in the protocol schema with regards to links. I will put in another issue to have this taken care of in the protocol schema. But this issue is to fix the report so that it will report this error prominently (like the other errors).

Comment entered 2011-05-06 10:49:58 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2011-05-06 10:49:58
BZCOMMENTOR::Volker Englisch
BZCOMMENT::1

I've updated the title to better reflect the actual problem.

Comment entered 2011-05-25 17:04:32 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2011-05-25 17:04:32
BZCOMMENTOR::Volker Englisch
BZCOMMENT::2

The error report has been modified to correctly identify XSLT errors and display those on the "Error" page.
PubStatus.py - R10094

This can be tested on FRANCK since it had recently been refreshed. Use the link
http://bach.nci.nih.gov/cgi-bin/cdr/PubStatus.py?id=8673&type=FilterFailure&flavor=error

and

http://franck.nci.nih.gov/cgi-bin/cdr/PubStatus.py?id=8673&type=FilterFailure&flavor=error

to compare the output result.

Comment entered 2011-06-01 12:21:40 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-06-01 12:21:40
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::3

The link to Bach doesn't show any errors but the one to Franck shows that three protocols have XSLT errors. However, it is difficult to interpret the errors:

"XSLT error: code: 61 msgtype:error code:61 interpret URI:cdr:CDR0000699481#_1/last line:42 node:attribute 'encoding' msg:could not open document 'cdr:/last'"

Comment entered 2011-06-01 13:10:42 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2011-06-01 13:10:42
BZCOMMENTOR::Volker Englisch
BZCOMMENT::4

(In reply to comment #3)
> The link to Bach doesn't show any errors but the one to Franck shows that
> three protocols have XSLT errors.

Exactly. On BACH we have the old program which does not display the three protocols with error and the link to FRANCK does display the errors since you're running the new program. The before and after should be different.

> However, it is difficult to interpret the errors:

That's OK. This report is meant to notify you of any problems. It's not meant to tell you exactly of what the problem is. It would be nice but error message aren't always user friendly.

Comment entered 2011-06-07 11:30:13 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-06-07 11:30:13
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::5

(In reply to comment #4)
> (In reply to comment #3)
> > The link to Bach doesn't show any errors but the one to Franck shows that
> > three protocols have XSLT errors.
> Exactly. On BACH we have the old program which does not display the three
> protocols with error and the link to FRANCK does display the errors since
> you're running the new program. The before and after should be different.
> > However, it is difficult to interpret the errors:
> That's OK. This report is meant to notify you of any problems. It's not meant
> to tell you exactly of what the problem is. It would be nice but error message
> aren't always user friendly.
In that case, I think we are good. Also, we changed the schema so that affected documents will fail validation.

Comment entered 2011-06-07 11:30:41 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-06-07 11:30:41
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::6

Closing this issue. Thanks!

Comment entered 2011-06-07 11:31:48 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-06-07 11:31:48
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::7

(In reply to comment #6)
> Closing this issue. Thanks!

oops. I just realized that you may have to promote this to Bach before closing this issue?

Comment entered 2011-06-07 14:46:20 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2011-06-07 14:46:20
BZCOMMENTOR::Volker Englisch
BZCOMMENT::8

The report has been copied to FRANCK and BACH:
PubStatus.py - R10094

Please verify on BACH and close this bug.

Comment entered 2011-06-07 15:50:22 by Osei-Poku, William (NIH/NCI) [C]

BZDATETIME::2011-06-07 15:50:22
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::9

(In reply to comment #8)
> The report has been copied to FRANCK and BACH:
> PubStatus.py - R10094
> Please verify on BACH and close this bug.

(In reply to comment #8)
> The report has been copied to FRANCK and BACH:
> PubStatus.py - R10094
> Please verify on BACH and close this bug.

The link to Bach now shows the errors.
Verified. Closing issue. Thank you!

Elapsed: 0:00:00.002094