openchs-models 1.33.66 → 1.33.68

package/dist/AbstractEncounter.js

@@ -270,10 +270,6 @@ class AbstractEncounter extends _BaseEntity.default {
     const inCancelObservations = new _ObservationsHolder.default(this.cancelObservations).replaceMediaObservation(originalValue, newValue, conceptUUID);
     return inObservations || inCancelObservations;
   }
-  replaceObservation(originalValue, newValue) {
-    new _ObservationsHolder.default(this.observations).updateObservationBasedOnValue(originalValue, newValue);
-    new _ObservationsHolder.default(this.cancelObservations).updateObservationBasedOnValue(originalValue, newValue);
-  }
   getEntityTypeName() {
     return this.encounterType.name;
   }

package/dist/Individual.js

@@ -642,9 +642,6 @@ class Individual extends _BaseEntity.default {
   replaceMediaObservation(originalValue, newValue, conceptUUID) {
     return new _ObservationsHolder.default(this.observations).replaceMediaObservation(originalValue, newValue, conceptUUID);
   }
-  replaceObservation(originalValue, newValue) {
-    new _ObservationsHolder.default(this.observations).updateObservationBasedOnValue(originalValue, newValue);
-  }
 
   //TODO use polymorphism to avoid if checks based on this
   isPerson() {

package/dist/ObservationsHolder.js

@@ -24,46 +24,6 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 function _defineProperty(e, r, t) { return (r = _toPropertyKey(r)) in e ? Object.defineProperty(e, r, { value: t, enumerable: !0, configurable: !0, writable: !0 }) : e[r] = t, e; }
 function _toPropertyKey(t) { var i = _toPrimitive(t, "string"); return "symbol" == typeof i ? i : i + ""; }
 function _toPrimitive(t, r) { if ("object" != typeof t || !t) return t; var e = t[Symbol.toPrimitive]; if (void 0 !== e) { var i = e.call(t, r || "default"); if ("object" != typeof i) return i; throw new TypeError("@@toPrimitive must return a primitive value."); } return ("string" === r ? String : Number)(t); }
-// Helper function to find media observation by value
-const findMediaObservationByValue = (observations, targetValue) => {
-  if (!observations || !Array.isArray(observations)) return null;
-  return _lodash.default.find(observations, obs => {
-    // Check direct value
-    if (obs.getValue && obs.getValue() === targetValue) return true;
-
-    // Check coded values (for MultipleCodedValues)
-    const valueJSON = obs.valueJSON;
-    if (valueJSON && valueJSON.answer && Array.isArray(valueJSON.answer)) {
-      return valueJSON.answer.includes(targetValue);
-    }
-    return false;
-  });
-};
-
-// Helper function to update media value
-const updateMediaValue = (mediaObs, oldVal, newVal) => {
-  if (!mediaObs) return false;
-
-  // For coded values, we need to update the array content
-  if (mediaObs.valueJSON && mediaObs.valueJSON.answer && Array.isArray(mediaObs.valueJSON.answer)) {
-    const answerIndex = mediaObs.valueJSON.answer.indexOf(oldVal);
-    if (answerIndex >= 0) {
-      mediaObs.valueJSON.answer[answerIndex] = newVal;
-      return true;
-    }
-  } else {
-    // Regular value update
-    mediaObs.setValue(mediaObs.concept.getValueWrapperFor(newVal));
-    return true;
-  }
-  return false;
-};
-
-// Process each question group observation
-const processQuestionGroupObservations = (observations, oldVal, newVal) => {
-  const mediaObs = findMediaObservationByValue(observations, oldVal);
-  return mediaObs ? updateMediaValue(mediaObs, oldVal, newVal) : false;
-};
 class ObservationsHolder {
   constructor(observations) {
     this.observations = observations;
@@ -84,9 +44,6 @@ class ObservationsHolder {
   getObservation(concept) {
     return this.findObservation(concept);
   }
-  findObservationByValue(value) {
-    return _lodash.default.find(this.observations, observation => observation.getValue() === value);
-  }
 
   /*
   Called from the wizard on changes done by the user for primitive fields (including Date)
@@ -431,49 +388,6 @@ class ObservationsHolder {
     }
   }
 
-  //private
-  updateObservationBasedOnValue(oldValue, newValue) {
-    // Try to find at top level first
-    const observation = this.findObservationByValue(oldValue);
-    if (observation) {
-      observation.setValue(observation.concept.getValueWrapperFor(newValue));
-      return true;
-    }
-
-    // Search in nested structures if not found at top level
-    let updated = false;
-
-    // Iterate through all observations
-    _lodash.default.forEach(this.observations, obs => {
-      // Skip non-question group observations
-      if (obs.concept.datatype !== _Concept.default.dataType.QuestionGroup) return;
-      const valueWrapper = obs.getValueWrapper && obs.getValueWrapper();
-      if (!valueWrapper) return;
-
-      // Handle RepeatableQuestionGroup
-      if (valueWrapper.isRepeatable && valueWrapper.isRepeatable()) {
-        const allGroups = valueWrapper.getAllQuestionGroupObservations && valueWrapper.getAllQuestionGroupObservations();
-        if (allGroups && allGroups.length) {
-          // Check each group in the repeatable question group
-          allGroups.forEach(group => {
-            const groupObservations = group.getValue && group.getValue();
-            if (processQuestionGroupObservations(groupObservations, oldValue, newValue)) {
-              updated = true;
-            }
-          });
-        }
-      }
-      // Handle regular QuestionGroup
-      else {
-        const groupObservations = valueWrapper.getValue && valueWrapper.getValue();
-        if (processQuestionGroupObservations(groupObservations, oldValue, newValue)) {
-          updated = true;
-        }
-      }
-    });
-    return updated;
-  }
-
   // Helper method to update media value in an observation
   _updateMediaValueInObservation(observation, oldValue, newValue) {
     if (!observation) return false;
@@ -482,7 +396,13 @@ class ObservationsHolder {
 
     // Handle ImageV2 type
     if (observation.concept.datatype === _Concept.default.dataType.ImageV2) {
-      const mediaObjects = JSON.parse(valueWrapper.getValue());
+      let mediaObjects;
+      try {
+        mediaObjects = JSON.parse(valueWrapper.getValue());
+      } catch (e) {
+        // A malformed value can't reference the file; one bad obs must not crash the whole scan.
+        return false;
+      }
       let updated = false;
       const newAnswers = _lodash.default.map(mediaObjects, mediaObject => {
         if (mediaObject.uri === oldValue) {
@@ -492,7 +412,7 @@ class ObservationsHolder {
         return mediaObject;
       });
       if (updated) {
-        observation.valueJSON = new _PrimitiveValue.default(JSON.stringify(newAnswers), _Concept.default.dataType.ImageV2);
+        observation.valueJSON = new _PrimitiveValue.default(JSON.stringify(newAnswers), _Concept.default.dataType.ImageV2, valueWrapper.answerSource);
         return true;
       }
     }
@@ -501,15 +421,19 @@ class ObservationsHolder {
       const answers = valueWrapper.getValue();
       const oldValueIndex = _lodash.default.indexOf(answers, oldValue);
       if (oldValueIndex >= 0) {
+        // Duplicate occurrences are not meaningful for media values — collapse them
+        // into the single replaced entry.
         const newAnswers = _lodash.default.reject(answers, answer => answer === oldValue);
         newAnswers.splice(oldValueIndex, 0, newValue);
-        observation.valueJSON = new _MultipleCodedValues.default(newAnswers);
+        observation.valueJSON = new _MultipleCodedValues.default(newAnswers, valueWrapper.answerSource);
         return true;
       }
     }
     // Handle single coded value
     else if (valueWrapper.getValue() === oldValue) {
-      observation.valueJSON = new _SingleCodedValue.default(newValue);
+      const newWrapper = observation.concept.getValueWrapperFor(newValue);
+      newWrapper.answerSource = valueWrapper.answerSource;
+      observation.valueJSON = newWrapper;
       return true;
     }
     return false;
@@ -524,7 +448,7 @@ class ObservationsHolder {
       const groupObservations = group && group.getValue && group.getValue();
       if (!Array.isArray(groupObservations)) return;
       _lodash.default.forEach(groupObservations, childObs => {
-        if (_Concept.default.dataType.Media.includes(childObs.concept.datatype) && this._updateMediaValueInObservation(childObs, oldValue, newValue)) {
+        if (childObs.concept.isMediaConcept() && this._updateMediaValueInObservation(childObs, oldValue, newValue)) {
           updated = true;
         }
       });
@@ -540,7 +464,7 @@ class ObservationsHolder {
   replaceMediaObservation(oldValue, newValue, conceptUUID) {
     let updated = false;
     _lodash.default.forEach(this.observations, obs => {
-      if (_Concept.default.dataType.Media.includes(obs.concept.datatype)) {
+      if (obs.concept.isMediaConcept()) {
         if (this._updateMediaValueInObservation(obs, oldValue, newValue)) updated = true;
       } else if (obs.concept.datatype === _Concept.default.dataType.QuestionGroup) {
         const wrapper = obs.getValueWrapper && obs.getValueWrapper();

package/dist/ProgramEnrolment.js

@@ -460,10 +460,6 @@ class ProgramEnrolment extends _BaseEntity.default {
     const inExitObservations = new _ObservationsHolder.default(this.programExitObservations).replaceMediaObservation(originalValue, newValue, conceptUUID);
     return inObservations || inExitObservations;
   }
-  replaceObservation(originalValue, newValue) {
-    new _ObservationsHolder.default(this.observations).updateObservationBasedOnValue(originalValue, newValue);
-    new _ObservationsHolder.default(this.programExitObservations).updateObservationBasedOnValue(originalValue, newValue);
-  }
   getName() {
     return "ProgramEnrolment";
   }

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "openchs-models",
     "description": "OpenCHS data model to be used by front end clients",
-    "version": "1.33.66",
+    "version": "1.33.68",
     "private": false,
     "repository": {
         "type": "git",