You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@johnzon.apache.org by rm...@apache.org on 2020/03/23 14:03:03 UTC

[johnzon] branch master updated: JOHNZON-306 more details on which field/method failed to bind

This is an automated email from the ASF dual-hosted git repository.

rmannibucau pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/johnzon.git


The following commit(s) were added to refs/heads/master by this push:
     new 695a887  JOHNZON-306 more details on which field/method failed to bind
695a887 is described below

commit 695a887a43f70b600cf92199c711e62eeb1e4569
Author: Romain Manni-Bucau <rm...@gmail.com>
AuthorDate: Mon Mar 23 15:02:58 2020 +0100

    JOHNZON-306 more details on which field/method failed to bind
---
 .../src/main/java/org/apache/johnzon/mapper/MapperException.java    | 4 ++++
 .../main/java/org/apache/johnzon/mapper/access/FieldAccessMode.java | 4 ++--
 .../java/org/apache/johnzon/mapper/access/MethodAccessMode.java     | 6 +++---
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/MapperException.java b/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/MapperException.java
index 6b5f00a..db7e385 100644
--- a/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/MapperException.java
+++ b/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/MapperException.java
@@ -28,4 +28,8 @@ public class MapperException extends JsonException {
     public MapperException(final String s) {
         super(s);
     }
+
+    public MapperException(final String s, final Throwable cause) {
+        super(s, cause);
+    }
 }
diff --git a/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/FieldAccessMode.java b/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/FieldAccessMode.java
index 42bc13f..d87cd1b 100644
--- a/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/FieldAccessMode.java
+++ b/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/FieldAccessMode.java
@@ -154,7 +154,7 @@ public class FieldAccessMode extends BaseAccessMode {
             try {
                 field.set(instance, value);
             } catch (final Exception e) {
-                throw new MapperException(e);
+                throw new MapperException("Error setting " + field, e);
             }
         }
 
@@ -174,7 +174,7 @@ public class FieldAccessMode extends BaseAccessMode {
             try {
                 return field.get(instance);
             } catch (final Exception e) {
-                throw new MapperException(e);
+                throw new MapperException("Error setting " + field, e);
             }
         }
 
diff --git a/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/MethodAccessMode.java b/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/MethodAccessMode.java
index 0b7a1ae..79e07ac 100644
--- a/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/MethodAccessMode.java
+++ b/johnzon-mapper/src/main/java/org/apache/johnzon/mapper/access/MethodAccessMode.java
@@ -184,7 +184,7 @@ public class MethodAccessMode extends BaseAccessMode {
             try {
                 method.invoke(instance, value);
             } catch (final Exception e) {
-                throw new MapperException(e);
+                throw new MapperException("Error calling " + method, e);
             }
         }
 
@@ -204,7 +204,7 @@ public class MethodAccessMode extends BaseAccessMode {
             try {
                 return method.invoke(instance);
             } catch (final Exception e) {
-                throw new MapperException(e);
+                throw new MapperException("Error calling " + method, e);
             }
         }
 
@@ -228,7 +228,7 @@ public class MethodAccessMode extends BaseAccessMode {
                         collection.addAll(Collection.class.cast(value));
                     }
                 } catch (final Exception e) {
-                    throw new MapperException(e);
+                    throw new MapperException("Error calling " + method, e);
                 }
             }
         }