The two I added are used to create a status with either the enum, or a
string.
zeus asked me to leave the original intact


On Thu, Mar 14, 2013 at 4:51 PM, Adrian Likins <notifications@github.com>wrote:

> In src/main/java/org/candlepin/model/Status.java:
>
> > @@ -50,6 +53,29 @@ public Status(Boolean result, String version, String release, Boolean standalone
> >          this.standalone = standalone;
> >          this.timeUTC = new Date();
> >          this.rulesVersion = rulesVersion;
> > +        this.setRulesSource(Rules.RulesSource.UNDEFINED);
> > +    }
> > +
> > +    public Status(Boolean result, String version, String release, Boolean standalone,
> > +        String rulesVersion, String rulesSource) {
> > +        this.result = result;
> > +        this.version = version;
> > +        this.release = release;
> > +        this.standalone = standalone;
> > +        this.timeUTC = new Date();
> > +        this.rulesVersion = rulesVersion;
> > +        this.setRulesSource(rulesSource);
> > +    }
> > +
> > +    public Status(Boolean result, String version, String release, Boolean standalone,
>
> Aside from test cases, we only seem to use one of these constructors, can
> we ditch the rest?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/candlepin/candlepin/pull/197/files#r3381417>
> .
>