Author: ludwig
Date: Thu Feb 3 19:44:37 2011
New Revision: 1066925
URL: http://svn.apache.org/viewvc?rev=1066925&view=rev
Log:
Java 5 for services Session, UI and UniqueId
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionListener.java
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionService.java
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionTool.java
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSession.java
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSessionService.java
turbine/core/trunk/src/java/org/apache/turbine/services/ui/TurbineUIService.java
turbine/core/trunk/src/java/org/apache/turbine/services/uniqueid/TurbineUniqueIdService.java
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionListener.java
URL:
http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionListener.java?rev=1066925&r1=1066924&r2=1066925&view=diff
==============================================================================
---
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionListener.java
(original)
+++
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionListener.java
Thu Feb 3 19:44:37 2011
@@ -61,6 +61,11 @@ public class SessionListener
// ---- HttpSessionListener implementation -----------------------------
/**
+ * Serial version.
+ */
+ private static final long serialVersionUID = -8083730704842809870L;
+
+ /**
* Called by the servlet container when a new session is created
*
* @param event Session creation event.
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionService.java
URL:
http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionService.java?rev=1066925&r1=1066924&r2=1066925&view=diff
==============================================================================
---
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionService.java
(original)
+++
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionService.java
Thu Feb 3 19:44:37 2011
@@ -50,7 +50,7 @@ public interface SessionService extends
*
* @return Collection of HttpSession objects
*/
- Collection getActiveSessions();
+ Collection<HttpSession> getActiveSessions();
/**
* Adds a session to the current list. This method should only be
@@ -88,7 +88,7 @@ public interface SessionService extends
*
* @return collection of org.apache.turbine.om.security.User objects
*/
- Collection getActiveUsers();
+ Collection<User> getActiveUsers();
/**
* Gets the User object of the the specified HttpSession.
@@ -113,6 +113,6 @@ public interface SessionService extends
* @param user the user
* @return Collection of HtttSession objects
*/
- Collection getSessionsForUser(User user);
+ Collection<HttpSession> getSessionsForUser(User user);
}
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionTool.java
URL:
http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionTool.java?rev=1066925&r1=1066924&r2=1066925&view=diff
==============================================================================
---
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionTool.java
(original)
+++
turbine/core/trunk/src/java/org/apache/turbine/services/session/SessionTool.java
Thu Feb 3 19:44:37 2011
@@ -22,6 +22,7 @@ package org.apache.turbine.services.sess
import java.util.Collection;
+
import javax.servlet.http.HttpSession;
import org.apache.turbine.om.security.User;
@@ -49,7 +50,7 @@ public class SessionTool
*
* @return List of HttpSession objects
*/
- public Collection getActiveSessions()
+ public Collection<HttpSession> getActiveSessions()
{
return TurbineSession.getActiveSessions();
}
@@ -99,7 +100,7 @@ public class SessionTool
*
* @return collection of org.apache.turbine.om.security.User objects
*/
- public Collection getActiveUsers()
+ public Collection<User> getActiveUsers()
{
return TurbineSession.getActiveUsers();
}
@@ -122,7 +123,7 @@ public class SessionTool
* @param user the user
* @return Collection of HtttSession objects
*/
- public Collection getSessionsForUser(User user)
+ public Collection<HttpSession> getSessionsForUser(User user)
{
return TurbineSession.getSessionsForUser(user);
}
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSession.java
URL:
http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSession.java?rev=1066925&r1=1066924&r2=1066925&view=diff
==============================================================================
---
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSession.java
(original)
+++
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSession.java
Thu Feb 3 19:44:37 2011
@@ -43,7 +43,7 @@ public abstract class TurbineSession
*
* @return List of HttpSession objects
*/
- public static Collection getActiveSessions()
+ public static Collection<HttpSession> getActiveSessions()
{
return getService().getActiveSessions();
}
@@ -93,7 +93,7 @@ public abstract class TurbineSession
*
* @return collection of org.apache.turbine.om.security.User objects
*/
- public static Collection getActiveUsers()
+ public static Collection<User> getActiveUsers()
{
return getService().getActiveUsers();
}
@@ -139,7 +139,7 @@ public abstract class TurbineSession
* @param user the user
* @return Collection of HtttSession objects
*/
- public static Collection getSessionsForUser(User user)
+ public static Collection<HttpSession> getSessionsForUser(User user)
{
return getService().getSessionsForUser(user);
}
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSessionService.java
URL:
http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSessionService.java?rev=1066925&r1=1066924&r2=1066925&view=diff
==============================================================================
---
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSessionService.java
(original)
+++
turbine/core/trunk/src/java/org/apache/turbine/services/session/TurbineSessionService.java
Thu Feb 3 19:44:37 2011
@@ -58,20 +58,20 @@ public class TurbineSessionService
implements SessionService
{
/** Map of active sessions */
- private Map activeSessions;
+ private Map<String, HttpSession> activeSessions;
/**
* Gets a list of the active sessions.
*
* @return A copy of the list of <code>HttpSession</code> objects.
*/
- public Collection getActiveSessions()
+ public Collection<HttpSession> getActiveSessions()
{
// Sync externally to allow ArrayList's ctor to iterate
// activeSessions' values in a thread-safe fashion.
synchronized (activeSessions)
{
- return new ArrayList(activeSessions.values());
+ return new ArrayList<HttpSession>(activeSessions.values());
}
}
@@ -120,17 +120,17 @@ public class TurbineSessionService
*
* @return A set of {@link org.apache.turbine.om.security.User} objects.
*/
- public Collection getActiveUsers()
+ public Collection<User> getActiveUsers()
{
- Collection users;
+ Collection<User> users;
synchronized (activeSessions)
{
// Pre-allocate a list which won't need expansion more
// than once.
- users = new ArrayList((int) (activeSessions.size() * 0.7));
- for (Iterator i = activeSessions.values().iterator(); i.hasNext();)
+ users = new ArrayList<User>((int) (activeSessions.size() * 0.7));
+ for (Iterator<HttpSession> i = activeSessions.values().iterator();
i.hasNext();)
{
- User u = getUserFromSession((HttpSession) i.next());
+ User u = getUserFromSession(i.next());
if (u != null && u.hasLoggedIn())
{
users.add(u);
@@ -170,7 +170,7 @@ public class TurbineSessionService
*/
public HttpSession getSession(String sessionId)
{
- return (HttpSession) this.activeSessions.get(sessionId);
+ return this.activeSessions.get(sessionId);
}
/**
@@ -180,14 +180,14 @@ public class TurbineSessionService
* @param user the user
* @return Collection of HtttSession objects
*/
- public Collection getSessionsForUser(User user)
+ public Collection<HttpSession> getSessionsForUser(User user)
{
- Collection sessions = new ArrayList();
+ Collection<HttpSession> sessions = new ArrayList<HttpSession>();
synchronized (activeSessions)
{
- for (Iterator i = activeSessions.values().iterator(); i.hasNext();)
+ for (Iterator<HttpSession> i = activeSessions.values().iterator();
i.hasNext();)
{
- HttpSession session = (HttpSession) i.next();
+ HttpSession session = i.next();
User u = this.getUserFromSession(session);
if (user.equals(u))
{
@@ -205,9 +205,10 @@ public class TurbineSessionService
/**
* Initializes the service
*/
+ @Override
public void init()
{
- this.activeSessions = new Hashtable();
+ this.activeSessions = new Hashtable<String, HttpSession>();
setInit(true);
}
@@ -215,6 +216,7 @@ public class TurbineSessionService
/**
* Returns to uninitialized state.
*/
+ @Override
public void shutdown()
{
this.activeSessions = null;
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/ui/TurbineUIService.java
URL:
http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/ui/TurbineUIService.java?rev=1066925&r1=1066924&r2=1066925&view=diff
==============================================================================
---
turbine/core/trunk/src/java/org/apache/turbine/services/ui/TurbineUIService.java
(original)
+++
turbine/core/trunk/src/java/org/apache/turbine/services/ui/TurbineUIService.java
Thu Feb 3 19:44:37 2011
@@ -144,7 +144,7 @@ public class TurbineUIService
/**
* The skin Properties store.
*/
- private HashMap skins = new HashMap();
+ private HashMap<String, Properties> skins = new HashMap<String,
Properties>();
/**
* Refresh the service by clearing all skins.
@@ -179,7 +179,7 @@ public class TurbineUIService
*/
private Properties getSkinProperties(String skinName)
{
- Properties skinProperties = (Properties) skins.get(skinName);
+ Properties skinProperties = skins.get(skinName);
return null != skinProperties ? skinProperties : loadSkin(skinName);
}
@@ -238,7 +238,7 @@ public class TurbineUIService
{
synchronized (skins)
{
- skins = new HashMap();
+ skins = new HashMap<String, Properties>();
}
log.debug("All skins were cleared.");
}
@@ -492,6 +492,7 @@ public class TurbineUIService
/**
* Initializes the service.
*/
+ @Override
public void init() throws InitializationException
{
Configuration cfg = Turbine.getConfiguration();
@@ -516,6 +517,7 @@ public class TurbineUIService
/**
* Returns to uninitialized state.
*/
+ @Override
public void shutdown()
{
clearSkins();
Modified:
turbine/core/trunk/src/java/org/apache/turbine/services/uniqueid/TurbineUniqueIdService.java
URL:
http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/uniqueid/TurbineUniqueIdService.java?rev=1066925&r1=1066924&r2=1066925&view=diff
==============================================================================
---
turbine/core/trunk/src/java/org/apache/turbine/services/uniqueid/TurbineUniqueIdService.java
(original)
+++
turbine/core/trunk/src/java/org/apache/turbine/services/uniqueid/TurbineUniqueIdService.java
Thu Feb 3 19:44:37 2011
@@ -59,6 +59,7 @@ public class TurbineUniqueIdService
* <p> Initializes the service upon first Turbine.doGet()
* invocation.
*/
+ @Override
public void init()
throws InitializationException
{
@@ -90,6 +91,7 @@ public class TurbineUniqueIdService
/**
* <p> Writes a message to the log upon system shutdown.
*/
+ @Override
public void shutdown()
{
log.info("Turbine instance running at " + turbineURL + " shutting
down.");