Added null check for newSkydData

vagisha · vagisha · commit 0802d89c8210 · 2025-03-18T17:51:00.000-07:00
diff --git a/src/org/labkey/targetedms/TargetedMSManager.java b/src/org/labkey/targetedms/TargetedMSManager.java
@@ -865,7 +865,7 @@ public static TargetedMSRun getRunByLsid(String lsid, Container c)
 
     public static boolean updateSkydDataId(ITargetedMSRun run, ExpData newSkydData, User user)
     {
-        if (!(run instanceof TargetedMSRun))
+        if (!(run instanceof TargetedMSRun) || newSkydData == null)
         {
             return false;
         }

Original file line number	Diff line number	Diff line change
`@@ -865,7 +865,7 @@ public static TargetedMSRun getRunByLsid(String lsid, Container c)`
`865`	`865`
`866`	`866`	`public static boolean updateSkydDataId(ITargetedMSRun run, ExpData newSkydData, User user)`
`867`	`867`	`{`
`868`		`- if (!(run instanceof TargetedMSRun))`
	`868`	`+ if (!(run instanceof TargetedMSRun) \|\| newSkydData == null)`
`869`	`869`	`{`
`870`	`870`	`return false;`
`871`	`871`	`}`