Sounds good to me.

Gili

Igor Vaynberg wrote:
fine, lets just have an overriddable String getDeploymentMode() on the app. that will solve the current issue and allow the user to fetch that value from anywhere. by default we can try a sys prop and servlet init param/context param whatever.

does that sound ok?

-Igor


On 2/19/06, *Martijn Dashorst* <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]> > wrote:

    Hmm,

    The setting could also come from a JNDI lookup, or some other
    programmatic way. Perhaps we should take that into account as well.

    Martijn


    On 2/19/06, Igor Vaynberg <[EMAIL PROTECTED]
    <mailto:[EMAIL PROTECTED]>> wrote:
     > because in this situation it makes sense to have this particular
    setting
     > configued externally outside of code so that you can deploy your
    app and not
     > have to recompile it just because now you are running in production!
     >
     > -Igor
     >
     >
     > On 2/19/06, Gili < [EMAIL PROTECTED]
    <mailto:[EMAIL PROTECTED]>> wrote:
     > >
     > >         I'd expect to be able to stick all my configuration
    stuff in
     > init().
     > > Why is it alright to move configure() into XML configuration
    and leave
     > > the rest of the settings in init()? All I'm saying is that
    there should
     > > be consistency.
     > >
     > >         Is there no way for us to make configure() work in
    init()? I
     > understand
     > > it is more work than simply removing configure(), but in my
    view that's
     > > like fixing a bug by removing the feature. Just my 2 cents...
     > >
     > > Gili
     > >
     > > Andrew Lombardi wrote:
     > > > +1 on this.  To make the programmatic configure() work just
    seems like a
     > > > lot of fluff, don't see why you'd ever need to call this
    during the
     > > > app's lifecycle, only at the beginning.
     > >
     > >
     > >
     > > -------------------------------------------------------
     > > This SF.net email is sponsored by: Splunk Inc. Do you grep
    through log
     > files
     > > for problems?  Stop!  Download the new AJAX search engine that
    makes
     > > searching your log files as easy as surfing the  web.  DOWNLOAD
    SPLUNK!
     > >
     >
    http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
    <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642>
     > > _______________________________________________
     > > Wicket-user mailing list
     > > Wicket-user@lists.sourceforge.net
    <mailto:Wicket-user@lists.sourceforge.net>
     > > https://lists.sourceforge.net/lists/listinfo/wicket-user
     > >
     >
     >


    --
    Living a wicket life...

    Martijn Dashorst - http://www.jroller.com/page/dashorst
    <http://www.jroller.com/page/dashorst>

    Wicket 1.1.1 is out: http://wicket.sourceforge.net/wicket-1.1


    -------------------------------------------------------
    This SF.net email is sponsored by: Splunk Inc. Do you grep through
    log files
    for problems?  Stop!  Download the new AJAX search engine that makes
    searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
    http://sel.as-us.falkag.net/sel?cmdlnk&kid3432&bid#0486&dat1642 
<http://sel.as-us.falkag.net/sel?cmdlnk&kid3432&bid#0486&dat1642>
    _______________________________________________
    Wicket-user mailing list
    Wicket-user@lists.sourceforge.net
    <mailto:Wicket-user@lists.sourceforge.net>
    https://lists.sourceforge.net/lists/listinfo/wicket-user



--
http://www.desktopbeautifier.com/


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Reply via email to