Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Sanitize latinamerican spanish #33

Merged
merged 2 commits into from Dec 20, 2013

Conversation

Projects
None yet
3 participants
Contributor

bquorning commented Dec 20, 2013

#sanitize_available_locales did not work when called on the Latin America Spanish "es-419" locale:

HttpAcceptLanguage::Parser.new("").sanitize_available_locales(["es-419"])
# => ["es"]

With this patch, the method will return ["es-419"].

The start_with?("x") may be over-simplified, but so are the specs ;-)

cc/ @pinkvelociraptor @grosser

@DouweM DouweM added a commit that referenced this pull request Dec 20, 2013

@DouweM DouweM Merge pull request #33 from bquorning/sanitize-latinamerican-spanish
Support numeric language code subtags.
a24ebb1

@DouweM DouweM merged commit a24ebb1 into iain:master Dec 20, 2013

Collaborator

DouweM commented Dec 20, 2013

Thank you!

Collaborator

grosser commented Jan 22, 2014

please release to rubygems!

Collaborator

DouweM commented Jan 23, 2014

I thought I'd wait until I'd fixed #32 as well, but unfortunately I haven't had time to do that yet. I'll release tonight.

Collaborator

DouweM commented Jan 23, 2014

Well, apparently releasing was easier than I remembered :) v2.0.1 is on rubygems.

@bquorning bquorning deleted the bquorning:sanitize-latinamerican-spanish branch Jan 23, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment