henrykuijpers commented on code in PR #61:
URL:
https://github.com/apache/sling-org-apache-sling-api/pull/61#discussion_r2159155924
##########
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:
I feel we should fix this in SlingPostServlet and in the appropriate UI
where this goes wrong, now we're adding this override in
`PersistenceException`, but what if another exception bubbles up somewhere, do
we also need to fix the `getMessage`-method there then?
--
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
