+1... But just to make sure: is the DEPLOYMENT configuration going to be
default?
--
Bruno Borges
Summa Technologies Inc.
www.summa-tech.com
(11) 8565-7739
(11) 3055-2060
(48) 8404-1300
On 5/31/07, Jean-Baptiste Quenot <[EMAIL PROTECTED]> wrote:
* Al Maw:
> - We default to DEVELOPMENT and we should make it more obvious that you
> should set DEPLOYMENT for production servers.
+1 many people I know deploy their application in DEVELOPMENT
mode. The Wicket control panel should be visible in development
mode without the possibility to turn it off, like you mentioned in
WICKET-599.
> - We currently provide the following methods to configure which mode
> you run in:
> - Set a System property.
> - Set a servlet context param.
> - Set a filter param.
>
> If you don't fit into any of those, it makes life difficult - you
> don't want to just call configure(DEPLOYMENT) in your init() method,
> because then configure() is called twice - once by Wicket internally,
> probably in default DEVELOPMENT mode, and once by you manually. This
> results in a bunch of DEVELOPMENT mode stuff running on your live
> environment, and it's not that obvious that this is what's going on.
> (See WICKET-6, where even JBQ doesn't seem to entirely get this.)
Yes I get it entirely but you didn't mention that in the issue
description (that configure() is called twice) so it was not very
clear.
> Err, can I call a vote on this please?
> [x] Yes, great idea, let's get it into 1.3.0-beta2
> [ ] No, that's rubbish, because...
+1 for getConfigurationType() with a default implementation for
backwards compatibility.
--
Jean-Baptiste Quenot
aka John Banana Qwerty
http://caraldi.com/jbq/
