snusnu / dm-is-localizable Goto Github PK

I had a problem with datamapper 1.1.0 and dm-is-localizable 1.1.0 when destroying objects.

Every time I was destroying a record, example: Item.destroy, I was getting "ArgumentError: condition :unique_index does not map to a property or relationship in DataMapper::I18n::Locale"

After some time I found that the problem was on the translation model creation, the unique_index was set on a belongs_to instead of a property (If I'm not wrong this worked with previous versions of dm).

The solution was changing the establish_relationships https://github.com/snusnu/dm-is-localizable/blob/master/lib/dm-is-localizable/model.rb#L45 to:

translation_model.property (configuration.translated_model_belongs_to_name.to_s + "_id").to_sym, Integer, :required => true, :unique_index => :unique_locales
translation_model.property :locale_tag, String,  :required => true, :unique_index => :unique_locales
translation_model.belongs_to configuration.translated_model_belongs_to_name, :repository   => translated_model.repository.name
translation_model.belongs_to :locale, DataMapper::I18n::Locale, :repository   => DataMapper::I18n.locale_repository_name

In the ItemTranslation table you set a Index for item_id and locale_tab that gives a great performance boost, but for some reason is not being created automatically. I had to manually create these indexes.