kwin commented on code in PR #2081:
URL: https://github.com/apache/jackrabbit-oak/pull/2081#discussion_r1965197570
##########
oak-core/src/main/java/org/apache/jackrabbit/oak/namepath/impl/LocalNameMapper.java:
##########
@@ -91,9 +91,10 @@ public synchronized String getJcrName(@NotNull String
oakName) {
}
}
}
+ } else {
+ throw new IllegalStateException("No namespace mapping found
for " + oakName);
Review Comment:
I wouldn't understand why `Session.addNode("foo:bar").getName()` wouldn't
end up here, because
a) uri is null (there is no namespace URI registered for "foo" neither
globally nor locally
b) there is no local namespace mapping for "foo"
This previously didn't lead to an exception. Now it does.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
