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

Timestamp:

Mar 29, 2011, 8:21:25 PM (13 years ago)

Author:

mlg

Message:

Complete rewriting of the api
The api now uses interfaces everywhere (for the dao, services and session manager). This allows the developer to use his own implementations if he wants.
Deletion of the singletons. There are now getters and setters. It allows to make dependency injection.
Use of sl4j in the api, instead of using log4j.

Location:

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

Files:

• . (added)
• TagServiceImpl.java (copied) (copied from extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/service/TagService.java) (5 diffs)

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

@@ -1 @@
-package fr.mael.jiwigo.service;
+package fr.mael.jiwigo.service.impl;
 
 import java.io.IOException;
 import java.util.List;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import fr.mael.jiwigo.dao.TagDao;
 import fr.mael.jiwigo.om.Image;
 import fr.mael.jiwigo.om.Tag;
+import fr.mael.jiwigo.service.TagService;
 import fr.mael.jiwigo.transverse.exception.ProxyAuthenticationException;
-import fr.mael.jiwigo.transverse.session.SessionManager;
 
 /*
@@ -27 +30 @@
  *
  */
-public class TagService {
+public class TagServiceImpl implements TagService {
     /**
      * Logger
      */
-    public static final org.apache.commons.logging.Log LOG = org.apache.commons.logging.LogFactory
-            .getLog(TagService.class);
+    private final Logger LOG = LoggerFactory.getLogger(TagServiceImpl.class);
 
-    /**
-     * The instance, to use a singleton
-     */
-    private static TagService instance;
-
-    private SessionManager sessionManager;
-
-    /**
-     * @return the singleton
-     */
-    public static TagService getInstance(SessionManager sessionManager) {
-        if (instance == null) {
-            instance = new TagService(sessionManager);
-        }
-        return instance;
-    }
-
-    /**
-     * private constructor
-     */
-    private TagService(SessionManager sessionManager) {
-        this.sessionManager = sessionManager;
-    }
+    private TagDao dao;
 
     /**
@@ -65 +45 @@
      */
     public List<Tag> list() throws IOException, ProxyAuthenticationException {
-        return TagDao.getInstance(sessionManager).list();
+        return dao.list();
     }
 
@@ -78 +58 @@
         Tag tag = new Tag();
         tag.setName(nom);
-        return TagDao.getInstance(sessionManager).create(tag);
+        return dao.create(tag);
     }
 
@@ -89 +69 @@
      */
     public List<Tag> tagsForImage(Image image) throws IOException, ProxyAuthenticationException {
-        return TagDao.getInstance(sessionManager).tagsForImage(image);
+        return dao.tagsForImage(image);
+    }
+
+    public TagDao getDao() {
+        return dao;
+    }
+
+    public void setDao(TagDao dao) {
+        this.dao = dao;
     }