You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by sv...@apache.org on 2020/11/14 23:37:48 UTC

[wicket] 03/05: WICKET-6847: swap listeners and RequestCycle onEndRequest

This is an automated email from the ASF dual-hosted git repository.

svenmeier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/wicket.git

commit 0cb3d6798f4056352c8214ade49fe5fd5947e5c9
Author: Emond Papegaaij <em...@topicus.nl>
AuthorDate: Wed Nov 11 16:21:19 2020 +0100

    WICKET-6847: swap listeners and RequestCycle onEndRequest
    
    This makes sure the session is destroyed (when needed) after the page
    store handles the touched pages. This latter action can create a new
    session if it was destroyed before, causing logout to fail.
---
 .../src/main/java/org/apache/wicket/request/cycle/RequestCycle.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/wicket-core/src/main/java/org/apache/wicket/request/cycle/RequestCycle.java b/wicket-core/src/main/java/org/apache/wicket/request/cycle/RequestCycle.java
index 2c0a8c7..33d1335 100644
--- a/wicket-core/src/main/java/org/apache/wicket/request/cycle/RequestCycle.java
+++ b/wicket-core/src/main/java/org/apache/wicket/request/cycle/RequestCycle.java
@@ -263,8 +263,8 @@ public class RequestCycle implements IRequestCycle, IEventSink, IMetadataContext
 		{
 			try
 			{
-				onEndRequest();
 				listeners.onEndRequest(this);
+				onEndRequest();
 			}
 			catch (RuntimeException e)
 			{