Merge pull request #432 from fmachado/master

Fixed setAccessible modif. on POJOs and changes are not reversed anymore
influxdata · Mar 22, 2018 · 77b5f37 · 77b5f37
2 parents c41b589 + 70f8c14
commit 77b5f37
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/main/java/org/influxdb/impl/InfluxDBResultMapper.java b/src/main/java/org/influxdb/impl/InfluxDBResultMapper.java
@@ -203,9 +203,10 @@ <T> void setFieldValue(final T object, final Field field, final Object value)
       return;
     }
     Class<?> fieldType = field.getType();
-    boolean oldAccessibleState = field.isAccessible();
     try {
-      field.setAccessible(true);
+      if (!field.isAccessible()) {
+        field.setAccessible(true);
+      }
       if (fieldValueModified(fieldType, field, object, value)
         || fieldValueForPrimitivesModified(fieldType, field, object, value)
         || fieldValueForPrimitiveWrappersModified(fieldType, field, object, value)) {
@@ -219,8 +220,6 @@ <T> void setFieldValue(final T object, final Field field, final Object value)
         + "The correct type is '%s' (current field value: '%s').";
       throw new InfluxDBMapperException(
         String.format(msg, object.getClass().getName(), field.getName(), value.getClass().getName(), value));
-    } finally {
-      field.setAccessible(oldAccessibleState);
     }
   }