Generate all translated slugs when model is saved

[shioyama]

Added a test for #10

[shioyama]

@n-rodriguez this should fix #10, can you just check it works for you? Then I'll merge and deploy it.

[n-rodriguez]

@shioyama I've just checked and it partially works. It still needs patch on should_generate_new_friendly_id? method.

[shioyama]

@shioyama I've just checked and it partially works. It still needs patch on should_generate_new_friendly_id? method.

Ah I can fix that too. Hold on I'll push another commit.

[n-rodriguez]

Ah I can fix that too. Hold on I'll push another commit.

Great!

[shioyama]

It still needs patch on should_generate_new_friendly_id? method.

Actually I'm not sure the issue with this one, can you clarify?

[n-rodriguez]

I was referring to #10 (comment)
To make the first test pass you need to uncomment should_generate_new_friendly_id? and translation_for patched methods.
To make the second test pass you need to uncomment the set_slug patched method.

[shioyama]

Ah so the problem seems to be fallbacks...

[n-rodriguez]

Ah so the problem seems to be fallbacks...

Yes

[shioyama]

Ok I added 9cb5194 which seems to fix the specs. The problem was that should_generate_new_friendly_id? was calling the slug attribute method, which was falling back so it seemed like it didn't need to regenerate the slug. By instead calling slug(locale: Mobility.locale) Mobility ignores fallbacks, which fixes the issues.

[n-rodriguez]

Awesome! It works!

[shioyama]

This has been released as 0.5.4. Thanks for your help!

[n-rodriguez]

This has been released as 0.5.4. Thanks for your help!

You're welcome! Thanks for this gems!