[jira] Updated: (STR-2965) avoid calling member methods that potentially can be overridden by subclasses

["Paul Benedict (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/struts/browse/STR-2965?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Paul Benedict updated STR-2965:
-------------------------------

    Fix Version/s: Future

> avoid calling member methods that potentially can be overridden by subclasses
> -----------------------------------------------------------------------------
>
>                 Key: STR-2965
>                 URL: https://issues.apache.org/struts/browse/STR-2965
>             Project: Struts 1
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Yusheng Wang
>            Priority: Minor
>             Fix For: Future
>
>
> The class ForwardConfig has two constructors that call other member method to set the class instances variables. This is generally a poor programming style. Since the member methods are not final and public. They can be overridden by subclasses. It will potntially end up with a object of class is constructed in invalid state

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.