search

google-code-export / google-guice

Automatically exported from code.google.com/p/google-guice
Apache License 2.0
2 stars 1 forks source link

Enable struts.devMode Support #355

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
The source for GuiceObjectFactory appears to scan for the struts.devMode
property, but does not do anything with it.  Attached is a patch containing
a suggested solution.

Original issue reported on code.google.com by andy22...@gmail.com on 6 Apr 2009 at 10:24

Attachments:

GoogleCodeExporter commented 9 years ago 
Bob, can you take care of this struts integration issue?

Original comment by limpbizkit on 26 Apr 2009 at 8:14

GoogleCodeExporter commented 9 years ago

Original comment by limpbizkit on 26 Apr 2009 at 9:29

GoogleCodeExporter commented 9 years ago 
FYI, we just removed the reference to struts.devMode since it was unused.
I've never used Guice's Stage feature before, but from reading the JavaDoc I'm 
not sure I'd really want to set it to development when I turn on 
struts.devMode.  The docs say that setting the stage to development will cause 
you to lose "up front error checking", which sounds like something I'd want 
during development though it's hard to say without more details in the docs.

Original comment by benjamin...@gmail.com on 4 Dec 2010 at 6:04