@@ -1,8 +1,10 @@ # save and validation support for associations. class ActiveRecord::Base - def associated_valid? + def associated_valid?(path = []) + return true if path.include?(self) # prevent recursion (if associated and parent are new records) + path << self # using [].all? syntax to avoid a short-circuit - with_unsaved_associated { |a| [a.valid?, a.associated_valid?].all? {|v| v == true} } + with_unsaved_associated { |a| [a.valid?, a.associated_valid?(path)].all? {|v| v == true} } end def save_associated lib/extensions/unsaved_associated.rb e38b2381baf24ed58df64719da324a9c1559b6a7 Tim Harper timcharper@gmail.com http://github.com/activescaffold/active_scaffold/commit/1e6a79b926729c4d27a1e5d0b9b24b0df6fface8 1e6a79b926729c4d27a1e5d0b9b24b0df6fface8 2008-06-29T01:04:59-07:00 2008-06-29T00:15:02-07:00 fixed infinite recursion issue in demo associated_valid? would get stuck in an infinite loop if creating two new associated records in the same save transaction. a2cc4fa9e3fd7a3f0b7fb6d14d09aae5c67908fa Tim Harper timcharper@gmail.com