Changeset 9919 for extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/dao/impl/TagDaoImpl.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/dao/impl

Files:

• . (added)
• TagDaoImpl.java (copied) (copied from extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/dao/TagDao.java) (5 diffs)

extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/dao/impl/TagDaoImpl.java

@@ -1 @@
-package fr.mael.jiwigo.dao;
+package fr.mael.jiwigo.dao.impl;
 
 import java.io.IOException;
@@ -5 +5 @@
 import java.util.List;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
@@ -10 +12 @@
 import org.w3c.dom.NodeList;
 
+import fr.mael.jiwigo.dao.TagDao;
 import fr.mael.jiwigo.om.Image;
 import fr.mael.jiwigo.om.Tag;
@@ -35 +38 @@
  *
  */
-public class TagDao {
+public class TagDaoImpl implements TagDao {
     /**
      * Logger
      */
-    public static final org.apache.commons.logging.Log LOG = org.apache.commons.logging.LogFactory.getLog(TagDao.class);
-    /**
-     * Instance, to use a singleton
-     */
-    private static TagDao instance;
+    private final Logger LOG = LoggerFactory.getLogger(ImageDaoImpl.class);
 
     private SessionManager sessionManager;
-
-    /**
-     * private constructor to use a singleton
-     */
-    private TagDao(SessionManager sessionManager) {
-        this.sessionManager = sessionManager;
-    }
-
-    /**
-     * @return the instance
-     */
-    public static TagDao getInstance(SessionManager sessionManager) {
-        if (instance == null) {
-            instance = new TagDao(sessionManager);
-        }
-        return instance;
-    }
 
     /**
@@ -135 +117 @@
     }
 
+    public SessionManager getSessionManager() {
+        return sessionManager;
+    }
+
+    public void setSessionManager(SessionManager sessionManager) {
+        this.sessionManager = sessionManager;
+    }
+
 }