fhirsmith 0.12.0 → 0.13.0

package/tx/workers/compare.js

@@ -11,7 +11,7 @@
 const { TerminologyWorker } = require('./worker');
 const {TxParameters} = require("../params");
 const {Extensions} = require("../library/extensions");
-const {Issue, OperationOutcome} = require("../library/operation-outcome");
+const { Issue, OperationOutcome, buildOperationOutcome, outcomeFromError } = require('../library/operation-outcome');
 const ValueSet = require("../library/valueset");
 const {ValueSetExpander} = require("./expand");
 const {SearchFilterText} = require("../library/designations");
@@ -59,18 +59,7 @@ class CompareWorker extends TerminologyWorker {
         oo.addIssue(error);
         return res.status(error.statusCode || 500).json(oo.jsonObj);
       } else {
-        const issueCode = error.issueCode || 'exception';
-        return res.status(statusCode).json({
-          resourceType: 'OperationOutcome',
-          issue: [{
-            severity: 'error',
-            code: issueCode,
-            details: {
-              text: error.message
-            },
-            diagnostics: error.message
-          }]
-        });
+        return res.status(statusCode).json(outcomeFromError(error));
       }
     }
   }
@@ -89,18 +78,7 @@ class CompareWorker extends TerminologyWorker {
       debugLog(error);
       req.logInfo = this.usedSources.join("|")+" - error"+(error.msgId  ? " "+error.msgId : "");
       const statusCode = error.statusCode || 500;
-      const issueCode = error.issueCode || 'exception';
-      return res.status(statusCode).json({
-        resourceType: 'OperationOutcome',
-        issue: [{
-          severity: 'error',
-          code: issueCode,
-          details: {
-            text : error.message
-          },
-          diagnostics: error.message
-        }]
-      });
+      return res.status(statusCode).json(outcomeFromError(error));
     }
   }
 
