Apply suggestions from code review

winterhazel · sureshanaparti · web-flow · commit 414cc6ec0bd0 · 2024-11-29T10:44:50.000-03:00
Co-authored-by: Suresh Kumar Anaparti &lt;sureshkumar.anaparti@gmail.com&gt;
diff --git a/engine/schema/src/main/java/com/cloud/projects/dao/ProjectAccountDaoImpl.java b/engine/schema/src/main/java/com/cloud/projects/dao/ProjectAccountDaoImpl.java
@@ -200,7 +200,9 @@ public void removeUserFromProjects(long userId) {
         sc.setParameters("userId", userId);
 
         int removedCount = remove(sc);
-        s_logger.debug(String.format("Removed user [%s] from %s project(s).", userId, count));
+        if (removedCount > 0) {
+            s_logger.debug(String.format("Removed user [%s] from %s project(s).", userId, removedCount));
+        }
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -200,7 +200,9 @@ public void removeUserFromProjects(long userId) {`
`200`	`200`	`sc.setParameters("userId", userId);`
`201`	`201`
`202`	`202`	`int removedCount = remove(sc);`
`203`		`- s_logger.debug(String.format("Removed user [%s] from %s project(s).", userId, count));`
	`203`	`+ if (removedCount > 0) {`
	`204`	`+ s_logger.debug(String.format("Removed user [%s] from %s project(s).", userId, removedCount));`
	`205`	`+ }`
`204`	`206`	`}`
`205`	`207`
`206`	`208`	`@Override`