search

tapajos / brazilian-rails

Brazilian Rails é um conjunto de gems para serem usadas com Ruby e com o Ruby on Rails e tem como objetivo unir alguns recursos úteis para os desenvolvedores brasileiros.
http://brazilian-rails.rubyforge.org
361 stars 100 forks source link

Remoção de atribuição desnecessária a constante FERIADOS_PATH #54

Closed douglasnomizo closed 11 years ago

douglasnomizo commented 11 years ago

Ao iniciar o servidor de uma aplicação que utiliza a biblioteca brdata sempre aparecia a mensagem de alerta:

/.rvm/gems/ruby-1.9.3-p194/gems/brdata-3.3.0/lib/brdata.rb:37: warning: already initialized constant FERIADOS_PATH

A constante recebia duas atribuições.

tinogomes commented 11 years ago

@douglasnomizo isso mesmo e não é desnecessária, pois caso a constante APP_PATH não esteja definida, a constante não será definida e vai ocorrer um erro ao tentar acessar a constante FERIADO_PATH.

talvez mudar a forma como a constante FERIADO_PATH é preenchida precisa de um refactoring, mas não remover como você fez.

FERIADO_PATH = if defined?(APP_PATH)
  File.expand_path(File.split(APP_PATH)[0] + "/feriados",  __FILE__)
else
  ""
end

Assim que der, faço esse ajuste, ou se puder, envie o patch com esta correção, pois apesar deste e-mail longo, estou sem tempo para arrumar isto agora na gem, mas obrigado pelo alerta.