snoopdave commented on code in PR #161:
URL: https://github.com/apache/roller/pull/161#discussion_r4238328854
##########
app/src/main/java/org/apache/roller/weblogger/webservices/atomprotocol/MediaCollection.java:
##########
@@ -420,15 +406,13 @@ public void putMedia(AtomRequest areq) throws
AtomException {
}
}
throw new AtomException("Incorrect path information");
-
+
} catch (WebloggerException re) {
throw new AtomException("Posting media");
- } catch (IOException ioe) {
- throw new AtomException("Posting media", ioe);
}
}
-
-
+
+
public void deleteEntry(AtomRequest areq) throws AtomException {
try {
String[] pathInfo = StringUtils.split(areq.getPathInfo(), "/");
Review Comment:
Fixed. The lookup now uses the path without .media-link, and a missing file
returns 404 instead of an NPE. Delete tests added.
--
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]
