Fix warning: method redefined;

[y-yagi]

This fixes the following warning:

/tmp/d20170727-7039-kmdtb1/app/app/models/user.rb:5: warning: method redefined; discarding old model_name
rails/activemodel/lib/active_model/naming.rb:222: warning: previous definition of model_name was here

[rails-bot]

r? @pixeltrix

(@rails-bot has picked a reviewer for you, use r? to override)

[lugray]

LGTM

[rafaelfranca]

Instead of removing the instance method can we override the class method that is exactly what Naming implements?

[y-yagi]

Instead of removing the instance method can we override the class method that is exactly what Naming implements?

I tried. However, it seems that the method of Naming did not use anythinge. Therefore I removed Naming extend.

[rafaelfranca]

If model_name is being used, it should do something. Naming defines a model_name at class level and a model_name at instance level that delegates to the class level, so in theory we could override model_name on the class level and keep Naming to remove the warning, or, if Naming is not needed anymore model_name should not be either.

[rafaelfranca]

I just tried to define the class method and it seems to work and the warning is not there anymore.

[y-yagi]

Thank you for the detailed explanation. I was misunderstanding.
I defines a model_name at class level.