Merge branch 'main' into rv_add_component_caching

Author: reeganviljoen

docs/CHANGELOG.md

@@ -10,6 +10,10 @@ nav_order: 5
 
 ## main
 
+* Fix bug where implicit locales in component filenames threw a `NameError`.
+
+    *Chloe Fons*
+
 * Register ViewComponent tests directory for `rails stats`.
 
     *Javier Aranda*

docs/index.md

@@ -228,6 +228,7 @@ ViewComponent is built by over a hundred members of the community, including:
 <img src="https://avatars.githubusercontent.com/allan-pires?s=64" alt="allan-pires" width="32" />
 <img src="https://avatars.githubusercontent.com/jasonkim?s=64" alt="jasonkim" width="32" />
 <img src="https://avatars.githubusercontent.com/tkowalewski" alt="tkowalewski" width="32" />
+<img src="https://avatars.githubusercontent.com/chloe-meister" alt="chloe-meister" width="32" />
 <img src="https://avatars.githubusercontent.com/zaratan" alt="zaratan" width="32" />
 
 ## Who uses ViewComponent?

lib/view_component/compiler.rb

@@ -192,7 +192,7 @@ def gather_templates
               path: path,
               lineno: 0,
               extension: path.split(".").last,
-              this_format: this_format,
+              this_format: this_format.to_s.split(".").last&.to_sym, # strip locale from this_format, see #2113
               variant: variant
             )
 

test/sandbox/app/components/localised_component.fr.html.erb

@@ -0,0 +1 @@
+<div>salut,monde!</div>

test/sandbox/app/components/localised_component.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+class LocalisedComponent < ViewComponent::Base
+end

test/sandbox/test/rendering_test.rb

@@ -1232,4 +1232,10 @@ def test_cache_component
     refute_selector(".cache-component__cache-key", text: component.cache_key)
     refute_selector(".cache-component__cache-message", text: "foo bar")
   end
+
+  def test_localised_component
+    render_inline(LocalisedComponent.new)
+
+    assert_selector("div", text: "salut,monde!")
+  end
 end