Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

remove garbage dependencies ... #215

Open
wants to merge 1 commit into from

3 participants

@rodrigobaron

After any story is removed, the app show garbages about the removed story. This happened because the dependencies of stories is loaded .. To correct that only put ":dependent => :destroy" in changests references on Story model.

@marnen

Better yet, have a foreign key constraint ON DELETE CASCADE in the DB (the Foreigner gem will help with this).

@rodrigobaron

humm .. Thanks @marnen ...

@malclocke
Owner

Re-opening as I think we will probably take this as is.

@malclocke malclocke reopened this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 27, 2014
  1. @rodrigobaron
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  app/models/story.rb
View
2  app/models/story.rb
@@ -25,7 +25,7 @@ class Story < ActiveRecord::Base
belongs_to :owned_by, :class_name => 'User'
validates :owned_by_id, :belongs_to_project => true
- has_many :changesets
+ has_many :changesets, :dependent => :destroy
has_many :notes do
# Creates a collection of rows on this story from a CSV::Row instance
Something went wrong with that request. Please try again.