I need a ContentState to represent unsaved state. I was using New but that seems to be a bit confusing. 

----- Original Message -----

> From: "Sean Flanigan" <notifications@github.com>
> To: "zanata/zanata-server" <zanata-server@noreply.github.com>
> Cc: "Patrick Huang" <pahuang@redhat.com>
> Sent: Thursday, July 18, 2013 10:17:05 AM
> Subject: Re: [zanata-server] New translation history UI (#54)

> In
> zanata-war/src/main/java/org/zanata/webtrans/client/ui/ContentStateRenderer.java:
> > + */
> > +@Singleton
> > +public class ContentStateRenderer extends EnumRenderer<ContentState>
> > +{
> > +   private final EnumMessages messages;
> > +
> > +   @Inject
> > +   public ContentStateRenderer(EnumMessages messages)
> > +   {
> > +      this.messages = messages;
> > +   }
> > +
> > +   @Override
> > +   public String render(ContentState object)
> > +   {
> > +      if (object == null)

> Rather than have to handle null here (and elsewhere), can't we find a way to
> make sure ContentState values are never null?

> —
> Reply to this email directly or view it on GitHub .