Skip to content

Commit 0ee3f5c

Browse files
committed
Fix #1843
1 parent 3a8737c commit 0ee3f5c

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

release-notes/VERSION

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ Project: jackson-databind
1313
(reported byb henryptung@github)
1414
#1807: Jackson-databind caches plain map deserializer and use it even map has `@JsonDeserializer`
1515
(reported by lexas2509@github)
16+
#1843: Include name of unsettable property in exception from `SetterlessProperty.set()`
17+
(suggested by andreh7@github)
1618

1719
2.9.2 (14-Oct-2017)
1820

src/main/java/com/fasterxml/jackson/databind/deser/impl/ObjectIdReferenceProperty.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public Object deserializeSetAndReturn(JsonParser p, DeserializationContext ctxt,
9393
} catch (UnresolvedForwardReference reference) {
9494
boolean usingIdentityInfo = (_objectIdInfo != null) || (_valueDeserializer.getObjectIdReader() != null);
9595
if (!usingIdentityInfo) {
96-
throw JsonMappingException.from(p, "Unresolved forward reference but no identity info.", reference);
96+
throw JsonMappingException.from(p, "Unresolved forward reference but no identity info", reference);
9797
}
9898
reference.getRoid().appendReferring(new PropertyReferring(this, reference, _type.getRawClass(), instance));
9999
return null;

src/main/java/com/fasterxml/jackson/databind/deser/impl/SetterlessProperty.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,10 +140,10 @@ public Object deserializeSetAndReturn(JsonParser p,
140140
deserializeAndSet(p, ctxt, instance);
141141
return instance;
142142
}
143-
143+
144144
@Override
145145
public final void set(Object instance, Object value) throws IOException {
146-
throw new UnsupportedOperationException("Should never call 'set' on setterless property");
146+
throw new UnsupportedOperationException("Should never call `set()` on setterless property ('"+getName()+"')");
147147
}
148148

149149
@Override

0 commit comments

Comments
 (0)