Changeset 9919 for extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/service/impl/CategoryServiceImpl.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)
• CategoryServiceImpl.java (copied) (copied from extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/service/CategoryService.java) (6 diffs)

extensions/jiwigo-ws-api/src/main/java/fr/mael/jiwigo/service/impl/CategoryServiceImpl.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.CategoryDao;
 import fr.mael.jiwigo.om.Category;
+import fr.mael.jiwigo.service.CategoryService;
 import fr.mael.jiwigo.transverse.exception.ProxyAuthenticationException;
-import fr.mael.jiwigo.transverse.session.SessionManager;
 
 /*
@@ -25 +28 @@
  *
  */
-public class CategoryService {
+public class CategoryServiceImpl implements CategoryService {
     /**
      * Logger
      */
-    public static final org.apache.commons.logging.Log LOG = org.apache.commons.logging.LogFactory
-            .getLog(CategoryService.class);
+    private final Logger LOG = LoggerFactory.getLogger(CategoryServiceImpl.class);
 
-    /**
-     * singleton
-     */
-    private static CategoryService instance;
-
-    private SessionManager sessionManager;
-
-    /**
-     * @return the singleton
-     */
-    public static CategoryService getInstance(SessionManager sessionManager) {
-        if (instance == null) {
-            instance = new CategoryService(sessionManager);
-        }
-        return instance;
-    }
-
-    /**
-     * private constructor to use a singleton
-     */
-    private CategoryService(SessionManager sessionManager) {
-        this.sessionManager = sessionManager;
-
-    }
+    private CategoryDao dao;
 
     /**
@@ -65 +44 @@
      */
     public List<Category> list(boolean recursive) throws IOException, ProxyAuthenticationException {
-        return CategoryDao.getInstance(sessionManager).list(recursive);
+        return dao.list(recursive);
     }
 
@@ -75 +54 @@
      */
     public List<Category> makeTree() throws IOException, ProxyAuthenticationException {
-        List<Category> list = CategoryDao.getInstance(sessionManager).list(true);
+        List<Category> list = dao.list(true);
         for (Category category : list) {
             for (Category category2 : list) {
@@ -103 +82 @@
         category.setDirectParent(parent);
         category.setName(nom);
-        return CategoryDao.getInstance(sessionManager).create(category);
+        return dao.create(category);
     }
 
@@ -115 +94 @@
         Category category = new Category();
         category.setName(nom);
-        return CategoryDao.getInstance(sessionManager).create(category);
+        return dao.create(category);
     }
+
+    public CategoryDao getDao() {
+        return dao;
+    }
+
+    public void setDao(CategoryDao dao) {
+        this.dao = dao;
+    }
+
 }