kwin commented on code in PR #61: URL: https://github.com/apache/sling-org-apache-sling-api/pull/61#discussion_r2160428799
########## src/main/java/org/apache/sling/api/resource/PersistenceException.java: ########## @@ -91,4 +95,34 @@ public String getResourcePath() { public String getPropertyName() { return this.propertyName; } + + /** + * Returns a message for this exception. + * If the cause is not null, it will be appended to the message. + * @return The message for this exception + */ + @Override + public String getMessage() { Review Comment: One example where the root cause is helpful for less technical users is a PersistenceException for ResourceResolver.create(). The most common causes (at least with the JCR provider) are: a) Resource with that name does already exist b) User does not have permission to create a resource there or c) Resource with the given type is not allowed there I think this is crucial to transmit even to end users because at least a) can be solved by the user himself without involving IT. -- 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: dev-unsubscr...@sling.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org