Yep can certainly merge 

----- Original Message -----

> From: "Alex Eng" <notifications@github.com>
> To: "zanata/zanata-server" <zanata-server@noreply.github.com>
> Cc: "Patrick Huang" <pahuang@redhat.com>
> Sent: Wednesday, September 18, 2013 10:15:05 AM
> Subject: Re: [zanata-server] rhbz1008251 - filter out invalid text flow
> target in TM result (#181)

> In
> zanata-war/src/main/java/org/zanata/webtrans/server/rpc/GetTransMemoryHandler.java:
> > @@ -189,6 +189,10 @@ private static boolean isValidResult(HTextFlowTarget
> > textFlowTarget)
> >        {
> >           return false;
> >        }
> > +      else if (!textFlowTarget.getState().isTranslated())

> Just personal preferences, should this condition check merge to the one
> above?

> if (textFlowTarget == null || !textFlowTarget.getState().isTranslated())
> { return false; }

> —
> Reply to this email directly or view it on GitHub .