Feature/disable users into release

Exactly! Nullable=false is needed because there were already records in
there. Then the default can be set. Also, new instances don't pick up the
server_default/default default value until they're saved. Couple things
like that made this choice of using the constructor seem like the way to go.

On Thu, Oct 2, 2014 at 4:31 PM, Brad (Bradley) Momberger <
notifications@github.com> wrote:

> In src/ggrc/models/person.py:
>
> > @@ -33,6 +34,12 @@ class Person(HasOwnContext, Base, db.Model):
> >        uselist=False,
> >        )
> >
> > +  def __init__(self, **kwargs):
>
> Ah, now I think I understand. I think to get it to reliably set the
> default value with the declarative layer, you also need nullable=False.
> At least that's been my experience with it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/reciprocity/ggrc-core/pull/1854/files#r18374556>.
>