[
https://issues.apache.org/jira/browse/OFBIZ-2948?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12759373#action_12759373
]
Shi Yusen commented on OFBIZ-2948:
----------------------------------
Yes, you're right. I wrote the following at the beginning. Honestly, I don't
know which way is more efficient.
if (Arrays.asList(m.getAnnotations()).contains("static")) {
result = m.invoke(null, dctx, context);
} else {
result = m.invoke(c.newInstance(), dctx, context);
}
> A minor change in org.ofbiz.service.engine.StandardJavaEngine
> -------------------------------------------------------------
>
> Key: OFBIZ-2948
> URL: https://issues.apache.org/jira/browse/OFBIZ-2948
> Project: OFBiz
> Issue Type: Wish
> Affects Versions: Release Branch 9.04
> Environment: FC10 + OpenJDK 1.6
> Reporter: Shi Yusen
> Priority: Trivial
> Fix For: Release Branch 9.04
>
>
> Change
> result = m.invoke(null, dctx, context);
> to
> result = m.invoke(c.newInstance(), dctx, context);
> This change will make it possible to invoke both static method and instance
> method. See
> http://java.sun.com/j2se/1.5.0/docs/api/java/lang/reflect/Method.html#invoke(java.lang.Object,%20java.lang.Object...).
> I've tested this in OFBiz 9.04 and works fine.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
