check if consoleSession is null at the beginning of the method

Author: bernardodemarco

server/src/main/java/com/cloud/api/ApiResponseHelper.java

@@ -5619,10 +5619,6 @@ public GuiThemeResponse createGuiThemeResponse(GuiThemeJoin guiThemeJoin) {
     }
 
     private void populateDomainFieldsOnConsoleSessionResponse(ConsoleSession consoleSession, ConsoleSessionResponse consoleSessionResponse) {
-        if (consoleSession == null) {
-            return;
-        }
-
         Domain domain = ApiDBUtils.findDomainById(consoleSession.getDomainId());
         if (domain != null) {
             consoleSessionResponse.setDomain(domain.getName());
@@ -5632,10 +5628,6 @@ private void populateDomainFieldsOnConsoleSessionResponse(ConsoleSession console
     }
 
     private void populateUserFieldsOnConsoleSessionResponse(ConsoleSession consoleSession, ConsoleSessionResponse consoleSessionResponse) {
-        if (consoleSession == null) {
-            return;
-        }
-
         User user = findUserById(consoleSession.getUserId());
         if (user != null) {
             consoleSessionResponse.setUser(user.getUsername());
@@ -5644,10 +5636,6 @@ private void populateUserFieldsOnConsoleSessionResponse(ConsoleSession consoleSe
     }
 
     private void populateAccountFieldsOnConsoleSessionResponse(ConsoleSession consoleSession, ConsoleSessionResponse consoleSessionResponse) {
-        if (consoleSession == null) {
-            return;
-        }
-
         Account account = ApiDBUtils.findAccountById(consoleSession.getAccountId());
         if (account != null) {
             consoleSessionResponse.setAccount(account.getAccountName());
@@ -5656,10 +5644,6 @@ private void populateAccountFieldsOnConsoleSessionResponse(ConsoleSession consol
     }
 
     private void populateHostFieldsOnConsoleSessionResponse(ConsoleSession consoleSession, ConsoleSessionResponse consoleSessionResponse) {
-        if (consoleSession == null) {
-            return;
-        }
-
         Host host = findHostById(consoleSession.getHostId());
         if (host != null) {
             consoleSessionResponse.setHostId(host.getUuid());
@@ -5668,10 +5652,6 @@ private void populateHostFieldsOnConsoleSessionResponse(ConsoleSession consoleSe
     }
 
     private void populateInstanceFieldsOnConsoleSessionResponse(ConsoleSession consoleSession, ConsoleSessionResponse consoleSessionResponse) {
-        if (consoleSession == null) {
-            return;
-        }
-
         VMInstanceVO instance = ApiDBUtils.findVMInstanceById(consoleSession.getInstanceId());
         if (instance != null) {
             consoleSessionResponse.setInstanceId(instance.getUuid());
@@ -5682,6 +5662,10 @@ private void populateInstanceFieldsOnConsoleSessionResponse(ConsoleSession conso
     @Override
     public ConsoleSessionResponse createConsoleSessionResponse(ConsoleSession consoleSession, ResponseView responseView) {
         ConsoleSessionResponse consoleSessionResponse = new ConsoleSessionResponse();
+        if (consoleSession == null) {
+            return consoleSessionResponse;
+        }
+
         consoleSessionResponse.setId(consoleSession.getUuid());
         consoleSessionResponse.setCreated(consoleSession.getCreated());
         consoleSessionResponse.setAcquired(consoleSession.getAcquired());