> -----Original Message-----
> From: Antonio Gallardo [mailto:[EMAIL PROTECTED] 
> Sent: Wednesday, January 14, 2004 8:36 AM
> To: [EMAIL PROTECTED]
> Subject: RE: [BUG] SessionModule creates sessions on default
> 
> 
> Reinhard Poetz dijo:
> > After following this thread I think the use of getSession(false) is 
> > the right behaviour because as Michael explained the 
> implicit session 
> > creation within a module can be very confusing and IMHO the wrong 
> > place. So +1 changing the method.
> 
> +0 for changing the method.
> 
> I have fear of the potential code we can broke by changing 
> the behavior of the getSession() method.

That's true but IMO this is the wrong place to create a session.
Additionally the behaviour of this input module is confusing because if
no value is found I think "null" is returned. Maybe we should throw an
exception in the case of no existing session because if you use an input
module you want to read out data, not more and not less.

--
Reinhard

Reply via email to