Changes between Initial Version and Version 1 of Ticket #9519, comment 22


Ignore:
Timestamp:
Oct 25, 2012, 3:00:07 PM (12 years ago)
Author:
Carl Meyer

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.
     1If 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.
Back to Top