[
http://jira.undersunconsulting.com/browse/OFBIZ-82?page=comments#action_13837 ]
Jacopo Cappellato commented on OFBIZ-82:
----------------------------------------
Is it something we should include now? Since the patch is probably old now (and its application could fail) we could close this issue and maybe get a new one in the future from Adam.
> protect the member variables in ContainerConfig, and use better OO separation
> -----------------------------------------------------------------------------
>
> Key: OFBIZ-82
> URL:
http://jira.undersunconsulting.com/browse/OFBIZ-82> Project: [OFBiz] Open For Business
> Type: Improvement
> Versions: SVN
> Reporter: Adam Heath
> Assignee: Jira Administrator
> Priority: Minor
> Fix For: SVN
> Attachments: oo-sep.patch
>
>
> The fields in the property/container classes are public. This is poor OO design. The attached patch changes all code to use accessor methods to get the data. This is needed for a later patch.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.undersunconsulting.com/secure/Administrators.jspa-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
Locked
