Patch #20506

redmine I18n autoload instead of require

Added by Ondřej Ezr over 2 years ago. Updated over 2 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:Code cleanup/refactoring
Target version:3.2.0

Description

redmine/i18n is required in redmine.rb, but row above it is autoloaded by rails and thus both mechanisms are loading it. It is problem in some of my patches. But it is basically bad idea to mix those behaviours. ( My vote goes for a rails autoloading... )

redmine_i18n_loaded_automaticaly.patch Magnifier (377 Bytes) Ondřej Ezr, 2015-08-11 01:56

Associated revisions

Revision 14509
Added by Jean-Philippe Lang over 2 years ago

Redmine I18n autoload instead of require (#20506).

Patch by Ondřej Ezr.

History

#2 Updated by Jean-Philippe Lang over 2 years ago

  • Category set to Code cleanup/refactoring
  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Target version set to 3.2.0

Patch committed in r14509, thanks.

Also available in: Atom PDF