antville/helma
antville/helma
1
1
Fork 0
Code Issues 5 Pull requests 9 Projects Releases 5 Activity Actions

* Invoke User.onLogout() if and only if a request evaluator is associated

Browse source 
with the current thread (which means it is called from an application script).
  Otherwise, we can assume to be called by the session timeout thread, which 
  takes care of calling User.onLogout().
This commit is contained in:
hns 2007-12-06 20:18:59 +00:00
parent 4afd475510
commit 3fcdfd72ce
1 changed files with 12 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

22
src/helma/framework/core/Session.java
View file

@ -101,23 +101,25 @@ public class Session implements Serializable {
*/
public void logout() {
if (userHandle != null) {
// invoke User.onLogout
RequestEvaluator reval = null;
try {
reval = app.getEvaluator();
reval.invokeInternal(userHandle, "onLogout", new Object[] {sessionId});
// Invoke User.onLogout() iff this is a transactor request with a request
// evaluator already associated (i.e., if this is called from an app/script).
// Otherwise, we assume being called from the scheduler thread, which takes
// care of calling User.onLogout().
RequestEvaluator reval = app.getCurrentRequestEvaluator();
if (reval != null) {
Node userNode = userHandle.getNode(app.nmgr.safe);
if (userNode != null)
reval.invokeDirectFunction(userNode, "onLogout", new Object[] {sessionId});
}
} catch (Exception x) {
// errors should already be logged by requestevaluator, but you never know
// errors should already be logged by request evaluator, but you never know
app.logError("Error in onLogout", x);
} finally {
// do the actual work
// do log out
userHandle = null;
uid = null;
lastModified = System.currentTimeMillis();
// release the evaluator
if (reval != null) {
app.releaseEvaluator(reval);
}
}
}
}