Issue 1164: Don't allow negative delta amounts when checking in samples (#2022)

Author: labkey-susanh

packages/components/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@labkey/components",
-  "version": "7.45.0",
+  "version": "7.45.1",
   "description": "Components, models, actions, and utility functions for LabKey applications and pages",
   "sideEffects": false,
   "files": [

packages/components/package.json

@@ -1,6 +1,10 @@
 # @labkey/components
 Components, models, actions, and utility functions for LabKey applications and pages
 
+### version 7.45.1
+*Released*: 25 June 2026
+- GH Issue #1164: Don't allow negative delta value during checkin
+
 ### version 7.45.0
 *Released*: 25 June 2026
 - GitHub Issue 1234: User management page update to use SiteUsers for inactive users grid

packages/components/releaseNotes/components.md

@@ -53,10 +53,14 @@ describe('UnitModel', () => {
         expect(new UnitModel(null, null).isValidForSubmit()).toBeTruthy();
         expect(new UnitModel(null, 'bad').isValidForSubmit()).toBeTruthy();
         expect(new UnitModel(null, 'mL').isValidForSubmit()).toBeFalsy();
+        expect(new UnitModel('no', 'mL').isValidForSubmit()).toBeFalsy();
+        expect(new UnitModel(-1, 'mL').isValidForSubmit()).toBeFalsy();
         expect(new UnitModel(0, null).isValidForSubmit()).toBeFalsy();
         expect(new UnitModel(0, null).isValidForSubmit()).toBeFalsy();
         expect(new UnitModel(0, 'bad').isValidForSubmit()).toBeFalsy();
+        expect(new UnitModel(-10.1, 'bad').isValidForSubmit()).toBeFalsy();
         expect(new UnitModel(0, 'mL').isValidForSubmit()).toBeTruthy();
+        expect(new UnitModel(-0, 'mL').isValidForSubmit()).toBeTruthy();
         expect(new UnitModel(1, 'uL').isValidForSubmit()).toBeTruthy();
     });
 });

packages/components/src/internal/util/measurement.test.ts

@@ -86,7 +86,7 @@ export class UnitModel {
     }
 
     isValidForSubmit(): boolean {
-        const hasBoth = this.value != undefined && this.unit != null;
+        const hasBoth = this.value != undefined && !isNaN(this.value) && this.value >= 0 && this.unit != null;
         const hasNeither = this.value == undefined && this.unit == null;
         return hasBoth || hasNeither;
     }