[PATCH] validates_uniqueness_of scope request.

Sorry, I don't have time to follow setup instuctions for making an official patch diff, but I modified the validates_uniqueness_of function locally to accept an array of column names for the scope (as shown above in the description)... if I have a chance this weekend, I'll check about making an official patch.

def validates_uniqueness_of(*attr_names)

configuration = { :message => ActiveRecord::Errors.default_error_messages[:taken] } configuration.update(attr_names.pop) if attr_names.last.is_a?(Hash) if scope = configuration[:scope]

validates_each(attr_names,configuration) do |record, attr_name, value|

if scope.is_a?(Array)

full_scope = "" scope.each do |s|

full_scope += sanitize_sql(["#{s} = ?", record.send(s)]) full_scope += " AND " if s != scope.last

end

else

full_scope = sanitize_sql(["#{scope} = ?", record.send(scope)])

end

record.errors.add(attr_name, configuration[:message]) if record.class.find_first(record.new_record? ? ["#{attr_name} = ? AND #{full_scope}", record.send(attr_name)] : ["#{attr_name} = ? AND #{record.class.primary_key} <> ? AND #{full_scope}", record.send(attr_name), record.send(:id)])

end

else

validates_each(attr_names,configuration) do |record, attr_name, value|

record.errors.add(attr_name, configuration[:message]) if record.class.find_first(record.new_record? ? ["#{attr_name} = ?", record.send(attr_name)] : ["#{attr_name} = ? AND #{record.class.primary_key} <> ?", record.send(attr_name), record.send(:id) ] )

end

end

end

Whoa! Sorry about that formatting... I just attached a file: validates_uniqueness_of.rb.

Setting all XPATCHs out to 1.x

I finally got around

I finally got around to putting together a patch for this request. It includes the actual change to validates_uniqueness_of, documentation updates, and a new unit test.

Fixed some typos in my comments. Ignore the first diff.

One more!!! My local connection changes are mistakenly in the second one.

(In [3206]) Allow validate_uniqueness_of to be scoped by more than just one column. Closes #1559.

Thanks for including comprehensive tests.