aboutsummaryrefslogtreecommitdiffstats
path: root/document
diff options
context:
space:
mode:
authorjonmv <venstad@gmail.com>2024-01-23 12:45:09 +0100
committerjonmv <venstad@gmail.com>2024-01-23 12:45:09 +0100
commitafa7f09051082266441e3150ef9dea84be725cdc (patch)
tree10c5be0a365f847937a4c782c07b806690069ea6 /document
parentbd941067f8c12cc604c39f68b9eb75f06a5b30e6 (diff)
Improve readability
Diffstat (limited to 'document')
-rw-r--r--document/src/main/java/com/yahoo/document/json/readers/MapReader.java8
1 files changed, 5 insertions, 3 deletions
diff --git a/document/src/main/java/com/yahoo/document/json/readers/MapReader.java b/document/src/main/java/com/yahoo/document/json/readers/MapReader.java
index c81d296db39..b45a0001fd1 100644
--- a/document/src/main/java/com/yahoo/document/json/readers/MapReader.java
+++ b/document/src/main/java/com/yahoo/document/json/readers/MapReader.java
@@ -97,8 +97,8 @@ public class MapReader {
FieldValue key = null;
ValueUpdate update;
- boolean elementFirst = UPDATE_ELEMENT.equals(buffer.currentName());
- if (elementFirst) {
+
+ if (UPDATE_ELEMENT.equals(buffer.currentName())) {
key = keyTypeForMapUpdate(buffer.currentText(), currentLevel);
buffer.next();
}
@@ -107,7 +107,9 @@ public class MapReader {
: readSingleUpdate(buffer, valueTypeForMapUpdate(currentLevel), buffer.currentName(), ignoreUndefinedFields);
buffer.next();
- if ( ! elementFirst) {
+ if (key == null) {
+ if ( ! UPDATE_ELEMENT.equals(buffer.currentName()))
+ throw new IllegalArgumentException("Expected match element, got " + buffer.current());
key = keyTypeForMapUpdate(buffer.currentText(), currentLevel);
buffer.next();
}