You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by Christian Stone <xt...@stonescape.net> on 2009/11/06 18:40:00 UTC

Re: WW-3291. Checkin and moving forward...

I was wondering if you had a chance to commit the patch I submitted  
with this bug [WW-3291].  I did more work in the project with another  
issue (using freemarker and velocity templates interchangeably as  
decorators with the same *full* value stack) and wish to push that.   
Since both bugs take place in the same codebase, I am wondering if I  
should just send you a single patch for both, or wait until the one is  
closed before creating a patch for the other issue.

On a side note, I moved the changes I made into several production  
clients.  I have websites with more than a dozen decorators and  
hundreds of view files.  I am _finally_ able to start the migration to  
Freemarker from Velocity that I wasn't able to do before.  I converted  
4 of the files to Freemarker without having the overhead of needing to  
convert all of them in one sitting.  It is glorious.  Also, I am able  
to access all the statics and whatnot that I couldn't before, so using  
static keys is a huge blessing for readability.  Also, the enhanced  
error reporting has already helped me several times diagnose issues on  
the server.

Anyways, congrats on the work you just accomplished.  Two of the  
biggest complaints on Struts 2 are speed and error reporting.  Any  
steps to mitigate either of these issues is a huge win!

-- Christian Stone


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


Re: WW-3291. Checkin and moving forward...

Posted by Musachy Barroso <mu...@gmail.com>.
I am sorry, I was to excited about the JUEL plugin and totally forgot
about it. If you have additional changes, create a new patch and I
will merge that one.

musachy

On Fri, Nov 6, 2009 at 9:40 AM, Christian Stone <xt...@stonescape.net> wrote:
> I was wondering if you had a chance to commit the patch I submitted with
> this bug [WW-3291].  I did more work in the project with another issue
> (using freemarker and velocity templates interchangeably as decorators with
> the same *full* value stack) and wish to push that.  Since both bugs take
> place in the same codebase, I am wondering if I should just send you a
> single patch for both, or wait until the one is closed before creating a
> patch for the other issue.
>
> On a side note, I moved the changes I made into several production clients.
>  I have websites with more than a dozen decorators and hundreds of view
> files.  I am _finally_ able to start the migration to Freemarker from
> Velocity that I wasn't able to do before.  I converted 4 of the files to
> Freemarker without having the overhead of needing to convert all of them in
> one sitting.  It is glorious.  Also, I am able to access all the statics and
> whatnot that I couldn't before, so using static keys is a huge blessing for
> readability.  Also, the enhanced error reporting has already helped me
> several times diagnose issues on the server.
>
> Anyways, congrats on the work you just accomplished.  Two of the biggest
> complaints on Struts 2 are speed and error reporting.  Any steps to mitigate
> either of these issues is a huge win!
>
> -- Christian Stone
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org