QRunIO/qqq
2
0
Fork 0
mirror of https://github.com/Kingsrook/qqq.git synced 2025-07-18 05:01:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

Avoid NPE if adding a log line before the script header is set

Browse Source
This commit is contained in:
darin.kelkhoff
2024-06-19 16:48:05 -05:00
parent 3d7df30db4
commit 564a5e1095
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/scripts/logging/BuildScriptLogAndScriptLogLineExecutionLogger.java
View File

@ -94,7 +94,7 @@ public class BuildScriptLogAndScriptLogLineExecutionLogger implements QCodeExecu
protected QRecord buildDetailLogRecord(String logLine) protected QRecord buildDetailLogRecord(String logLine)
{ {
return (new QRecord() return (new QRecord()
.withValue("scriptLogId", scriptLog.getValue("id")) .withValue("scriptLogId", scriptLog == null ? null : scriptLog.getValue("id"))
.withValue("timestamp", Instant.now()) .withValue("timestamp", Instant.now())
.withValue("text", truncate(logLine))); .withValue("text", truncate(logLine)));
} }
@ -145,6 +145,14 @@ public class BuildScriptLogAndScriptLogLineExecutionLogger implements QCodeExecu
{ {
this.executeCodeInput = executeCodeInput; this.executeCodeInput = executeCodeInput;
this.scriptLog = buildHeaderRecord(executeCodeInput); this.scriptLog = buildHeaderRecord(executeCodeInput);
if(scriptLogLines != null)
{
for(QRecord scriptLogLine : scriptLogLines)
{
scriptLogLine.setValue("scriptLogId", scriptLog.getValue("id"));
}
}
} }
catch(Exception e) catch(Exception e)
{ {