@@ -158,15 +136,9 @@ class CompareWorker extends TerminologyWorker {
    * @param {string} message - Diagnostic message
    * @returns {Object} OperationOutcome resource
    */
-  operationOutcome(severity, code, message) {
-    return {
-      resourceType: 'OperationOutcome',
-      issue: [{
-        severity,
-        code,
-        diagnostics: message
-      }]
-    };
+  operationOutcome(severity, code, message, txIssueType = null) {
+    // the shared builder, so that every outcome has details.text and a tx-issue-type coding
+    return buildOperationOutcome(severity, code, message, txIssueType);
   }
 
   async readValueSet(res, prefix, params) {

package/tx/workers/expand.js

@@ -14,7 +14,7 @@ const {Designations, SearchFilterText} = require("../library/designations");
 const {Extensions} = require("../library/extensions");
 const {getValuePrimitive, getValueName, validateParameter} = require("../../library/utilities");
 const {div} = require("../../library/html");
-const {Issue, OperationOutcome} = require("../library/operation-outcome");
+const { Issue, OperationOutcome, buildOperationOutcome, outcomeFromError } = require('../library/operation-outcome');
 const crypto = require('crypto');
 const ValueSet = require("../library/valueset");
 const {VersionUtilities} = require("../../library/version-utilities");
@@ -447,8 +447,21 @@ class ValueSetExpander {
           }
         }
 
+        // '*' asks for every property the code system defines for the concept (R6's
+        // ValueSet.compose.property allows it, and a request can name it too). It is handled
+        // here rather than by expanding it into a list of names, because which properties
+        // exist is a property of the concept, not of the request
+        if (this.params.properties.includes('*') && csProps != null && cs != null) {
+          for (const cp of csProps) {
+            const vn = getValueName(cp);
+            this.defineProperty(expansion, n, this.getPropUrl(cs, cp.code, cp), cp.code, vn, cp[vn], true);
+          }
+        }
+
         for (const pn of this.params.properties) {
-          if (pn === 'definition') {
+          if (pn === '*') {
+            continue; // done above
+          } else if (pn === 'definition') {
             if (definition) {
               this.defineProperty(expansion, n, 'http://hl7.org/fhir/concept-properties#definition', pn, "valueString", definition);
             }
@@ -1094,7 +1107,7 @@ class ValueSetExpander {
           this.worker.opContext.log('prep filters');
           const prep = await cs.getPrepContext(true);
           if (!filter.isNull) {
-            await cs.searchFilter(filter, prep, true);
+            await cs.searchFilter(prep, filter, true);
           }
 
           if (cs.specialEnumeration()) {
@@ -1699,16 +1712,23 @@ class ValueSetExpander {
     if (!expansion.property) {
       expansion.property = [];
     }
-    let pd = expansion.property.find(t1 => t1.uri == url || t1.code == code);
+    // match on the uri only when there is one. A code system may declare several properties with
+    // no uri at all, and those are told apart by their code - matching on an undefined uri would
+    // collapse all of them into whichever one was declared first
+    let pd = url
+      ? expansion.property.find(t1 => t1.uri == url || t1.code == code)
+      : expansion.property.find(t1 => t1.code == code);
     if (!pd) {
       pd = {};
       expansion.property.push(pd);
-      pd.uri = url;
       pd.code = code;
-    } else if (!pd.uri) {
+      if (url) {
+        pd.uri = url;
+      }
+    } else if (url && !pd.uri) {
       pd.uri = url;
     }
-    if (pd.uri != url) {
+    if (url && pd.uri != url) {
       throw new Error('URL mismatch on expansion: ' + pd.uri + ' vs ' + url + ' for code ' + code);
     }
     return pd.code;
@@ -1738,7 +1758,7 @@ class ValueSetExpander {
       }
     }
     for (const definition of definitions) {
-      if (definition.uri && used.has(definition.code)) {
+      if (used.has(definition.code)) {
         this.declareProperty(expansion, definition.uri, definition.code);
       }
     }
@@ -1754,10 +1774,9 @@ class ValueSetExpander {
     if (value === undefined || value == null) {
       return;
     }
-    // we only define it if the code system has a definition
-    if (url) {
-      code = this.declareProperty(expansion, url, code);
-    }
+    // every property that appears in contains.property has to be declared in expansion.property,
+    // whether or not the code system gave it a uri
+    code = this.declareProperty(expansion, url, code);
 
     if (!contains.property) {
       contains.property = [];
@@ -1950,18 +1969,7 @@ class ExpandWorker extends TerminologyWorker {
         oo.addIssue(error);
         return res.status(error.statusCode || 500).json(oo.jsonObj);
       } else {
-        const issueCode = error.issueCode || 'exception';
-        return res.status(statusCode).json({
-          resourceType: 'OperationOutcome',
-          issue: [{
-            severity: 'error',
-            code: issueCode,
-            details: {
-              text: error.message
-            },
-            diagnostics: error.message
-          }]
-        });
+        return res.status(statusCode).json(outcomeFromError(error));
       }
     }
   }
@@ -1980,18 +1988,7 @@ class ExpandWorker extends TerminologyWorker {
       debugLog(error);
       req.logInfo = this.usedSources.join("|")+" - error"+(error.msgId  ? " "+error.msgId : "");
       const statusCode = error.statusCode || 500;
-      const issueCode = error.issueCode || 'exception';
-      return res.status(statusCode).json({
-        resourceType: 'OperationOutcome',
-        issue: [{
-          severity: 'error',
-          code: issueCode,
-          details: {
-            text : error.message
-          },
-          diagnostics: error.message
-        }]
-      });
+      return res.status(statusCode).json(outcomeFromError(error));
     }
   }
 
@@ -2253,15 +2250,9 @@ class ExpandWorker extends TerminologyWorker {
    * @param {string} message - Diagnostic message
    * @returns {Object} OperationOutcome resource
    */
-  operationOutcome(severity, code, message) {
-    return {
-      resourceType: 'OperationOutcome',
-      issue: [{
-        severity,
-        code,
-        diagnostics: message
-      }]
-    };
+  operationOutcome(severity, code, message, txIssueType = null) {
+    // the shared builder, so that every outcome has details.text and a tx-issue-type coding
+    return buildOperationOutcome(severity, code, message, txIssueType);
   }
 
 }

package/tx/workers/lookup.js

@@ -12,7 +12,7 @@ const { FhirCodeSystemProvider } = require('../cs/cs-cs');
 const { Designations} = require("../library/designations");
 const {TxParameters} = require("../params");
 const {Parameters} = require("../library/parameters");
-const {Issue, OperationOutcome} = require("../library/operation-outcome");
+const { Issue, OperationOutcome, buildOperationOutcome, outcomeFromError } = require('../library/operation-outcome');
 const {debugLog} = require("../operation-context");
 const {checkAddedParameters} = require("../library/parameter-check");
 
@@ -49,16 +49,7 @@ class LookupWorker extends TerminologyWorker {
       this.log.error(error);
       debugLog(error);
       req.logInfo = this.usedSources.join("|")+" - error"+(error.msgId  ? " "+error.msgId : "");
-      const statusCode = error.statusCode || 500;
-      const issueCode = error.issueCode || 'exception';
-      return res.status(statusCode).json({
-        resourceType: 'OperationOutcome',
-        issue: [{
-          severity: 'error',
-          code: issueCode,
-          diagnostics: error.message
-        }]
-      });
+      return res.status(error.statusCode || 500).json(outcomeFromError(error));
     }
   }
 
@@ -76,15 +67,7 @@ class LookupWorker extends TerminologyWorker {
       this.log.error(error);
       debugLog(error);
       req.logInfo = this.usedSources.join("|")+" - error"+(error.msgId  ? " "+error.msgId : "");
-      const issueCode = error.issueCode || 'exception';
-      return res.status(400).json({
-        resourceType: 'OperationOutcome',
-        issue: [{
-          severity: 'error',
-          code: issueCode,
-          diagnostics: error.message
-        }]
-      });
+      return res.status(400).json(outcomeFromError(error));
     }
   }
 
@@ -140,8 +123,11 @@ class LookupWorker extends TerminologyWorker {
       }
 
       if (!csProvider) {
-        const systemUrl = params.system || params.coding?.system;
-        const versionStr = params.version || params.coding?.version;
+        // params is a Map-like: params.system is always undefined, so the message read
+        // "CodeSystem not found: undefined" for every unknown system or version.
+        const coding = params.has('coding') ? params.get('coding') : null;
+        const systemUrl = params.has('system') ? params.get('system') : coding?.system;
+        const versionStr = params.has('version') ? params.get('version') : coding?.version;
         const msg = versionStr
           ? `CodeSystem not found: ${systemUrl} version ${versionStr}`
           : `CodeSystem not found: ${systemUrl}`;
@@ -169,8 +155,7 @@ class LookupWorker extends TerminologyWorker {
         oo.addIssue(error);
         return res.status(error.statusCode || 500).json(oo.jsonObj);
       } else {
-        return res.status(error.statusCode || 500).json(this.operationOutcome(
-          'error', error.issueCode || 'exception', error.message));
+        return res.status(error.statusCode || 500).json(outcomeFromError(error));
       }
     }
   }
@@ -233,8 +218,7 @@ class LookupWorker extends TerminologyWorker {
         oo.addIssue(error);
         return res.status(error.statusCode || 500).json(oo.jsonObj);
       } else {
-        return res.status(error.statusCode || 500).json(this.operationOutcome(
-          'error', error.issueCode || 'exception', error.message));
+        return res.status(error.statusCode || 500).json(outcomeFromError(error));
       }
     }
   }
@@ -271,7 +255,9 @@ class LookupWorker extends TerminologyWorker {
       if (locateResult?.message) {
         message += ' ('+locateResult.message+')';
       }
-      throw new Issue('error', 'not-found', null, null, message, null, 404);
+      // invalid-code, not not-found: the code system was found, the code in it was not.
+      // not-found is for a code system or value set that could not be resolved at all.
+      throw new Issue('error', 'not-found', null, null, message, 'invalid-code', 404);
     }
 
     const ctxt = locateResult.context;
@@ -441,21 +427,17 @@ class LookupWorker extends TerminologyWorker {
   }
 
   /**
-   * Build an OperationOutcome
+   * Build an OperationOutcome. Delegates to the shared builder so that every outcome this
+   * server emits has details.text and a tx-issue-type coding: diagnostics is stripped by
+   * the test harness, so nothing a caller needs may live there.
    * @param {string} severity - error, warning, information
-   * @param {string} code - Issue code
-   * @param {string} message - Diagnostic message
+   * @param {string} code - FHIR issue type
+   * @param {string} message - the human readable account of the problem
+   * @param {string} [txIssueType] - tx-issue-type; defaulted from code when not given
    * @returns {Object} OperationOutcome resource
    */
-  operationOutcome(severity, code, message) {
-    return {
-      resourceType: 'OperationOutcome',
-      issue: [{
-        severity,
-        code,
-        details: {text : message}
-      }]
-    };
+  operationOutcome(severity, code, message, txIssueType = null) {
+    return buildOperationOutcome(severity, code, message, txIssueType);
   }
 }
 

package/tx/workers/read.js

@@ -7,6 +7,7 @@
 const { TerminologyWorker } = require('./worker');
 const {debugLog} = require("../operation-context");
 
+const { buildOperationOutcome } = require('../library/operation-outcome');
 class ReadWorker extends TerminologyWorker {
   /**
    * @param {OperationContext} opContext - Operation context
@@ -50,27 +51,13 @@ class ReadWorker extends TerminologyWorker {
           return await this.handleConceptMap(req, res, id);
 
         default:
-          return res.status(404).json({
-            resourceType: 'OperationOutcome',
-            issue: [{
-              severity: 'error',
-              code: 'not-found',
-              diagnostics: `Unknown resource type: ${resourceType}`
-            }]
-          });
+          return res.status(404).json(buildOperationOutcome('error', 'not-found', `Unknown resource type: ${resourceType}`));
       }
     } catch (error) {
       this.log.error(error);
       debugLog(error);
       req.logInfo = this.usedSources.join("|")+" - error"+(error.msgId  ? " "+error.msgId : "");
-      return res.status(500).json({
-        resourceType: 'OperationOutcome',
-        issue: [{
-          severity: 'error',
-          code: 'exception',
-          diagnostics: error.message
-        }]
-      });
+      return res.status(500).json(buildOperationOutcome('error', 'exception', error.message));
     }
   }
 
@@ -127,14 +114,7 @@ class ReadWorker extends TerminologyWorker {
       }
     }
 
-    return res.status(404).json({
-      resourceType: 'OperationOutcome',
-      issue: [{
-        severity: 'error',
-        code: 'not-found',
-        diagnostics: `CodeSystem/${id} not found`
-      }]
-    });
+    return res.status(404).json(buildOperationOutcome('error', 'not-found', `CodeSystem/${id} not found`));
   }
 
   /**
@@ -151,14 +131,7 @@ class ReadWorker extends TerminologyWorker {
       }
     }
 
-    return res.status(404).json({
-      resourceType: 'OperationOutcome',
-      issue: [{
-        severity: 'error',
-        code: 'not-found',
-        diagnostics: `ValueSet/${id} not found`
-      }]
-    });
+    return res.status(404).json(buildOperationOutcome('error', 'not-found', `ValueSet/${id} not found`));
   }
   /**
    * Handle ConceptMap read
@@ -174,14 +147,7 @@ class ReadWorker extends TerminologyWorker {
       }
     }
 
-    return res.status(404).json({
-      resourceType: 'OperationOutcome',
-      issue: [{
-        severity: 'error',
-        code: 'not-found',
-        diagnostics: `ConceptMap/${id} not found`
-      }]
-    });
+    return res.status(404).json(buildOperationOutcome('error', 'not-found', `ConceptMap/${id} not found`));
   }
 }
 

package/tx/workers/search.js

@@ -10,6 +10,7 @@ const {Utilities} = require("../../library/utilities");
 const {debugLog} = require("../operation-context");
 const {acceptsHtml} = require("../tx-html");
 
+const { buildOperationOutcome } = require('../library/operation-outcome');
 class SearchWorker extends TerminologyWorker {
   /**
    * @param {OperationContext} opContext - Operation context
@@ -127,14 +128,7 @@ class SearchWorker extends TerminologyWorker {
       this.log.error(error);
       debugLog(error);
       req.logInfo = "error "+(error.msgId || error.className);
-      return res.status(500).json({
-        resourceType: 'OperationOutcome',
-        issue: [{
-          severity: 'error',
-          code: 'exception',
-          diagnostics: error.message
-        }]
-      });
+      return res.status(500).json(buildOperationOutcome('error', 'exception', error.message));
     }
   }
 

package/tx/workers/subsumes.js

@@ -7,7 +7,7 @@
 // POST /CodeSystem/{id}/$subsumes
 //
 
-const { TerminologyWorker } = require('./worker');
+const { TerminologyWorker, Unknown_Code_in_VersionSCT, SCTVersion } = require('./worker');
 const { FhirCodeSystemProvider } = require('../cs/cs-cs');
 const {TxParameters} = require("../params");
 const {Parameters} = require("../library/parameters");
@@ -51,8 +51,7 @@ class SubsumesWorker extends TerminologyWorker {
         oo.addIssue(error);
         return res.status(error.statusCode || 500).json(oo.jsonObj);
       } else {
-        return res.status(error.statusCode || 500).json(this.operationOutcome(
-          'error', error.issueCode || 'exception', error.message));
+        return res.status(error.statusCode || 500).json(this.unexpectedErrorOutcome(error));
       }
     }
   }
@@ -74,8 +73,7 @@ class SubsumesWorker extends TerminologyWorker {
         oo.addIssue(error);
         return res.status(error.statusCode || 500).json(oo.jsonObj);
       } else {
-        return res.status(error.statusCode || 500).json(this.operationOutcome(
-          'error', error.issueCode || 'exception', error.message));
+        return res.status(error.statusCode || 500).json(this.unexpectedErrorOutcome(error));
       }
     }
   }
@@ -87,13 +85,14 @@ class SubsumesWorker extends TerminologyWorker {
   async handleTypeLevelSubsumes(req, res) {
     this.deadCheck('subsumes-type-level');
 
-    // Handle tx-resource and cache-id parameters from Parameters resource
-    if (req.body && req.body.resourceType === 'Parameters') {
-      this.setupAdditionalResources(req.body);
-    }
-
-    // Parse parameters from request
+    // Parse the parameters first, then hand them to setupAdditionalResources. It used to be
+    // handed req.body, which meant a GET (no body) never reached the cache-id handling: when a
+    // client front-loads a suite's resources into a server-side cache the request carries no
+    // tx-resource at all, only the X-Cache-Id header, and every code system supplied that way
+    // was invisible. On a GET parseParameters turns the query string into the same Parameters
+    // shape, and the cache-id comes off the operation context
     const params = new Parameters(this.parseParameters(req));
+    this.setupAdditionalResources(params.jsonObj);
     const txp = new TxParameters(this.opContext.i18n.languageDefinitions, this.opContext.i18n);
     txp.readParams(params.jsonObj);
 
@@ -101,7 +100,9 @@ class SubsumesWorker extends TerminologyWorker {
     let codingA, codingB;
     let csProvider;
 
+    let names = ['codeA', 'codeB'];
     if (params.has('codingA') && params.has('codingB')) {
+      names = ['codingA', 'codingB'];
       // Using codingA and codingB (only from Parameters resource)
       codingA = params.get('codingA');
       codingB = params.get('codingB');
@@ -138,7 +139,7 @@ class SubsumesWorker extends TerminologyWorker {
     }
 
     // Perform the subsumes check
-    const result = await this.doSubsumes(csProvider, codingA, codingB);
+    const result = await this.doSubsumes(csProvider, codingA, codingB, txp, names);
     req.logInfo = this.usedSources.join("|")+txp.logInfo();
     return res.status(200).json(result);
   }
@@ -159,13 +160,9 @@ class SubsumesWorker extends TerminologyWorker {
       throw new Issue('error', 'not found', null, null, `CodeSystem/${id} not found`, null, 404);
     }
 
-    // Handle tx-resource and cache-id parameters from Parameters resource
-    if (req.body && req.body.resourceType === 'Parameters') {
-      this.setupAdditionalResources(req.body);
-    }
-
-    // Parse parameters from request
+    // See handleTypeLevelSubsumes: parse first, so a GET reaches the cache-id handling too
     const params = new Parameters(this.parseParameters(req));
+    this.setupAdditionalResources(params.jsonObj);
     const txp = new TxParameters(this.opContext.i18n.languageDefinitions, this.opContext.i18n);
     txp.readParams(params.jsonObj);
 
@@ -178,7 +175,9 @@ class SubsumesWorker extends TerminologyWorker {
     // Get the codings
     let codingA, codingB;
 
+    let names = ['codeA', 'codeB'];
     if (params.has('codingA') && params.has('codingB')) {
+      names = ['codingA', 'codingB'];
       codingA = params.get('codingA');
       codingB = params.get('codingB');
     } else if (params.has('codeA') && params.has('codeB')) {
@@ -198,7 +197,7 @@ class SubsumesWorker extends TerminologyWorker {
     }
 
     // Perform the subsumes check
-    const result = await this.doSubsumes(csProvider, codingA, codingB);
+    const result = await this.doSubsumes(csProvider, codingA, codingB, txp, names);
     req.logInfo = this.usedSources.join("|")+txp.logInfo();
     return res.json(result);
   }
@@ -263,9 +262,11 @@ class SubsumesWorker extends TerminologyWorker {
    * @param {CodeSystemProvider} csProvider - CodeSystem provider
    * @param {Object} codingA - First coding
    * @param {Object} codingB - Second coding
+   * @param {TxParameters} txp - parsed parameters (for languages)
+   * @param {Array<string>} names - the parameters the two codes came from, for `expression`
    * @returns {Object} Parameters resource with subsumes result
    */
-  async doSubsumes(csProvider, codingA, codingB) {
+  async doSubsumes(csProvider, codingA, codingB, txp, names = ['codeA', 'codeB']) {
     this.deadCheck('doSubsumes');
 
     const csSystem = csProvider.system();
@@ -287,18 +288,12 @@ class SubsumesWorker extends TerminologyWorker {
     // Validate both codes exist
     const locateA = await csProvider.locate(codingA.code);
     if (!locateA || !locateA.context) {
-      const error = new Error(`Invalid code: '${codingA.code}' not found in CodeSystem '${csSystem}'`);
-      error.statusCode = 404;
-      error.issueCode = 'not-found';
-      throw error;
+      throw this.unknownCodeIssue(csProvider, codingA.code, names[0], locateA ? locateA.message : null, txp);
     }
 
     const locateB = await csProvider.locate(codingB.code);
     if (!locateB || !locateB.context) {
-      const error = new Error(`Invalid code: '${codingB.code}' not found in CodeSystem '${csSystem}'`);
-      error.statusCode = 404;
-      error.issueCode = 'not-found';
-      throw error;
+      throw this.unknownCodeIssue(csProvider, codingB.code, names[1], locateB ? locateB.message : null, txp);
     }
 
     let equal = false;
@@ -324,21 +319,43 @@ class SubsumesWorker extends TerminologyWorker {
   }
 
   /**
-   * Build an OperationOutcome
-   * @param {string} severity - error, warning, information
-   * @param {string} code - Issue code
-   * @param {string} message - Diagnostic message
+   * Build the Issue for a code that isn't in the code system. Uses the same message ids
+   * and tx-issue-type detail code as $validate-code, so clients get consistent errors
+   * @param {CodeSystemProvider} csProvider - CodeSystem provider
+   * @param {string} code - the code that wasn't found
+   * @param {string} path - the parameter the code came from ('codeA' / 'codeB')
+   * @param {string} message - any explanation the provider gave for not finding it
+   * @param {TxParameters} txp - parsed parameters (for languages)
+   * @returns {Issue}
+   */
+  unknownCodeIssue(csProvider, code, path, message, txp) {
+    const system = csProvider.system();
+    const version = csProvider.version();
+    const msgId = Unknown_Code_in_VersionSCT(system, version);
+    const langs = txp ? txp.HTTPLanguages : null;
+    const msg = this.i18n.translate(msgId, langs, [code, system, version, SCTVersion(system, version)]);
+    const issue = new Issue('error', 'code-invalid', path, msgId, msg, 'invalid-code', 404);
+    if (message) {
+      issue.withDiagnostics(message);
+    }
+    return issue;
+  }
+
+  /**
+   * Wrap an error we did not expect as an OperationOutcome. The message goes in details.text,
+   * not diagnostics: TxTester strips diagnostics from every issue it compares, so anything
+   * reported that way is invisible in a failing test - all you see is 'exception'
+   * @param {Error} error - the error that escaped
    * @returns {Object} OperationOutcome resource
    */
-  operationOutcome(severity, code, message) {
-    return {
-      resourceType: 'OperationOutcome',
-      issue: [{
-        severity,
-        code,
-        diagnostics: message
-      }]
-    };
+  unexpectedErrorOutcome(error) {
+    // txIssueType, when the thrower set one, becomes the details.coding - that is where
+    // a client looks for the machine-readable reason (e.g. cannot-determine).
+    const issue = new Issue('error', error.issueCode || 'exception', null, null,
+      error.message || String(error), error.txIssueType || null, error.statusCode || 500);
+    const oo = new OperationOutcome();
+    oo.addIssue(issue);
+    return oo.jsonObj;
   }
 }