From 7ca599cb444c3896beb9318c90a460adb1f276b9 Mon Sep 17 00:00:00 2001 From: SwapnilPathak Date: Fri, 15 Sep 2017 11:24:06 +0530 Subject: Sonar Major Move the string literal on the left side of this string comparison. Change-Id: I8086a6fc1dc24719c3ffd3785a12c086ab453ae7 Issue-ID: POLICY-230 Signed-off-by: SwapnilPathak --- .../org/onap/policy/drools/event/comm/bus/internal/BusConsumer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'policy-endpoints') diff --git a/policy-endpoints/src/main/java/org/onap/policy/drools/event/comm/bus/internal/BusConsumer.java b/policy-endpoints/src/main/java/org/onap/policy/drools/event/comm/bus/internal/BusConsumer.java index c4155b77..8171c35d 100644 --- a/policy-endpoints/src/main/java/org/onap/policy/drools/event/comm/bus/internal/BusConsumer.java +++ b/policy-endpoints/src/main/java/org/onap/policy/drools/event/comm/bus/internal/BusConsumer.java @@ -310,7 +310,7 @@ public interface BusConsumer { consumerInstance, fetchTimeout, fetchLimit, useHttps); // super constructor sets servers = {""} if empty to avoid errors when using DME2 - if ((servers.size() == 1 && servers.get(0).equals("")) || (servers == null) + if ((servers.size() == 1 && ("".equals(servers.get(0)))) || (servers == null) || (servers.isEmpty())) { throw new IllegalArgumentException("Must provide at least one host for HTTP AAF"); } -- cgit 1.2.3-korg