Opened 9 years ago
Last modified 9 years ago
#24902 closed Bug
Unapply squashed migration when any of the individual migrations are unapplied — at Version 2
Reported by: | Markus Holtermann | Owned by: | nobody |
---|---|---|---|
Component: | Migrations | Version: | 1.8 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
This is basically the inverse of #24628: separate migrations, that have been applied, and at a later stage the squash has been marked as applied. When you roll back before the squash it won't be marked as unapplied.
Change History (2)
comment:1 by , 9 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:2 by , 9 years ago
Description: | modified (diff) |
---|---|
Summary: | Unapply separate migrations when squashed migrations exist → Unapply squashed migration when any of the individual migrations are unapplied |
Note:
See TracTickets
for help on using tickets.
Actually, look at the implementation of
apply_migration
andunapply_migration
- when a squash migration is applied or unapplied, it marks only all the replaced migrations as applied or unapplied. So in the case described here, the individual replaced migrations _will_ be marked as unapplied, but the squash migration won't be.In practice this doesn't really matter because the migration loader automatically sets the 'applied' state of any squash migration based on the applied state of all its replaced migrations. #24628 mattered only because you might remove all the replaced migrations and remove the 'replaces' kwarg from the squash migration, and then it isn't a squash migration anymore. You're only supposed to do that once everything is applied everywhere.
That said, I don't see any harm in correctly maintaining the applied state of the squash migration in the database in all cases; it does seem more robust and may avoid future problems.
Updated title and description for accuracy.