Changeset 10505 for extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/service/impl/CommentServiceImpl.java

Timestamp:

Apr 20, 2011, 5:17:02 AM (13 years ago)

Author:

anthony43

Message:

huge refactoring to introduce JiwigoException : a generic exception encapsulating low level exceptions (IOException, ProxyAuthentication, etc...)
The aim is to have, on the consumer side, just one exception to catch them all.
this refactoring may need a code review to check the changes.

File:

• extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/service/impl/CommentServiceImpl.java (modified) (3 diffs)

extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/service/impl/CommentServiceImpl.java

@@ -10 +10 @@
 import fr.mael.jiwigo.om.Comment;
 import fr.mael.jiwigo.service.CommentService;
+import fr.mael.jiwigo.transverse.exception.JiwigoException;
 import fr.mael.jiwigo.transverse.exception.ProxyAuthenticationException;
 
@@ -43 +44 @@
      * @throws ProxyAuthenticationException
      */
-    public List<Comment> list(Integer imageId) throws IOException, ProxyAuthenticationException {
+    public List<Comment> list(Integer imageId) throws JiwigoException {
         return dao.list(imageId);
     }
@@ -56 +57 @@
      * @throws ProxyAuthenticationException
      */
-    public boolean create(String content, Integer imageId, String auteur) throws IOException,
-            ProxyAuthenticationException {
+    public boolean create(String content, Integer imageId, String auteur) throws JiwigoException {
         Comment comment = new Comment();
         comment.setContent(content);