[ 
http://www.stripesframework.org/jira/browse/STS-429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=11197#action_11197
 ] 

John Newman commented on STS-429:
---------------------------------

Anyone else think this is getting a bit unnecessarily complicated? KISS.  If 
you need that much control over the variable for specific events, can't you 
just manage it yourself with code? 

IMO all this thing should do

public @interface session  {
   int timeout() default -1;
   TimeUnit timeoutUnit() default TimeUnit.MINUTES;
   String key() default "";
}

before binding: pull from session based on supplied key or field name if none, 
check timeout
let binding phase possibly overwrite
after binding, write to session.

all that other stuff is complication outside of our favorite 80/20 rule. 

> @Session annotation to save and restore fields automatically
> ------------------------------------------------------------
>
>                 Key: STS-429
>                 URL: http://www.stripesframework.org/jira/browse/STS-429
>             Project: Stripes
>          Issue Type: New Feature
>            Reporter: Christian Poitras
>            Priority: Minor
>         Attachments: session-plugin.jar, session-plugin2.jar, 
> session-plugin3.jar, session-plugin4.jar, session-plugin5.jar
>
>
> I've added a @Session annotation and an interceptor that automatically saves 
> annotated fields in session and restores the fields on following request.
> This could be a great add-on to Stripes.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://www.stripesframework.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to