aboutsummaryrefslogtreecommitdiffstats
path: root/spec/javascript
diff options
context:
space:
mode:
authorTeddy Wing2017-09-11 12:39:14 +0200
committerTeddy Wing2017-09-11 12:49:45 +0200
commit7ff60302d391afd850733d7ddade238bef502d70 (patch)
tree5e8f19e9cab501452044438e07bab3052a2e158d /spec/javascript
parent70215298fcb592e52b0f3033a939a594f511e379 (diff)
downloadchouette-core-7ff60302d391afd850733d7ddade238bef502d70.tar.bz2
ImportMessage: Remove `dependent: :destroy` on :resource association
It doesn't really make sense to have a `dependent: :destroy` here. For one thing, the `Import` class has the necessary `dependent: :destroy` definitions to remove its associated objects when it gets deleted. At this point, according to Luc, we don't envision deleting `ImportResource`s separately from `Import`s. The other weird thing is that: * `ImportResource` has many `ImportMessage`s * `ImportMessage` has one `ImportResource` With the `dependent: :destroy` set up the way it is here on `ImportMessage`, we could potentially be deleting an `ImportResource` with other `ImportMessage` children that would then become orphaned. For both these reasons, it seems better to remove this trigger. Refs #4412
Diffstat (limited to 'spec/javascript')
0 files changed, 0 insertions, 0 deletions