Did any of this code ever get accepted into DBForms?  My application that is based on Turbine is starting to add more and more requirements that are difficult to meet with just DBForms..  If I could integrate it better into my application versus having it seperate, it would help a lot.  Being able to subclass the controller seems like a very useful idea anyway.
 
Eric
 
 
-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]]On Behalf Of Mark Dimon
Sent: Tuesday, May 21, 2002 6:41 PM
To: [EMAIL PROTECTED]
Subject: [Jdbforms-developers] Small modification to enable inclusion within other fameworks

Hi All
 
I've successfully managed to get dbforms working within another framework , Apache's Jetpseed portal. And it works very well indeed :) Great Work !
 
To do this I had to make a small modification as follows:
 
in controller.java
 
 
private void process(HttpServletRequest request, HttpServletResponse response)
................
 
  if (errors.size() != 0 && fue != null && fue.trim().length()>0)
 
    //request.getRequestDispatcher(fue).forward(request, response);
                                forwardRequest(fue , request, response); //********
   else
    //request.getRequestDispatcher(e.getFollowUp()).forward(request, response);   
                                forwardRequest(e.getFollowUp() , request, response); //*********
..............
 
//
// protected so can be subclassed and forward changed to include *********
//
protected void forwardRequest(String fue , HttpServletRequest request, HttpServletResponse response ) throws ServletException, IOException{
    
            request.getRequestDispatcher(fue).forward(request, response);       
 
        }
 
In this way I can subclass the controller servlet to perform includes rather than forwards , the only other alternative is to reproduce the entire controller servlet which is not ideal.
 
Would it be possible to include this into the CVS ? , alternatively  a config option could be included to change the requestDispatcher from forward to include ?
 
Regards
 
    Mark
 
 

Reply via email to