[jira] [Closed] (OFBIZ-7591) Enforce noninstantiability to UtilJavaParse class

["Arun Patidar (JIRA)" <ji...@apache.org>]

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

Arun Patidar closed OFBIZ-7591.
-------------------------------
       Resolution: Fixed
    Fix Version/s: Upcoming Branch

Committed changes in trunk at rev: 1750969.

Thanks [~rishisolankii] and [~rohit.koushal] for your contribution.

> Enforce noninstantiability to UtilJavaParse class
> -------------------------------------------------
>
>                 Key: OFBIZ-7591
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7591
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Rishi Solanki
>            Assignee: Arun Patidar
>            Priority: Minor
>             Fix For: Upcoming Branch
>
>         Attachments: OFBIZ-7591.patch
>
>
> - Make class as final.
> - Add a private constructor.
> - Make the following public static variables as private and move them to top of class. Also move the static block initializing it.
> {code}
>         // FIXME: Not thread safe
>     public static Set<String> serviceMethodNames = new HashSet<String>();
>     static {
>         serviceMethodNames.add("runSync");
>         serviceMethodNames.add("runSyncIgnore");
>         serviceMethodNames.add("runAsync");
>         serviceMethodNames.add("runAsyncWait");
>         serviceMethodNames.add("registerCallback");
>         serviceMethodNames.add("schedule"); // NOTE: the service name may be the 1st, 2nd or 3rd param for variations on this
>         serviceMethodNames.add("addRollbackService");
>         serviceMethodNames.add("addCommitService");
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)