This is an automated email from the ASF dual-hosted git repository.

acosentino pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/master by this push:
     new 5a25098  camel-box: fix a possible NPE when dereferencing BoxUser 
information (#4144)
5a25098 is described below

commit 5a25098e3b83c5343a45abc37e3d4992efe83cff
Author: Otavio Rodolfo Piske <[email protected]>
AuthorDate: Sun Aug 30 10:23:34 2020 +0200

    camel-box: fix a possible NPE when dereferencing BoxUser information (#4144)
---
 .../main/java/org/apache/camel/component/box/api/BoxTasksManager.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/components/camel-box/camel-box-api/src/main/java/org/apache/camel/component/box/api/BoxTasksManager.java
 
b/components/camel-box/camel-box-api/src/main/java/org/apache/camel/component/box/api/BoxTasksManager.java
index 59ecf10..1140188 100644
--- 
a/components/camel-box/camel-box-api/src/main/java/org/apache/camel/component/box/api/BoxTasksManager.java
+++ 
b/components/camel-box/camel-box-api/src/main/java/org/apache/camel/component/box/api/BoxTasksManager.java
@@ -210,13 +210,13 @@ public class BoxTasksManager {
                                // == null)' clause is dead code.
     public BoxTask addAssignmentToTask(String taskId, BoxUser assignTo) {
         try {
-            LOG.debug("Assigning task(id=" + taskId + ") to user(id=" + 
assignTo.getID() + ")");
             if (taskId == null) {
                 throw new IllegalArgumentException("Parameter 'commentId' can 
not be null");
             }
             if (assignTo == null) {
                 throw new IllegalArgumentException("Parameter 'assignTo' can 
not be null");
             }
+            LOG.debug("Assigning task(id=" + taskId + ") to user(id=" + 
assignTo.getID() + ")");
 
             BoxTask task = new BoxTask(boxConnection, taskId);
             task.addAssignment(assignTo);

Reply via email to