Re: LocalizedTextUtil mods

2014-03-06 Thread Lukasz Lenart
2014-03-06 17:51 GMT+01:00 Greg Huber : > Following this http://struts.apache.org/git-for-struts.html > > url https://github.com/apachestruts/struts don't look correct? 404 >> > apache/struts It suppose to be https://github.com/apache/struts - where did f

Re: LocalizedTextUtil mods

2014-03-06 Thread Greg Huber
Following this http://struts.apache.org/git-for-struts.html url https://github.com/apachestruts/struts don't look correct? 404 >> apache/struts On 6 March 2014 13:34, Lukasz Lenart wrote: > 2014-03-06 13:38 GMT+01:00 Greg Huber : > > Seeing this mod t

Re: LocalizedTextUtil mods

2014-03-06 Thread Greg Huber
Every will scan up the package hierarchy ie is does these getMessage(..) findText(..) calls: ListEvents.title : org.events.ui.struts2.editor.Events ListEvents.title : org.events.ui.struts2.editor.EventBase ListEvents.title : org.events.ui.struts2.util.ActionBase ListEvents.title : com.opensymph

Re: LocalizedTextUtil mods

2014-03-06 Thread Lukasz Lenart
2014-03-06 13:38 GMT+01:00 Greg Huber : > Seeing this mod the other day has reminded me that I do a local change on > LocalizedTextUtil to speed up the screens. > > What I do is to add a check to see if the property comes from the > getDefaultMessage(..) method (ie from ApplicationResources.propert

Re: LocalizedTextUtil mods

2014-03-06 Thread Dave Newton
What's the actual speedup? It might be nice if it was configurable; I'm not sure what the stats would be regarding how many apps use S2 vs. "regular" localization. Dave On Thu, Mar 6, 2014 at 7:38 AM, Greg Huber wrote: > Seeing this mod the other day has reminded me that I do a local change

LocalizedTextUtil mods

2014-03-06 Thread Greg Huber
Seeing this mod the other day has reminded me that I do a local change on LocalizedTextUtil to speed up the screens. What I do is to add a check to see if the property comes from the getDefaultMessage(..) method (ie from ApplicationResources.properties) and if it does skip all the checking that it