Patch #20506

redmine I18n autoload instead of require

Added by Ondřej Ezr about 3 years ago. Updated about 3 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 about 3 years ago

Redmine I18n autoload instead of require (#20506).

Patch by Ondřej Ezr.

History

#2 Updated by Jean-Philippe Lang about 3 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