Merge pull request #850 from vlad-pisanov/vp_fix_i18n_lazy_lookup

Fix default configuration for Rails/I18nLazyLookup
rubocop · Nov 8, 2022 · 0189504 · 0189504
2 parents 7855505 + 5b54cfe
commit 0189504
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/changelog/fix_i18n_lazy_lookup_configuration.md b/changelog/fix_i18n_lazy_lookup_configuration.md
@@ -0,0 +1 @@
+* [#850](https://github.com/rubocop/rubocop-rails/pull/850): Fix default configuration for `Rails/I18nLazyLookup`. ([@vlad-pisanov][])
diff --git a/config/default.yml b/config/default.yml
@@ -540,7 +540,7 @@ Rails/I18nLazyLookup:
   Enabled: pending
   VersionAdded: '2.14'
   Include:
-    - 'controllers/**/*'
+    - 'app/controllers/**/*.rb'
 
 Rails/I18nLocaleAssignment:
   Description: 'Prefer the usage of `I18n.with_locale` instead of manually updating `I18n.locale` value.'

diff --git a/lib/rubocop/cop/rails/i18n_lazy_lookup.rb b/lib/rubocop/cop/rails/i18n_lazy_lookup.rb
@@ -34,6 +34,8 @@ class I18nLazyLookup < Base
 
         MSG = 'Use "lazy" lookup for the text used in controllers.'
 
+        RESTRICT_ON_SEND = %i[translate t].freeze
+
         def_node_matcher :translate_call?, <<~PATTERN
           (send nil? {:translate :t} ${sym_type? str_type?} ...)
         PATTERN
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		* [#850](https://github.com/rubocop/rubocop-rails/pull/850): Fix default configuration for `Rails/I18nLazyLookup`. ([@vlad-pisanov][])