Sanitize data from manual careportal entries so only fields with actu…

…al data are sent to the server (nightscout#5619)
LukyCZ · May 5, 2020 · 9236771 · 9236771
1 parent 7c5a69d
commit 9236771
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/lib/client/careportal.js b/lib/client/careportal.js
@@ -226,11 +226,17 @@ function init (client, $) {
       , duration: times.msecs(parse_duration($('#duration').val())).mins < 1 ? $('#duration').val() : times.msecs(parse_duration($('#duration').val())).mins
       , percent: $('#percent').val()
       , profile: $('#profile').val()
-      , preBolus: parseInt($('#preBolus').val())
+      , preBolus: $('#preBolus').val()
       , notes: $('#notes').val()
       , units: client.settings.units
     };
 
+    data.preBolus = parseInt(data.preBolus);
+
+    if (isNaN(data.preBolus)) {
+      delete data.preBolus;
+    }
+
     var reasons = inputMatrix[eventType]['reasons'];
     var reason = _.find(reasons, function matches (r) {
       return r.name === selectedReason;
@@ -273,7 +279,11 @@ function init (client, $) {
       data.splitExt = parseInt($('#insulinSplitExt').val()) || 0;
     }
 
-    return data;
+    let d = {};
+    Object.keys(data).forEach(function(key) {
+      if (data[key] != "" && data[key] != null) d[key] = data[key];
+    });
+    return d;
   }
 
   careportal.save = function save (event) {