457 early pre published alert pkp integration into sdpi a

.ci/asciidoc-converter/src/main/kotlin/org/sdpi/asciidoc/extension/SdpiInformationCollector.kt

@@ -1129,6 +1129,11 @@ class SdpiInformationCollector(
         }
     }
 
+    /*
+    Matches actors with contributions for transactions referenced from profiles using
+    the syntax in the form:
+    sdpi_include_transaction::DEV-23[initiator="required", receiver="optional"]
+     */
     private fun linkActorsToTransactionReferences() {
         for (profile in profiles.values) {
             linkActorsToTransactionReferences(profile.transactionReferences)
@@ -1147,10 +1152,15 @@ class SdpiInformationCollector(
                     for (obl in ref.obligations) {
                         if (obl.actorId == null) {
                             val contrib = obl.contribution
-                            val role = transaction.actorRoles?.firstOrNull { it.contribution == contrib }
-                            if (role != null) {
-                                val strActorId = role.actorId
-                                obl.actorId = strActorId
+                            if (transaction.actorRoles != null) {
+                                val matchingRoles = transaction.actorRoles.filter{it.contributions.contains(contrib)}
+                                check(matchingRoles.size <= 1) {
+                                    logger.error("More than one actor contributes $contrib. Specify actor in $BLOCK_MACRO_NAME_INCLUDE_TRANSACTION")
+                                }
+                                if (matchingRoles.size == 1) {
+                                    val strActorId = matchingRoles[0].actorId
+                                    obl.actorId = strActorId
+                                }
                             }
                         }
                     }

.ci/asciidoc-converter/src/main/kotlin/org/sdpi/asciidoc/extension/TransactionActorsProcessor.kt

@@ -55,26 +55,32 @@ class TransactionActorsProcessor : BlockProcessor(BLOCK_NAME_TRANSACTION_ACTORS)
         val roles = mutableListOf<SdpiActorRole>()
 
         var strActor: String? = null
-        var contribution: Contribution? = null
+        var contributions = mutableListOf<Contribution>()
         val description = mutableListOf<String>()
 
         for (strLine in blockLines) {
             if (strLine.startsWith('[')) {
-                if (strActor != null && contribution != null && description.isNotEmpty()) {
-                    roles.add(SdpiActorRole(strActor, contribution, description.toList()))
+
+                // remove trailing blank lines.
+                while(description.isNotEmpty() && description.last().isEmpty()) {
+                    description.removeLast()
+                }
+
+                if (strActor != null && contributions.isNotEmpty() && description.isNotEmpty()) {
+                    roles.add(SdpiActorRole(strActor, contributions, description.toList()))
                 }
                 description.clear()
 
                 val result = parseContributorAttributes(strLine)
                 strActor = result.first
-                contribution = result.second
+                contributions = result.second.toMutableList()
             } else {
                 description.add(strLine)
             }
         }
 
-        if (strActor != null && contribution != null && description.isNotEmpty()) {
-            roles.add(SdpiActorRole(strActor, contribution, description.toList()))
+        if (strActor != null && contributions.isNotEmpty() && description.isNotEmpty()) {
+            roles.add(SdpiActorRole(strActor, contributions, description.toList()))
         }
 
         return roles
@@ -104,12 +110,12 @@ class TransactionActorsProcessor : BlockProcessor(BLOCK_NAME_TRANSACTION_ACTORS)
 
         for (actorRole in roles) {
             val strActorId = "RefActor:${actorRole.actorId}[]"
-            val contribution = actorRole.contribution
+            val contributions = actorRole.contributions
             val strDescription = actorRole.description.joinToString("\r\n")
 
             val row = createTableRow(roleTable)
             row.cells.add(createTableCell(colActor, strActorId))
-            row.cells.add(createTableCell(colContribution, contribution.keyword))
+            row.cells.add(createTableCell(colContribution, contributions.joinToString { it.keyword }))
 
             row.cells.add(createTableCell(colDescription, strDescription))
 
@@ -119,31 +125,37 @@ class TransactionActorsProcessor : BlockProcessor(BLOCK_NAME_TRANSACTION_ACTORS)
         return roleTable
     }
 
-    private fun parseContributorAttributes(strLine: String): Pair<String, Contribution> {
+    private fun parseContributorAttributes(strLine: String): Pair<String, List<Contribution>> {
         val attributes = parseAttributes(strLine)
 
-        val strActorId = attributes[Roles.Transaction.ACTOR_ID.key]
-        checkNotNull(strActorId) {
+        val actorId = attributes.firstOrNull{ it.first == Roles.Transaction.ACTOR_ID.key}
+        checkNotNull(actorId) {
             logger.error("Actor in transaction is missing an actor id")
         }
-        val strContribution = attributes[Roles.Transaction.CONTRIBUTION.key]
-        checkNotNull(strContribution) {
-            logger.error("Actor $strActorId in transaction is missing a contribution")
+        val strActorId = actorId.second
+        val contributions = mutableListOf<Contribution>()
+        for(contribution in attributes.filter{it.first == Roles.Transaction.CONTRIBUTION.key}) {
+            val strContribution = contribution.second
+            val cont = parseContribution(strContribution)
+            checkNotNull(cont) {
+                logger.error("Actor $strActorId's contribution '$strContribution' is not recognized")
+            }
+            contributions.add(cont)
         }
-        val contribution = parseContribution(strContribution)
-        checkNotNull(contribution) {
-            logger.error("Actor $strActorId's contribution '$strContribution' is not recognized")
+
+        check(contributions.isNotEmpty()) {
+            logger.error("Actor $strActorId in transaction is missing a contribution")
         }
 
-        return Pair(strActorId, contribution)
+        return Pair(strActorId, contributions)
     }
 
-    private fun parseAttributes(strValue: String): Map<String, String> {
-        val attributes = mutableMapOf<String, String>()
+    private fun parseAttributes(strValue: String): List<Pair<String, String>> {
+        val attributes = mutableListOf<Pair<String, String>>()
 
         for (match in RE_ATTRIBUTES.findAll(strValue)) {
             val (key, value) = match.destructured
-            attributes[key] = value
+            attributes.add(Pair<String,String>(key, value))
         }
 
         return attributes

.ci/asciidoc-converter/src/main/kotlin/org/sdpi/asciidoc/model/SdpiActor.kt

@@ -22,7 +22,7 @@ data class SdpiActor(
 @Serializable
 data class SdpiActorRole(
     val actorId: String,
-    val contribution: Contribution,
+    val contributions: List<Contribution>,
     val description: List<String>,
 ) {
 }

.ci/asciidoc-converter/src/main/kotlin/org/sdpi/asciidoc/model/SdpiTransaction.kt

@@ -19,28 +19,14 @@ data class SdpiTransaction(
         return "link:#$strAnchor[$id]"
     }
 
-    fun getContributionFor(contribution: Contribution): Contribution? {
-        if (actorRoles == null) {
-            return null
-        }
-
-        for(actor in actorRoles) {
-            if (actor.contribution == contribution) {
-                return actor.contribution
-            }
-        }
-
-        return null
-    }
-
-    fun getContributionForActor(strActorId: String): Contribution? {
+    fun getContributionForActor(strActorId: String): List<Contribution>? {
         if (actorRoles == null) {
             return null
         }
 
         for(actor in actorRoles) {
             if (actor.actorId == strActorId) {
-                return actor.contribution
+                return actor.contributions
             }
         }
 

CHANGELOG.md

@@ -18,9 +18,10 @@ Each section shall contain a list of action items of the following format: `<bri
 
 ### Added
 
-- Clarifying note to R7000 ([#501](https://github.com/IHE/DEV.SDPi/issues/501)).
-- Clarifying statements to DEV-23, DEV-24, DEV-46, and DEV-47 w.r.t to ad-hoc and managed mode discovery([#409](https://github.com/IHE/DEV.SDPi/issues/409)).
+- Clarifying note to R7000 ([#501](https://github.com/IHE/DEV.SDPi/issues/501))
+- Clarifying statements to DEV-23, DEV-24, DEV-46, and DEV-47 w.r.t to ad-hoc and managed mode discovery([#409](https://github.com/IHE/DEV.SDPi/issues/409))
 - OID Framework to TF-3Z ([#474](https://github.com/IHE/DEV.SDPi/issues/474))
+- Distributed Alarm System option and related transactions DEV-48 and DEV-49 (a.k.a. early "pre-published" integration of the SDC A-PKP standard) ([#457](https://github.com/IHE/DEV.SDPi/issues/457))
 - Make restriction of xsd:durations to hours, minutes, seconds, and fractional seconds mandatory for all xsd:duration values ([#517](https://github.com/IHE/DEV.SDPi/issues/517))
 
 ### Changes

articles/sdpi-article-ihe-tf-asciidoc-cookbook.adoc

@@ -475,7 +475,9 @@ or overriding obligations that apply when the actor option is selected for a par
 
 A profile, profile option or actor option references a <<semantic-transactions,transaction>> using the `sdpi_include_transaction` [include processor](https://docs.asciidoctor.org/asciidoctorj/latest/extensions/include-processor/) and declares the obligations of contributors. 
 
-There are two forms for `sdpi_include_transaction`. The first form defines obligations for the actor roles defined in the transaction. That is, only the contribution (`initiator`, `receiver`, `responder`) and obligation (`required`, `optional`) are required. It is not necessary to repeat the actor. For example:
+There are two forms for `sdpi_include_transaction`. The first form defines obligations for the actor roles defined in the transaction. That is, only the contribution (`initiator`, `receiver`, `responder`) and obligation (`required`, `optional`) are required. It is not necessary to repeat the actor provided the transaction actor obligation is not ambiguous. Transaction actor's obligations are ambiguous when more than one actor has the same obligation (e.g., two actors both initiate messages in the transaction).
+
+For example:
 
 [source, asciidoc]
 ----
@@ -629,6 +631,25 @@ Listens for vol2_clause_dev_23_message_hello messages to identify any <<vol1_spe
 
 Attaching the `transaction` role to a section marks it as a [transaction](https://profiles.ihe.net/DEV/SDPi/index.html#vol2). A (document-unique) identifier must be defined in the `DEV-23` attribute. The `reftext` attribute is used for the transaction label in tables, etc. 
 
+Actors may make multiple contributions to a transaction. For example:
+
+----
+[sdpi_transaction_actors]
+--
+
+[actor-id="somds_medical_alert_consumer",contribution=Initiator,contribution=Receiver]
+Requests a <<vol1_spec_sdpi_a_actor_somds_medical_alert_provider>> to end a Distributed Alarm System.
+Listens for notifications from a <<vol1_spec_sdpi_a_actor_somds_medical_alert_provider>> that end
+a Distributed Alarm System.
+
+[actor-id="somds_medical_alert_provider",contribution=Responder,contribution=Initiator]
+Listens for requests from a <<vol1_spec_sdpi_a_actor_somds_medical_alert_consumer>> to end a 
+Distributed Alarm System and grants such requests. Alternatively ends a Distributed Alarm System 
+on its own initiative and notifies the <<vol1_spec_sdpi_a_actor_somds_medical_alert_consumer>>.
+
+--
+----
+
 The transactions object identifier (oid) may <<defining-oids,be set>> using the `oid-arcs` attribute. If not present, object identifiers will 
 be set automatically from the transaction id. When assigned automatically, two oids will be applied to each transaction: