commit f3973047c40b581cc6132859530005a2fb7ec951
author Ray Milkey <ray@onlab.us> Wed Jun 22 13:34:17 2016 -0700
committer Ray Milkey <ray@onlab.us> Wed Jun 22 15:37:58 2016 -0700
tree d7a35640e8315ba6026062283e6063bd35f381ba
parent 7fae18929c62d68ffbf5d968c40f0b06d3df7dfd

Fix for ONOS-4740

Properly determine if the JSON payload contained any errors.

Change-Id: Id8e816f6627580dcaca3ccb6b7356d61be62392d

web/api/src/main/java/org/onosproject/rest/resources/NetworkConfigWebResource.java

diff --git a/web/api/src/main/java/org/onosproject/rest/resources/NetworkConfigWebResource.java b/web/api/src/main/java/org/onosproject/rest/resources/NetworkConfigWebResource.java
index b8dbb09..3afeb3a 100644
--- a/web/api/src/main/java/org/onosproject/rest/resources/NetworkConfigWebResource.java
+++ b/web/api/src/main/java/org/onosproject/rest/resources/NetworkConfigWebResource.java
@@ -204,7 +204,7 @@
                     errorMsgs.addAll(consumeJson(service, (ObjectNode) root.path(sk),
                                                  service.getSubjectFactory(sk)));
                 });
-        if (errorMsgs.toString().length() > 0) {
+        if (errorMsgs.size() > 0) {
             return Response.status(MULTI_STATUS_RESPONE).entity(produceErrorJson(errorMsgs)).build();
         }
         return Response.ok().build();
@@ -227,7 +227,7 @@
         NetworkConfigService service = get(NetworkConfigService.class);
         ObjectNode root = (ObjectNode) mapper().readTree(request);
         List<String> errorMsgs = consumeJson(service, root, service.getSubjectFactory(subjectClassKey));
-        if (errorMsgs.toString().length() > 0) {
+        if (errorMsgs.size() > 0) {
             return Response.status(MULTI_STATUS_RESPONE).entity(produceErrorJson(errorMsgs)).build();
         }
         return Response.ok().build();