Hubert Rabago wrote:
Well, I extracted the form bean initialization into an
initModuleFormBeans(moduleConfig) method which matches other methods like it. Here's a snippet from ActionServlet.init():


            initModuleConfigFactory();
            // Initialize modules as needed
            ModuleConfig moduleConfig = initModuleConfig("", config);
            initModuleMessageResources(moduleConfig);
            initModuleDataSources(moduleConfig);
            initModulePlugIns(moduleConfig);
            moduleConfig.freeze();

With other config elements being extended, the method
could be called processModuleExtensions(moduleConfig) and be called along with these methods.


These methods look like they can be moved elsewhere, but
that's another thread.  :)

Any comment on the extends implementation?  Should I proceed,
or is the timing not right what with everybody working on the
chain implementation and repo reorg?

I have yet to take a look at the implementation, but regarding timing, I think it is perfect and am very grateful to you for taking this on. If I remember correctly, this is the last major feature we wanted to put in 1.3. After this gets in, we can stabilize the code, figure out what the release should look like with all these subprojects, and get 1.3 out the door. Thanks again for looking into this!


Don


- Hubert


On Tue, 22 Feb 2005 09:39:54 -0600, Joe Germuska <[EMAIL PROTECTED]> wrote:

At 9:07 AM -0600 2/22/05, Hubert Rabago wrote:

I tried this over the weekend, starting with form beans.
Here's what my sample app had:

Hubert: the description of intent for the XML is right-on.


The extension gets realized before the loop which creates and
registers the DynaActionFormClass instances.  This code is still
in the ActionServlet.initModuleConfig() method, so I've extracted
this into its own method.  Simple checks for circular inheritance
can be easily done by FormBeanConfig, but more complex checking
should probably be done by a separate method in ActionServlet, or
maybe in a util class, such as ModuleUtils.

Comments, please.

Now may not be the time, but my long-term preference would be to extract as much as possible out of the ActionServlet, so that it simply becomes a choreographer. I think this is a good pre-cursor to having more flexible options for how to configure Struts (as well as forcing us to be more deliberate about where we permit dependencies on the Servlet API). From a purely conceptual standpoint, I almost think that everything inside initModuleConfig() should be moved into the ModuleConfigFactory.

But in the meantime, I think putting it in ModuleUtils is probably
better than having it in the ActionServlet itself.

I don't know if other people share my opinions about this.

Joe

--
Joe Germuska
[EMAIL PROTECTED]
http://blog.germuska.com
"Narrow minds are weapons made for mass destruction"  -The Ex



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to