Are you confident the check before should not more correctly be "factType
!= null" ?


On 5 March 2014 00:39, Petr Široký <notifications@github.com> wrote:

> In
> drools-wb-screens/drools-wb-guided-rule-editor/drools-wb-guided-rule-editor-client/src/main/java/org/drools/workbench/screens/guided/rule/client/widget/ActionCallMethodWidget.java:
>
> > @@ -112,7 +112,7 @@ private void getMethodInfosForBasicFactType() {
> >          String factType = getFactTypeLHS();
> >
> >          if (factType == null) {
> > -            factType = getFactTypeFromRHS(factType);
> > +            factType = getFactTypeFromRHS(null);
>
> The factType is clearly null at this line, so make it clear that the
> method getFactTypeFromRHS() is called with null as param.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/droolsjbpm/drools-wb/pull/9/files#r10283070>
> .
>