[ 
http://www.stripesframework.org/jira/browse/STS-785?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Marcus Kraßmann updated STS-785:
--------------------------------

    Attachment: publicEventGetterAndProtectedEventConstant.patch

> Add "event" for subclasses of OnwardResolution
> ----------------------------------------------
>
>                 Key: STS-785
>                 URL: http://www.stripesframework.org/jira/browse/STS-785
>             Project: Stripes
>          Issue Type: Improvement
>    Affects Versions: Release 1.5.4
>            Reporter: Nikolaos
>            Priority: Trivial
>             Fix For: Release 1.5.6, Release 1.6
>
>         Attachments: publicEventGetterAndProtectedEventConstant.patch
>
>
> When attempting to create things like LocalizedRedirectResolution one 
> subclasses RedirectResolution
> However in:  net.sourceforge.stripes.action.OnwardResolution
> We have:
>     private static final String VALUE_NOT_SET = "VALUE_NOT_SET";
>     private String path;
>     private String event = VALUE_NOT_SET;
>     private Map<String,Object> parameters = new HashMap<String,Object>();
>     private String anchor;
> Problem #1:
> ----------------
> path, anchor, and parameters have public getters/setters however 'event' 
> attribute does not?
> REQUEST:  Please add public getters/setters for event
> NOTE:  One could make the above things protected to be accessible in 
> subclasses but I suggest public as it is in-line with how the rest of the 
> class is written.
> Problem #2:
> ----------------
> Having public access to to 'event' is great but one needs to be able to 
> compare to see if the value is not set however the following is private:
>     private static final String VALUE_NOT_SET = "VALUE_NOT_SET";
> REQUEST:  Please make the above static at least protected vs. private so as 
> to be able to compare within subclasses.
> This is a really trivial request that makes the class more consistent.
> --Nikolaos

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

------------------------------------------------------------------------------
Gaining the trust of online customers is vital for the success of any company
that requires sensitive data to be transmitted over the Web.   Learn how to 
best implement a security strategy that keeps consumers' information secure 
and instills the confidence they need to proceed with transactions.
http://p.sf.net/sfu/oracle-sfdevnl 
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to