CDR Tickets

Issue Number 3644
Summary [Summaries] Module attributes - need to identify all modules (including stand-alone summaries)
Created 2013-08-16 08:58:50
Issue Type Improvement
Submitted By Juthe, Robin (NIH/NCI) [E]
Assigned To Kline, Bob (NIH/NCI) [C]
Status Closed
Resolved 2013-09-03 08:48:25
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.112016
Description

We would like to implement a way to identify documents that are designed to be used as both modules and stand-alone summaries. The current attribute on summary modules ("ModuleOnly=Yes") is only used when the document is NOT intended to be published as a stand-alone document. We may need to add a second attribute, but this needs more discussion to determine the best solution.

Comment entered 2013-08-20 13:44:35 by chengep

CDR Developer Discussion: This was given story points = 2 to (a) decide (with the users) which attribute change(s) to make, and to actually make the change(s).

Comment entered 2013-08-30 12:07:24 by Kline, Bob (NIH/NCI) [C]

I propose that we add a top-level optional attribute AvailableAsModule="Yes" as the safer of the approaches to this request, since we wouldn't have to search for all the software which uses the existing ModuleOnly attribute to find out what would need to be changed if we replaced it with an attribute which could identify all three possibilities in a single attribute (for example, Module with values "Always," "Optional," and "Never"), which would introduce extra work and risk. We don't have a separate issue for a report to show the different types of summaries, but that should be easy to handle with an ad-hoc query, which wouldn't need a separate ticket, nor coordination with CBIIT.

Does this proposed approach work for everyone?

Comment entered 2013-08-30 12:14:00 by Juthe, Robin (NIH/NCI) [E]

To clarify, are you suggesting this new attribute in addition to ModuleOnly=Yes?

Comment entered 2013-08-30 12:24:06 by Kline, Bob (NIH/NCI) [C]

That's right. Leaving the existing attribute alone avoids the risk that we'll inadvertently break something that depends on it.

Comment entered 2013-08-30 12:52:32 by Juthe, Robin (NIH/NCI) [E]

Margaret and I just discussed this and we think this is a good approach. The two attributes ought to give us enough flexibility to identify docs that can be used as modules but are not only modules.

We may want to make some changes down the road to limit SummaryModuleLink elements to only allow links to documents with this new attribute (rather than any summary). We may also want to take another look at the summaries reports that we just modified in OCECDR-3575 to determine whether we should revise the logic that adds "(Module)" after anything that has ModuleOnly=Yes.

Comment entered 2013-09-03 08:48:25 by Kline, Bob (NIH/NCI) [C]

Attribute installed on DEV.

Comment entered 2013-09-03 11:18:39 by Juthe, Robin (NIH/NCI) [E]

Verified new attribute in summary document on DEV. I added the new attribute to the Peutz-Jeghers module (CDR0000738176).

Comment entered 2013-09-16 12:20:10 by Englisch, Volker (NIH/NCI) [C]

We will need to modify our publishing document since the current SQL query for selecting summaries to be published will exclude Peutz-Jeghers due to its ModuleOnly=Yes attribute.

Comment entered 2013-09-16 12:35:06 by Kline, Bob (NIH/NCI) [C]

Peutz-Jeghers should be excluded if it has ModuleOnly=Yes. The new attribute only says it can be used as a module. Doesn't say anything about whether it can be published as a standalone summary.

Comment entered 2013-09-16 12:46:18 by Englisch, Volker (NIH/NCI) [C]

So, for Peutz-Jeghers to be published as both, a stand-alone summary and a module the ModuleOnly=Yes attribute would be removed and AvailableAsModule=Yes would be added, right?
I had thought the current setup - both attributes set to Yes - should indicate both publishing options.

Comment entered 2013-09-16 12:56:12 by Kline, Bob (NIH/NCI) [C]

There are three logical possibilities:

1. In the most common case, neither attribute is set. Such as summary cannot be used as a module, but is intended for standalone publication only.

2. ModuleOnly=Yes means the document cannot be published by itself, but only included in another summary. This attribute implies AvailableAsModule=Yes (though it would be cleaner for such documents to have both attributes set.

3. AvailableAsModule=Yes means the document can be included as part of another document. If the ModuleOnly attribute is not present, the document can also be published standalone.

Comment entered 2013-11-07 15:24:40 by Juthe, Robin (NIH/NCI) [E]

Verified the schema on QA.

Comment entered 2013-12-03 13:22:22 by Juthe, Robin (NIH/NCI) [E]

Verified the schema in production.

Elapsed: 0:00:00.001747