[#700] complain about useless :unique => true declaration

tslocke · Sep 21, 2011 · bbe3ac5 · bbe3ac5
1 parent aceb7af
commit bbe3ac5
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/hobo_fields/lib/hobo_fields/model.rb b/hobo_fields/lib/hobo_fields/model.rb
@@ -170,7 +170,11 @@ def self.add_formatting_for_field(name, type, args)
 
     def self.add_index_for_field(name, args, options)
       to_name = options.delete(:index)
-      return unless to_name
+      unless to_name
+        # passing :unique => true doesn't do anything without an index
+        Rails.logger.error('ERROR: passing :unique => true without :index => true does nothing. Use :unique instead.') if options[:unique]
+        return
+      end
       index_opts = {}
       index_opts[:unique] = :unique.in?(args) || options.delete(:unique)
       # support :index => true declaration