On Thu, Oct 01, 2015 at 04:14:23PM +0100, Finucane, Stephen wrote: > > On Thu, Oct 01, 2015 at 03:52:25PM +0100, Stephen Finucane wrote: > > > @@ -243,7 +243,7 @@ class Patch(models.Model): > > > date = models.DateTimeField(default=datetime.datetime.now) > > > submitter = models.ForeignKey(Person) > > > delegate = models.ForeignKey(User, blank = True, null = True) > > > - state = models.ForeignKey(State, > > default=get_default_initial_patch_state) > > > + state = models.ForeignKey(State, null=True) > > > > I was wondering, is it necessary to change the null contraint here? > > To the best of my knowledge, yes. We need a default but we don't > always have any entries in the 'State' table (for example, when > testing). As a result the only possible "default" is None. Note that > this will never be set to a null value however due to the additions to > the 'save' function.
I guess it's a bit weird to weaken the db schema for the testing use case but it's doesn't really matter much to me. By the way this patch already had an a-b tag from me. Acked-by: Damien Lespiau <[email protected]> -- Damien _______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
