Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Try to use global language by default, if regional language is missing (EnUs is missing). #65

Closed
hubrix opened this issue Apr 22, 2013 · 4 comments
Assignees
Milestone

Comments

@hubrix
Copy link

hubrix commented Apr 22, 2013

NameError: uninitialized constant NumbersAndWords::Translations::EnUs
NameError: uninitialized constant NumbersAndWords::Strategies::FiguresConverter::Languages::EnUs

@hubrix
Copy link
Author

hubrix commented Apr 22, 2013

adding this should fix it but causes another problem

module NumbersAndWords
module Translations
class EnUs < En
end
end
module Strategies
module FiguresConverter
module Languages
class EnUs < En

    end
  end
end

end
end

NoMethodError: undefined method result' for nil:NilClass from /usr/local/Cellar/ruby/1.9.3-p194/lib/ruby/gems/1.9.1/bundler/gems/numbers_and_words-6fff2e5a6b63/lib/numbers_and_words/strategies/figures_converter/languages/en.rb:34:inmaybe_ordinal'

@kslazarev
Copy link
Owner

Why you want to use EnUs, instead just En. (default En is 'en-US')?

@eLod
Copy link
Collaborator

eLod commented Apr 24, 2013

@kslazarev i had the same thought, e.g. maybe @hubrix is just missing that with simple 'en' it is the US english (and for the fact en-GB has just some alterations on top of that), however on a second thought he is maybe forced to use the 'en-US' string, in that case it would be nice if we would just use 'en'. do you think we can make that possible with little modification?

@kslazarev
Copy link
Owner

@eLod I'll see it later, how to do that simple.

@ghost ghost assigned kslazarev Apr 29, 2013
kslazarev added a commit that referenced this issue Apr 29, 2013
…ssing (EnUs is missing). Raise error, if language not supported. Some I18n refactorings.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants