commit acc51e2addbc9f509d1e69621a8841393ae87b38
author Hector Oswaldo Caballero <hector.caballero@ericsson.com> Tue Jul 03 17:31:09 2018 -0400
committer Hector Oswaldo Caballero <hector.caballero@ericsson.com> Wed Jul 04 10:05:18 2018 -0400
tree 2a00accc6165607aaf31e6af8eb455e1fa875fe5
parent 5effa7f6e73dc1b926f843185b6b46aaa015ca12

Use logger built-in format

Change-Id: I593514b2f3ab75bc58e8edcec5f3981de4c4c82a

src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/DefaultAccessRights.java

diff --git a/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/DefaultAccessRights.java b/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/DefaultAccessRights.java
index 466fb55..a4b56c1 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/DefaultAccessRights.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/DefaultAccessRights.java
@@ -146,7 +146,7 @@
     try {
       RefPattern.validateRegExp(refPattern);
     } catch (InvalidNameException e) {
-      log.error("Invalid ref name: " + e.getMessage());
+      log.error("Invalid ref name: {}", e.getMessage());
       return false;
     }
     return true;

src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/ProjectCreationValidator.java

diff --git a/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/ProjectCreationValidator.java b/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/ProjectCreationValidator.java
index e422f17..278987c 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/ProjectCreationValidator.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/projectgroupstructure/ProjectCreationValidator.java
@@ -139,10 +139,9 @@
           .getBoolean(DISABLE_GRANTING_PROJECT_OWNERSHIP, false);
     } catch (NoSuchProjectException e) {
       log.error(
-          "Failed to check project config for "
-              + parentCtrl.getProject().getName()
-              + ": "
-              + e.getMessage(),
+          "Failed to check project config for {}: {}",
+          parentCtrl.getProject().getName(),
+          e.getMessage(),
           e);
       throw new ValidationException(AN_ERROR_OCCURRED_MSG);
     }
@@ -173,7 +172,7 @@
       }
       return AccountGroup.UUID.parse(groupInfo.id);
     } catch (RestApiException | OrmException | IOException e) {
-      log.error("Failed to create project " + name + ": " + e.getMessage(), e);
+      log.error("Failed to create project {}: {}", name, e.getMessage(), e);
       throw new ValidationException(AN_ERROR_OCCURRED_MSG);
     }
   }