On 8/17/05, Corey Probst <[EMAIL PROTECTED]> wrote:
> I see this commit for TilesPreProcessor but the nightly build still
> seems to have the older version referring to the deprecated
> ForwardConfig constructor.
> 
> Am I missing something here?

For Tiles, you have to be more specific about knowing which module the
commit updated, and which nightly build you are talking about ... the
version of Tiles integrated into Struts, or the standalone Tiles
version in the sandbox.  In this particular case, Joe's changes were
to the former, but they haven't yet been ported over to the latter. 
I'll try to catch up on those this weekend, if nobody beats me to it.

Craig

> 
> On 8/16/05, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
> > Author: germuska
> > Date: Tue Aug 16 12:50:39 2005
> > New Revision: 233045
> >
> > URL: http://svn.apache.org/viewcvs?rev=233045&view=rev
> > Log:
> > fix ForwardConfig constructor to reflect removal of deprecated form in 
> > struts-core
> >
> > Modified:
> >    
> > struts/tiles/trunk/src/java/org/apache/struts/tiles/commands/TilesPreProcessor.java
> >
> > Modified: 
> > struts/tiles/trunk/src/java/org/apache/struts/tiles/commands/TilesPreProcessor.java
> > URL: 
> > http://svn.apache.org/viewcvs/struts/tiles/trunk/src/java/org/apache/struts/tiles/commands/TilesPreProcessor.java?rev=233045&r1=233044&r2=233045&view=diff
> > ==============================================================================
> > --- 
> > struts/tiles/trunk/src/java/org/apache/struts/tiles/commands/TilesPreProcessor.java
> >  (original)
> > +++ 
> > struts/tiles/trunk/src/java/org/apache/struts/tiles/commands/TilesPreProcessor.java
> >  Tue Aug 16 12:50:39 2005
> > @@ -215,9 +215,7 @@
> >             // by an AbstractPerformForward later as if our ForwardConfig
> >             // were the one actually returned by an executing Action
> >             log.info("Tiles process complete; forward to " + uri);
> > -            // :FIXME: How do we need to coordinate the "context-relative" 
> > value
> > -            // with other places it might be set.  For now, hardcode to 
> > true.
> > -            sacontext.setForwardConfig( new ForwardConfig("tiles-chain", 
> > uri, false, true) );
> > +            sacontext.setForwardConfig( new ForwardConfig("tiles-chain", 
> > uri, false) );
> >             return (false);
> >         }
> >     }
> >
> >
> >
> > ---------------------------------------------------------------------
> > 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]
> 
>

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

Reply via email to