I have a table in production that is integrated everywhere in the system, now I need to add a new column in the table with a default value, but don't want to change all the existing logic, what is the best way to do that?
class People(models.Model):
name = models.CharField(max_length=20)
gender = models.CharField(max_length=20)
class = models.CharField(max_length=20)
in the system, we have this kind of query everywhere
People.objects.filter(gender='male')
People.objects.filter(gender='female', )
...
Now we need to add a new field:
class People(models.Model):
name = models.CharField(max_length=20)
gender = models.CharField(max_length=20)
class = models.CharField(max_length=20)
graduated = models.BooleanField(default=False)
Assume that all the existing data should have graduated
is False, so all the existing logic should work if we can add graduated=False
on every query, but is there any way we can do so we don't need to change any of the existing code but they will assume graduated=False
?
CodePudding user response:
Yes, you can make a manager such that .objects
will only retain People
with graduated=False
:
class PeopleManager(models.Manager):
def get_queryset(self):
return super().get_queryset().filter(graduated=False)
class People(models.Model):
name = models.CharField(max_length=20)
gender = models.CharField(max_length=20)
class = models.CharField(max_length=20)
graduated = models.BooleanField(default=False)
objects = PeopleManager()
all = models.Manager()
You can use People.all.all()
to retrieve all People
, and People.objects.all()
to retrieve all People
that did not graduate.
That being said, I do not recommend this: often People.objects.all()
gives the impression that one will retrieve all People
. As the Zen of Python says: "explicit over implicit": it is better that the code explains and hints what it is doing, not move filtering somewhere hidden in a manager.