Changes between Initial Version and Version 1 of Ticket #9519, comment 22
- Timestamp:
- Oct 25, 2012, 3:00:07 PM (12 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #9519, comment 22
initial v1 1 If I'm not mistaken, the need for cascade is entirely determined by the model schema; in other words,for a given call to `bulk_delete`, the need for cascade won't change at runtime. In other words, you're not going to have code calling `bulk_delete` that usually works but occasionally fails. So it seems fine to me to omit (3) and just raise an error if `bulk_delete` is used on a model requiring cascade. Implementing (3) sounds to me like a lot of additional deletion code and bug surface area, for little benefit.1 If I'm not mistaken, the need for cascade is entirely determined by the model schema; for a given call to `bulk_delete`, the need for cascade won't change at runtime. In other words, you're not going to have code calling `bulk_delete` that usually works but occasionally fails. So it seems fine to me to omit (3) and just raise an error if `bulk_delete` is used on a model requiring cascade. Implementing (3) sounds to me like a lot of additional deletion code and bug surface area, for little benefit.