beehive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rich Feit <richf...@gmail.com>
Subject Re: fix for BEEHIVE-1003
Date Fri, 11 Nov 2005 18:14:23 GMT
All the public methods in FlowControllerFactory for creating page flows
also store the page flow in the session.  It's pretty similar to hitting
the page flow directly.  When another page flow is hit, or when the
session expires, the controls in the page flow will be cleaned up.

Rich

Carlin Rogers wrote:

>Eddie,
>
>Thanks for the help. The scenario/interest for doing this is the case of
>something like a portal that creates an instance of a page flow using the
>FlowControllerFactory.
>
>Good questions about the contract to go through the entire life cycle of the
>controller in this case and the concern for leaks. I'm not sure about the
>answers... Maybe I can defer to Rich here.
>
>Yes, I can definitely look at improving the test to drive the page flow
>through its life cycle.
>
>Regards,
>Carlin
>
>On 11/11/05, Eddie O'Neil <ekoneil@gmail.com> wrote:
>  
>
>>Carlin--
>>
>>I've been looking at this fix and at the test and am curious about a
>>couple of things. First, why is this an interesting thing to do --
>>ie, what's the use case? If a controller is created outside of the
>>runtime, isn't there a contract that says that the flow controller
>>will be driven through its entire lifecycle?
>>
>>The problem is that simply by creating the page flow, controls will
>>be initialized, but without driving the controller through its
>>lifecycle, they're never torn down. So, what's the contract between
>>the caller and the JPF instance here? I've not tried this specific
>>case, but if the control being used from the test accessed a DB, it
>>would leak ResultSets in this case. Similar things could be true for
>>other control types.
>>
>>Also, can we expand the test to ensure that the right thing happens
>>to the control bean context when the JPF *is* driven through its
>>lifecycle?
>>
>>Thanks...
>>
>>Eddie
>>
>>
>>
>>---------- Forwarded message ----------
>>From: crogers@apache.org <crogers@apache.org>
>>Date: Nov 11, 2005 12:37 AM
>>Subject: svn commit: r332482 - in /beehive/trunk/netui:
>>src/pageflow/org/apache/beehive/netui/pageflow/internal/
>>test/webapps/drt/coreWeb/WEB-INF/
>>test/webapps/drt/coreWeb/miniTests/createPageFlow/
>>test/webapps/drt/src/miniTests/createPageFlow/ test/webapps/drt...
>>To: commits@beehive.apache.org
>>
>>
>>Author: crogers
>>Date: Thu Nov 10 23:37:13 2005
>>New Revision: 332482
>>
>>URL: http://svn.apache.org/viewcvs?rev=332482&view=rev
>>Log:
>>Fix for http://issues.apache.org/jira/browse/BEEHIVE-1003 : Calling
>>FlowControllerFactory.createPageFlow() outside of page flow request
>>processor and page filter does not initialize controls correctly.
>>
>>tests: drt, bvt in netui (WinXP)
>>
>>
>>Added:
>>beehive/trunk/netui/test/webapps/drt/coreWeb/miniTests/createPageFlow/
>>
>>beehive/trunk/netui/test/webapps/drt/coreWeb/miniTests/createPageFlow/Controller.java
>>(with props)
>>
>>beehive/trunk/netui/test/webapps/drt/coreWeb/miniTests/createPageFlow/test.jsp
>>(with props)
>>beehive/trunk/netui/test/webapps/drt/src/miniTests/createPageFlow/
>>
>>beehive/trunk/netui/test/webapps/drt/src/miniTests/createPageFlow/CreatePageFlowServlet.java
>>(with props)
>>
>>beehive/trunk/netui/test/webapps/drt/src/miniTests/createPageFlow/controls/
>>
>>beehive/trunk/netui/test/webapps/drt/src/miniTests/createPageFlow/controls/Portfolio.java
>>(with props)
>>
>>beehive/trunk/netui/test/webapps/drt/src/miniTests/createPageFlow/controls/PortfolioControl.java
>>(with props)
>>
>>beehive/trunk/netui/test/webapps/drt/src/miniTests/createPageFlow/controls/PortfolioControlImpl.java
>>(with props)
>>
>>beehive/trunk/netui/test/webapps/drt/src/miniTests/createPageFlow/controls/Stock.java
>>(with props)
>>beehive/trunk/netui/test/webapps/drt/testRecorder/tests/CreatePageFlow.xml
>>(with props)
>>Modified:
>>
>>beehive/trunk/netui/src/pageflow/org/apache/beehive/netui/pageflow/internal/JavaControlUtils.java
>>beehive/trunk/netui/test/webapps/drt/coreWeb/WEB-INF/web.xml
>>beehive/trunk/netui/test/webapps/drt/testRecorder/config/testRecorder-
>>tests.xml
>>
>>    
>>
>
>  
>

Mime
View raw message