From 1d837a835c04078ec66325e9fb9de21ac401874e Mon Sep 17 00:00:00 2001
From: Alexander Alexeev
Date: Wed, 30 Nov 2016 15:10:15 +0700
Subject: prevent string concatenation for disabled loggers

---
 .../src/main/java/com/juick/components/Notifications.java         | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'juick-notifications/src/main/java/com/juick/components/Notifications.java')

diff --git a/juick-notifications/src/main/java/com/juick/components/Notifications.java b/juick-notifications/src/main/java/com/juick/components/Notifications.java
index dac4d22e..f98fdcce 100644
--- a/juick-notifications/src/main/java/com/juick/components/Notifications.java
+++ b/juick-notifications/src/main/java/com/juick/components/Notifications.java
@@ -133,12 +133,12 @@ public class Notifications implements AutoCloseable {
                     MulticastResult result = GCMSender.send(message, regids, 3);
                     List<Result> results = result.getResults();
                     for (int i = 0; i < results.size(); i++) {
-                        logger.info("RES " + i + ": " + results.get(i).toString());
+                        logger.info("RES {}: {}", i, results.get(i));
                     }
                 } catch (IOException ex) {
                     logger.error(ex.getMessage(), ex);
                 } catch (IllegalArgumentException err) {
-                    logger.warn("Android: Invalid API Key");
+                    logger.warn("Android: Invalid API Key", err);
                 }
             } else {
                 logger.info("GMS: no recipients");
@@ -183,7 +183,7 @@ public class Notifications implements AutoCloseable {
                             + "</toast>";
                     logger.trace(xml);
                     for (String url : urls) {
-                        logger.info("WNS: " + url);
+                        logger.info("WNS: {}", url);
                         sendWNS(wnsToken, url, xml);
                     }
                 } catch (IOException | IllegalStateException ex) {
@@ -207,7 +207,7 @@ public class Notifications implements AutoCloseable {
             if (!tokens.isEmpty()) {
                 for (String token : tokens) {
                     String payload = APNS.newPayload().alertTitle("@" + jmsg.getUser().getName()).alertBody(jmsg.getText()).build();
-                    logger.info("APNS: " + token);
+                    logger.info("APNS: {}", token);
                     apns.push(token, payload);
                 }
             } else {
-- 
cgit v1.2.3