Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #9447 (closed defect: duplicate)

Opened 1 year ago

Last modified 1 year ago

[PATCH] Remove deprecated associations options from trunk

Reported by: kampers Assigned to: core
Priority: normal Milestone: 1.x
Component: ActiveRecord Version: edge
Severity: normal Keywords:
Cc:

Description

As suggested in #9127, here's a patch to remove :dependent => true and :exclusively_dependent => true from the trunk. Rails 2.0 here we come, sayonara cruft.

Also includes these changes:

  • Fixes the docs to match, and has some doc cleanup on top of that to clarify things.
  • Removes :dependent => true for has_one, for consistency.
  • I got to remove a no-longer-needed test which had a side effect! (And another test was depending on it! Shame!) That makes me feel warm and buttery inside.

Attachments

remove_deprecated_dependency_options_from_trunk.diff (9.5 kB) - added by kampers on 08/30/07 18:33:09.

Change History

08/30/07 18:33:09 changed by kampers

  • attachment remove_deprecated_dependency_options_from_trunk.diff added.

08/30/07 18:44:12 changed by lifofifo

  • status changed from new to closed.
  • resolution set to duplicate.

Heh..a lil late ;-)

Closing in favor of #9387

08/30/07 18:53:49 changed by kampers

wow. depressing. I even searched for "deprecated associations". oh well.

I guess I'll put up a patch with the doc changes once yours makes it through.