Uniqueness validation does not work

[shioyama]

Uniqueness validation does not work

[shioyama]

That's an issue with uniqueness, because it checks at the database column level. Globalize required a hack to get that to work (I actually built that hack, so I know it well).

If you remove the uniqueness: true it works for presence only, right?

[shioyama]

Here's the PR on Globalize: globalize/globalize#121

I don't want to do this type of hack within Mobility. Maybe create an external gem which you'd have to include, say mobility-uniqueness or something. Need to think about it.

I'm removing the "bug" label because I don't consider this a bug, but a feature request.

[kevin-jj]

Thanks for quick response.

It seems that more compatible works need to do if friendly_id , globalize, mobility gem work together.

[shioyama]

So I checked this quickly. Presence validation works fine as expected. Uniqueness does not work. Depending on the backend, it will either not work as expected, or it will raise an error (complaining that a column does not exist).

It seems that more compatible works need to do if friendly_id , globalize, mobility gem work together.

Are you using globalize and mobility together? That seems a bit strange.

About compatiblity, the only issue I see here is with uniqueness validation. Are there any other issues you are experiencing?

[kevin-jj]

Are you using globalize and mobility together? That seems a bit strange.

About compatiblity, the only issue I see here is with uniqueness validation. Are there any other issues >you are experiencing?

Yes. I'd like to look into which gem is suitable to our application.

[shioyama]

I see. Well, the uniqueness problem can be solved, as a plugin or extension to Mobility.

For now, if you want a quick fix, you can add this class somewhere in your application:

class MobilityUniquenessValidator < ::ActiveRecord::Validations::UniquenessValidator
  def validate_each(record, attribute, value)
    klass = record.class
    if klass.translated_attribute_names.include?(attribute.to_s)
      relation = klass.i18n.where(attribute => value)
      relation = relation.where.not(klass.primary_key => record.id) if record.persisted?
      relation = relation.merge(options[:conditions]) if options[:conditions]

      if relation.exists?
        error_options = options.except(:case_sensitive, :scope, :conditions)
        error_options[:value] = value

        record.errors.add(attribute, :taken, error_options)
      end
    else
      super
    end
  end
end

Now in your models, before you call validates, just add this line:

class UniquenessValidator < ::MobilityUniquenessValidator; end

This is a very basic uniqueness validation. I should work with any backend other than the serialized backend, but without the scope option.

[shioyama]

Basic uniqueness validation, including scopes, should work now using master as long as you include Mobility before adding validations to your model.

[daniel-gato]

not working in rails 7 properly.

We want the model to validate the presence of all locales (name_en, name_fr, etc)

This works:

  validates :name_en, :name_fr, presence: true

This doesn't:

  validates :name_en, :name_fr, presence: true, uniqueness: true

But this works when we use only name:

  validates :name, presence: true, uniqueness: true